Hi Aleksej,
First of all, I'm not a reviewer for 8u.
The test is fine in proving that no NPE is thrown, which indicates the
2nd nodeset is not null. However, there's a 2nd aspect of this: it
should be empty. I suggest improving the test so that it further proves
that it's not only "not null" but indeed empty. To do that, you may
change the 1st nodeset to return meaningful result to show that the
leading function returns the 1st set, which proves that the 2nd is empty.
Thanks,
Joe
On 12/3/2015 4:25 PM, Aleksej Efimov wrote:
Hi,
Please, help to review and approve JDK-8133924 backport to JDK8. The
source fix is identical to JDK9 changes, but there was no test added
for this issue in JDK9.
The existing
jdk/test/javax/xml/jaxp/transform/8062518/XSLTFunctionsTest.java test
was modified to test the reported problem in JDK8 and it is a subject
of review:
http://cr.openjdk.java.net/~aefimov/8133924/8/00
No regression tests failures were observed for the proposed fix in
JDK8 + the fix is in JDK9 for 3 months already.
With Best Regards,
Aleksej
JBS:
https://bugs.openjdk.java.net/browse/JDK-8133924
JDK9 changeset:
http://hg.openjdk.java.net/jdk9/dev/jaxp/rev/7c045de70a66
JDK9 Review thread:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-August/034969.html
