On 11/21/13 4:58 AM, Patrick Zhang wrote:
Hi Everyone,

I am working on https://bugs.openjdk.java.net/browse/JDK-8027973. The
problem is caused by wrong URL format on windows. file://c:\xxx is one
invalid schema.

Solution:
Replace "file://" with "file:///" then it will work on windows and it
will not impact other platforms. And remove the test from ProblemList.txt

Hi Patrick,

I didn't know windows needed a triple '/' - I think I learned something
today ;-)

May I suggest that you add the description of the problem & solution
(the text you wrote above) in comment in the test?
It may save a lot of time to future maintainers - I think I would
have had a hard time to figure that out by myself :-)

best regards,

-- daniel


webrev:
http://cr.openjdk.java.net/~pzhang/8027973/webrev/

Result on windows:
http://cr.openjdk.java.net/~pzhang/8027973/XSLTExFuncTest.jtr
Result on Linux:
http://cr.openjdk.java.net/~pzhang/8027973/XSLTExFuncTest.linux.jtr

Regards
Patrick

Reply via email to