Code review request: 7077389 Reflection classes do not build with javac -Xlint:all -Werror
Joe Darcy
joe.darcy at oracle.com
Wed Aug 10 19:39:46 UTC 2011
On 8/10/2011 11:36 AM, Alexandre Boulgakov wrote:
> Hello Joe,
>
> Could you please review these small changes to remove javac build
> warnings from the reflection classes?
>
> Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7077389
> webrev: http://cr.openjdk.java.net/~mduigou/7077389/0/webrev/
> <http://cr.openjdk.java.net/%7Emduigou/7077389/0/webrev/>
>
> I haven't updated the copyrights to avoid polluting the webrev.
> I did not add JAVAC_WARNINGS_FATAL=true to the makefile because it is
> responsible for other packages which still have build warnings.
>
> Thanks,
> Sasha
Hi Sasha.
I approve the code changes with the following exceptions:
src/share/classes/sun/reflect/UnsafeFieldAccessorImpl.java
Offhand, I'm not sure of the semantics of this change.
While adding serialVersionUIDs in sun/reflect/annotation/* is a good
change, I believe this interacts with another bug I'm working on so
please hold off on those changes for now.
Thanks,
-Joe
More information about the core-libs-dev
mailing list