JAXP repo warnings fixed

huizhe wang huizhe.wang at oracle.com
Wed Mar 4 05:09:22 UTC 2015


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
>>
>>
>>
>




More information about the core-libs-dev mailing list