JAXP repo warnings fixed

[huizhe wang]

Hi Ivan,

I really appreciate you and others for proactively contributing to the 
jaxp code.  Unfortunately, I have to say sorry that it may be not 
feasible to apply this patch. The reason is that the patch covers just 
the BCEL component, but the upstream source had been updated already. 
The proper way to fix these warnings is therefore getting the updates 
from the upstream source.

As described here http://openjdk.java.net/contribute/, it would be nice 
if you could discuss your intended change within the alias before 
investing significant time working on it (I saw that you did list three 
bugs that you intended to work on).

Thanks,
Joe

On 3/3/2015 9:09 PM, huizhe wang wrote:
>
> On 3/3/2015 6:21 PM, Joseph D. Darcy wrote:
>> Hi Ivan,
>>
>> I've skimmed over the patch and didn't see anything technically 
>> amiss. However, I believe at least some of the changes, such as 
>> adding @Deprecated and the generification updates will need to go 
>> through an internal review process before they can be pushed.
>>
>> Joe Wang (cc'ed as huizhe.wang at oracle.com) is the jaxp lead and he 
>> will also need to review of the changes for jaxp-specific concerns.
>
> Just coming back from a two-week vacation, I have a lot of catch-ups 
> to do. Please allow me some time to get to this.
>
> Thanks,
> Joe
>
>>
>> Thanks,
>>
>> -Joe
>>
>> On 3/3/2015 10:06 AM, Ivan St. Ivanov wrote:
>>> Hi Joe,
>>>
>>> The links are correct, but our server was down for a while. It is 
>>> available now.
>>>
>>> Regards,
>>> Ivan
>>>
>>> On Tue, Mar 3, 2015 at 2:05 AM, Joseph D. Darcy 
>>> <joe.darcy at oracle.com <mailto:joe.darcy at oracle.com>> wrote:
>>>
>>>     Hello,
>>>
>>>     Sorry for the belated reply, I tried the URLs and some obvious
>>>     variations, but the webrevs didn't come up as being there.
>>>
>>>     Have they been moved?
>>>
>>>     Thanks,
>>>
>>>     -Joe
>>>
>>>     On 2/18/2015 12:17 PM, Ivan St. Ivanov wrote:
>>>
>>>         Hi Joe, core libs developers,
>>>
>>>         Last night we had a hackathon in Bulgarian Java User Group.
>>>         Its purpose was to work on fixing some compiler warnings in
>>>         OpenJDK. The day before the meeting we ran the following 
>>> command:
>>>
>>>         make clean
>>> JAVAC_WARNINGS="-Xlint:all,deprecation,rawtypes,unchecked,cast,serial,dep-ann,static,fallthrough,try,varargs,empty,finally
>>>         -Xmaxwarns 10000" DISABLE_WARNINGS="-Xlint:all" LOG=info images
>>>
>>>         After that, looking at the build.log, we identified the
>>>         following number of warnings per repo left to be fixed:
>>>
>>>         359 corba
>>>         100 jaxp
>>>         500 jaxws
>>>         0 nashorn
>>>
>>>         We decided to take the jaxp repository, where the situation
>>>         was the following:
>>>
>>>         rawtypes 43
>>>         unchecked 14
>>>         serial 32
>>>         cast 10
>>>         dep-ann 1
>>>
>>>         The latter three types were easy to fix. After we did them,
>>>         we created our first webrev for the night:
>>>
>>> http://bgjug.sty.bz/bgjug/~bgjug/fix-warnings-jaxp-part1/webrev.00/
>>> <http://bgjug.sty.bz/bgjug/%7Ebgjug/fix-warnings-jaxp-part1/webrev.00/>
>>> <http://bgjug.sty.bz/bgjug/%7Ebgjug/fix-warnings-jaxp-part1/webrev.00/>
>>>
>>>         Later we continued with rawtypes and unchecked. We didn't fix
>>>         them all, as some of the warnings concerned public method
>>>         parameters, which change might virtually break client code.
>>>         The good news here is that all these public methods belong to
>>>         classes located in 'internal' packages. Which we hope will
>>>         not be public any more once project Jigsaw meets java.xml
>>>         module :)
>>>
>>>         Anyway, this is the second webrev that we created:
>>>
>>> http://bgjug.sty.bz/bgjug/~bgjug/fix-warnings-jaxp-full/webrev.00/
>>> <http://bgjug.sty.bz/bgjug/%7Ebgjug/fix-warnings-jaxp-full/webrev.00/>
>>> <http://bgjug.sty.bz/bgjug/%7Ebgjug/fix-warnings-jaxp-full/webrev.00/>
>>>
>>>         As its name implies, it contains all the fixes that we did
>>>         last night.
>>>
>>>         We would be happy to get feedback from you and even happier
>>>         if this becomes our first contribution to OpenJDK as part of
>>>         our adoption efforts.
>>>
>>>         Regards,
>>>         Ivan, Bulgarian JUG
>>>
>>>
>>>
>>
>