Re: Core review request for use of java.util.Object.equals in net and beans

2011-05-02 Thread Mike Duigou
Both patches look fine to me. Mike On May 2 2011, at 02:13 , Joe Darcy wrote: > Hello. > > Please review the patch below to replace several local two-argument equals > methods with the JDK 7 library method java.util.Objects.equals. A full build > is successful and the regression test results

Re: Core review request for use of java.util.Object.equals in net and beans

2011-05-02 Thread Alan Bateman
Joe Darcy wrote: Hello. Please review the patch below to replace several local two-argument equals methods with the JDK 7 library method java.util.Objects.equals. A full build is successful and the regression test results are good. Looks fine to me. -Alan.

Core review request for use of java.util.Object.equals in net and beans

2011-05-02 Thread Joe Darcy
Hello. Please review the patch below to replace several local two-argument equals methods with the JDK 7 library method java.util.Objects.equals. A full build is successful and the regression test results are good. For reference, the code of the library method is public static boolean eq