On 9/13/11 10:55 PM, Weijun Wang wrote: > I apply the patch to my local repository and do a clean rebuild of > jdk-only. It shows 1 error and 92 warnings in javax and stopped. Most in > src/share/classes/javax/xml/crypto/dsig and I remember Sean said it's > not easy to remove all warnings there because the codes are shared > between JDK and some Apache projects.
The APIs in javax.xml.crypto.dsig and subpackages are defined by JSR 105, which is a standalone JSR. I am fairly sure we can't eliminate warnings that involve API changes (such as generifying the APIs) without first going through a maintenance release of the JSR. It is possible that some of the warnings can be fixed, but my guess is most of them involve generics. So my recommendation to Kurchi was to defer fixing the warnings there until we have resources to do a maintenance release of JSR 105. --Sean > > -Max > > On 09/14/2011 03:13 AM, Alan Bateman wrote: >> Kurchi Hazra wrote: >>> Something went wrong in the pasting. Can you check if this works fine: >>> http://cr.openjdk.java.net/~chegar/7090158/webrev/ >> Yes, this webrev has what I expected to see. >> >> -Alan