[OpenJDK 2D-Dev] 8017109: Cleanup overrides warning in src/solaris/classes/sun/print/AttributeClass.java

[Kurchi Hazra]

Hi,

   The class src/solaris/classes/sun/print/AttributeClass.java overrides 
Object.equals() but not Object.hashCode() because of which
we get a warning everytime we build jdk8/tl. Here is a suggestion to 
remove the warning:

http://cr.openjdk.java.net/~khazra/8017109/webrev.00/ 
<http://cr.openjdk.java.net/%7Ekhazra/8017109/webrev.00/>

I was also wondering whether the implementation of the equals() method 
is correct, and whether it is instead meant to be:

diff -r eb1a3c50a2a9 src/solaris/classes/sun/print/AttributeClass.java
--- a/src/solaris/classes/sun/print/AttributeClass.java Tue Jun 18 
14:11:45 2013 -0700
+++ b/src/solaris/classes/sun/print/AttributeClass.java Wed Jun 19 
14:20:27 2013 -0700
@@ -250,9 +250,8 @@

      public boolean equals(Object obj) {
          return
-            obj != null &&
              obj instanceof AttributeClass &&
-            obj.toString().equals (((AttributeClass) obj).toString());
+            toString().equals (((AttributeClass) obj).toString());
      }

      public String toString() {

What is the preferable way of getting this change into the code - I can 
push via tl. What are the tests I would need to run?

If this discussion belongs elsewhere, let me know.

Thanks for your help,
- Kurchi