RFR (JDK10/jaxp) 8181154: Fix lint warnings in JAXP repo: deprecation

Joe Wang huizhe.wang at oracle.com
Wed Jul 5 18:48:08 UTC 2017


Hi,

This patch fixes deprecation warnings in the jaxp repo. I'm limiting the 
changes to deprecation or the 102 classes. The cup-generated classes, 
XPath*.java, could use some cleanup or even rewrite, but that will be 
left in future works (JDK-8183580).

Most of the changes are straight-forward, but I did have trouble with 
the replacing clazz.newInstance() with 
clazz.getDeclaredConstructor().newInstance() since in some cases I got 
RuntimePermission problems. So for most of such cases, I've replaced 
them with getConstructor() instead, thanks Alan for the suggestion. For 
specific cases such as xsltc/compiler/Parser.java, I'm using 
getDeclaredConstructor() because some of the classes didn't have a 
public constructor.

Again, please bear with me for the cup-generated classes that have super 
long lines (e.g. XPathParser.java). Please read with Udiffs instead of 
Sdiffs.

Some of the boxing (e.g. Integer.valueOf()) in the BCEL classes may not 
be necessary, but that's how they look in BCEL to which an update is 
scheduled in JDK 10.

All tests passed.

JBS: https://bugs.openjdk.java.net/browse/JDK-8181154
webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8181154/webrev/

Thanks
Joe



More information about the core-libs-dev mailing list