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