The fix looks good to me.

 Thanks,
 Alexandr.

On 8/5/2014 1:24 PM, Sergey Bylokhov wrote:
On 05.08.2014 13:06, Alexander Scherbatiy wrote:
On 8/5/2014 1:00 PM, Sergey Bylokhov wrote:
Hi, Alexander.
No, it covers some warnings inside inner classes.

  Could the annotation be moved to the inner classes?
Yes, we can.
New version:
http://cr.openjdk.java.net/~serb/6521727/webrev.01

  Thanks,
  Alexandr.

On 05.08.2014 12:54, Alexander Scherbatiy wrote:

Should the @SuppressWarnings("serial") annotation be also removed?

Thanks,
Alexandr.

On 8/4/2014 4:00 PM, Sergey Bylokhov wrote:
Hello.
Please review a small fix for jdk9.

Bug: https://bugs.openjdk.java.net/browse/JDK-6521727
Webrev can be found at: http://cr.openjdk.java.net/~serb/6521727/webrev.00








Reply via email to