RFR: 8290047: (fs) FileSystem.getPathMatcher does not check for ":" at last index [v5]

Brian Burkhalter bpb at openjdk.org
Tue Aug 2 17:08:05 UTC 2022


> For a `String` “s”, `s.indexOf(int)` can never return a value `>= s.length()` so change the check
> 
>         int pos = syntaxAndInput.indexOf(':');
>         if (pos <= 0 || pos == syntaxAndInput.length())
> 
> to
> 
>         if (pos <= 0)

Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

  8290047: Improve description of syntaxAndPattern

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/9595/files
  - new: https://git.openjdk.org/jdk/pull/9595/files/1af2a4b4..b9de2604

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=9595&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=9595&range=03-04

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/9595.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9595/head:pull/9595

PR: https://git.openjdk.org/jdk/pull/9595


More information about the nio-dev mailing list