RFR of 8073445: (fs) FileSystem.getPathMatcher(...) should check syntax component without regard to case
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  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...
More information about the nio-dev