JAXP repo warnings fixed

[Ivan St. Ivanov]

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/

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/

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