RFR: 8267587: Update java.util to use enhanced switch [v3]

Tagir F.Valeev tvaleev at openjdk.java.net
Tue May 25 06:06:05 UTC 2021


On Tue, 25 May 2021 03:48:41 GMT, Tagir F. Valeev <tvaleev at openjdk.org> wrote:

>> src/java.base/share/classes/java/util/regex/CharPredicates.java line 217:
>> 
>>> 215:             case "WORD" -> WORD();
>>> 216:             default -> null;
>>> 217:         };
>> 
>> This file has lots of changes which are difficult to review. Maybe it should be split out of this PR.
>
> *sigh* GitHub diff tool is really poor and outdated. Here's how it looks in IntelliJ IDEA diff view:
> ![image](https://user-images.githubusercontent.com/5114450/119436474-6b455b80-bd46-11eb-8865-8b7f30826a8d.png)

I played with indentations and found a way to make GitHub diff happier. Now, lines like `? UPPERCASE().union(LOWERCASE(), TITLECASE())` are probably a little bit too far to the right but it's still acceptable and diff looks much easier to review now.

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

PR: https://git.openjdk.java.net/jdk/pull/4161



More information about the security-dev mailing list