#192: Remvove 'obj == this' from equals methods.

Signed-off-by: Franz Wilhelmstötter <franz.wilhelmstoetter@gmail.com>

Author: jenetics

jpx/src/main/java/io/jenetics/jpx/Bounds.java

@@ -117,8 +117,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof  Bounds bounds &&
+		return obj instanceof  Bounds bounds &&
 			Objects.equals(bounds._minLatitude, _minLatitude) &&
 			Objects.equals(bounds._minLongitude, _minLongitude) &&
 			Objects.equals(bounds._maxLatitude, _maxLatitude) &&
@@ -140,7 +139,6 @@ public String toString() {
 	 * Return a collector which calculates the bounds of a given way-point
 	 * stream. The following example shows how to calculate the bounds of all
 	 * track-points of a given GPX object.
-	 *
 	 * {@snippet lang="java":
 	 * final Bounds bounds = gpx.tracks()
 	 *     .flatMap(Track::segments)

jpx/src/main/java/io/jenetics/jpx/Copyright.java

@@ -103,8 +103,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof Copyright cr &&
+		return obj instanceof Copyright cr &&
 			Objects.equals(cr._author, _author) &&
 			Objects.equals(cr._year, _year) &&
 			Objects.equals(cr._license, _license);

jpx/src/main/java/io/jenetics/jpx/DGPSStation.java

@@ -120,8 +120,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof DGPSStation dgps &&
+		return obj instanceof DGPSStation dgps &&
 			dgps._value == _value;
 	}
 

jpx/src/main/java/io/jenetics/jpx/Email.java

@@ -105,8 +105,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof Email email &&
+		return obj instanceof Email email &&
 			Objects.equals(email._id, _id) &&
 			Objects.equals(email._domain, _domain);
 	}

jpx/src/main/java/io/jenetics/jpx/GPX.java

@@ -472,8 +472,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof GPX gpx &&
+		return obj instanceof GPX gpx &&
 			Objects.equals(gpx._creator, _creator) &&
 			Objects.equals(gpx._version, _version) &&
 			Objects.equals(gpx._metadata, _metadata) &&

jpx/src/main/java/io/jenetics/jpx/Length.java

@@ -192,8 +192,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof Length lng &&
+		return obj instanceof Length lng &&
 			Double.compare(lng._value, _value) == 0;
 	}
 

jpx/src/main/java/io/jenetics/jpx/Link.java

@@ -101,8 +101,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof Link link &&
+		return obj instanceof Link link &&
 			Objects.equals(link._href, _href) &&
 			Objects.equals(link._text, _text) &&
 			Objects.equals(link._type, _type);

jpx/src/main/java/io/jenetics/jpx/Metadata.java

@@ -270,8 +270,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof Metadata meta &&
+		return obj instanceof Metadata meta &&
 			Objects.equals(meta._name, _name) &&
 			Objects.equals(meta._description, _description) &&
 			Objects.equals(meta._author, _author) &&

jpx/src/main/java/io/jenetics/jpx/NonNullList.java

@@ -292,7 +292,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object o) {
-		return o == this || _adoptee.equals(o);
+		return o instanceof List<?> list && _adoptee.equals(list);
 	}
 
 }

jpx/src/main/java/io/jenetics/jpx/Person.java

@@ -116,8 +116,7 @@ public int hashCode() {
 
 	@Override
 	public boolean equals(final Object obj) {
-		return obj == this ||
-			obj instanceof Person person &&
+		return obj instanceof Person person &&
 			Objects.equals(person._name, _name) &&
 			Objects.equals(person._email, _email) &&
 			Objects.equals(person._link, _link);