RFR 7156085: ArrayIndexOutOfBoundsException throws in UTF8Reader of SAXParser
Martin Buchholz
martinrb at google.com
Wed Oct 29 22:59:00 UTC 2014
On Wed, Oct 29, 2014 at 2:37 PM, huizhe wang <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/
>
>
Your re-shuffling of files in your webrev looks fine. Feel free to commit
this change, (or I can do it)
> 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.
More information about the core-libs-dev
mailing list