RFR 7156085: ArrayIndexOutOfBoundsException throws in UTF8Reader of SAXParser

huizhe wang huizhe.wang at oracle.com
Wed Oct 29 23:09:00 UTC 2014


On 10/29/2014 3:59 PM, Martin Buchholz wrote:
>
>
> On Wed, Oct 29, 2014 at 2:37 PM, huizhe wang <huizhe.wang at oracle.com 
> <mailto:huizhe.wang at oracle.com>> wrote:
>
>     Hi Martin,
>
>     You're welcome in the jaxp land :-)   Bandaid is what we need for
>     now. There are many things we would like to do to this aging code
>     base, unfortunately, we have to focus on a few.
>
>     The change looks okay. I did a full-test run and saw no failures.
>
>     One thing to note is that the SQE team has been working on
>     migrating jaxp unit and functional tests into the jaxp repo.
>     Before that's done, we should continue putting unit tests in
>     jdk/test. I've moved your test to jdk/test and also removed a few
>     unused imports.
>
>     http://cr.openjdk.java.net/~joehw/jdk9/7156085/webrev/
>     <http://cr.openjdk.java.net/%7Ejoehw/jdk9/7156085/webrev/>
>
>
> Your re-shuffling of files in your webrev looks fine. Feel free to 
> commit this change, (or I can do it)

Please do.

>     I saw that you included a license header in the test that had a
>     structure like: Google copyright + GNU GPL. I saw that you've
>     already done it before (e.g. JDK-8058520). I don't have a problem
>     with the lic header itself. But I'd like to know that the License
>     Structure was approved by legal. Has it been approved?
>
>
> yes, this is what new files contributed by Google have looked like for 
> many years.

Awesome!

Thanks,
Joe





More information about the core-libs-dev mailing list