On 2020-06-17 07:43, Alan Bateman wrote:
On 17/06/2020 14:47, Erik Joelsson wrote:


I've tried to make this clearer in the comment.

http://cr.openjdk.java.net/~erikj/8213214/webrev.02/
I would prefer if you could move the notes/comment from L125-157 to the bug report to avoid cluttering the test. Instead, the test just needs a one line comment to say that toRealPath does not work with environments using the legacy subst mechanism. In the mean-time, I think Brian is trying to page in list of consistency issues that adding support for subst introduces.

Ah, misunderstood you then. I added a comment to JDK-8213216 with more details and shortened the comment in the test:

http://cr.openjdk.java.net/~erikj/8213214/webrev.03/

/Erik

Reply via email to