Overall, seem OK to me. Two comments:
-is the change in make/CompileJavaModules.gmk really needed given the adjusted JAVAC_WARNINGS in make/common/SetupJavaCompilers.gmk?

-the reliance on literal "true" in TypeEnter seems suspicious to me. What if the value of the attribute is a compile-time constant? I suspect this may be non-trivial to fix, so no strict need to fix that under this patch, but I think we should have a good idea how we want this fixed, and there should be a JBS entry filled for that.


On 18.10.2016 00:10, Jonathan Gibbons wrote:
Repeat, with subject line.

-- Jon

On 10/17/2016 03:03 PM, Jonathan Gibbons wrote:
Compiler folk, build folk,

Please review this update for javac and some build files, for JEP 277.

The work supports the enhanced Deprecation attribute, and generates a
new warning when items are referenced that are declared with
@Deprecated(forRemoval=true), in line with the proposals in JEP 277.

The warnings are on by default, and can be suppressed with
-Xlint:-removal or @SuppressWarnings("removal").  Because the warnings
are on by default, and because the warnings currently show up when
building JDK, some minor build changes are temporarily required to
suppress the warnings that get generated during the build.

-- Jon

JEP: http://openjdk.java.net/jeps/277
JBS: https://bugs.openjdk.java.net/browse/JDK-8145471
Webrev: http://cr.openjdk.java.net/~jjg/8145471/webrev.01/

Reply via email to