Review request for 8001536 updated (original) (raw)
Alan Bateman Alan.Bateman at oracle.com
Thu Nov 1 18:03:25 UTC 2012
- Previous message: hg: jdk8/tl/langtools: 7153951: Add new lint option -Xlint:auxiliaryclass
- Next message: about JavaOne2012 Finding and Solving Java Deadlocks
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On 31/10/2012 15:08, Lance Andersen - Oracle wrote:
Here is revised webrev taking into account Remi's suggestions http://cr.openjdk.java.net/~lancea/8001536/webrev.01/
I skimmed over the updated webrev and the changes mostly look okay to me.
One comment on the clone method is that "The internal {@code Blob} field will be set to null" doesn't seem right. Shouldn't this say that the resulting object doesn't have an underlying Blob?
I don't know if you want formatting/type comments but a couple of nits:
- In both classes then it looks like the javadoc comment on equals has been shunted right by space
- In equals then "if(" seems to be missing a space between "if" and "(". There's another one in readObject.
- The spacing around "+" in hashCode is a bit odd, it doesn't matter of course but would be good to be consistent
- the first line of both readObject has also been shunted right by one space.
-Alan.
- Previous message: hg: jdk8/tl/langtools: 7153951: Add new lint option -Xlint:auxiliaryclass
- Next message: about JavaOne2012 Finding and Solving Java Deadlocks
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]