RFR 8009581: Xpathexception does not honor initcause()
huizhe wang
huizhe.wang at oracle.com
Wed Jun 5 16:43:04 UTC 2013
Aleksej,
When you're ready, after fixing the remaining things as Alan commented,
please push the change to hg.openjdk.java.net/jdk8/tl-gate/jaxp.
For 7u, send a request to jdk7u-dev and indicate that the changes are
identical to that for jdk8.
Thanks,
Joe
On 6/5/2013 3:09 AM, Alan Bateman wrote:
> On 04/06/2013 16:19, Aleksej Efimov wrote:
>> Alan, Jason,
>> Thanks for clarification. I have moved the serial data from binary
>> files to test class:
>> http://cr.openjdk.java.net/~dmeetry/8009581/webrev.5/
>> <http://cr.openjdk.java.net/%7Edmeetry/8009581/webrev.5/>
>>
>> -Aleksej
>>
> Thanks, this looks much better.
>
> Just a few remaining comments on the test:
>
> - It needs a copyright header
>
> - I think twocauses and normaljdk7ser should have a comment to detail
> how the bytes are generated. One idea (and you'll see this in a few
> other tests) is to have an option to the test to generate the bytes.
> Alternatively put the code in comments so that anyone can re-generate
> them if needed. As they are constants then they can be final and
> probably should be in uppercase.
>
> Otherwise I think this is good to go.
>
> -Alan.
More information about the core-libs-dev
mailing list