[9] RFR of 8073445: (fs) FileSystem.getPathMatcher(...) should check syntax component without regard to case

Brian Burkhalter brian.burkhalter at oracle.com
Wed Mar 4 19:32:06 UTC 2015

Please review this updated patch at your convenience.


A multi-platform regression test job is in progress.



On Feb 28, 2015, at 1:14 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> On 27/02/2015 23:14, Brian Burkhalter wrote:
>> Please review at your convenience.
>> Issue:	https://bugs.openjdk.java.net/browse/JDK-8073445
>> Patch:	http://cr.openjdk.java.net/~bpb/8073445/webrev.00/
> The zip provider also needs to be updated, the changes to the default and jrt providers look good.
> The PathMatcher/Basic.java test already has a test for an unknown syntax so that might be a better place to check that you can use both regex and glob in mixed case.
>> Simply change to use the String method equalsIgnoreCase() instead of equals() on the syntax. Note that somewhat confusingly the specification in [1] states
>>> For both the glob and regex syntaxes, the matching details, such as whether the matching is case sensitive, are implementation-dependent and therefore not specified.
>> Does the proposed code change imply an update to this specification?
> This should be okay, this is talking about the resulting PathMatcher rather than the case of the syntax name specified to getPathMatcher.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20150304/ef1966d1/attachment-0001.html>

More information about the nio-dev mailing list