<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</head>
<body>
<div dir="ltr">
<div></div>
<div>
<div dir="ltr">The last replace seems a bit strange, I would expect it should first normalize the case and then the hyphen, otherwise it won’t match the replace?</div>
<div dir="ltr"><br>
</div>
<div dir="ltr">Looks to me like not using toUpperCase in the trace messages would be more efficient and produces shorter code.</div>
<div dir="ltr"><br>
</div>
<div dir="ltr">Isn’t it customary to use the ROOT locale instead of ENGLISH?</div>
<div dir="ltr">Gruss</div>
<div dir="ltr">Bernd</div>
</div>
<div id="ms-outlook-mobile-signature">
<div style="direction:ltr">-- </div>
<div style="direction:ltr">http://bernd.eckenfels.net</div>
</div>
</div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>Von:</b> security-dev <security-dev-retn@openjdk.java.net> im Auftrag von Xue-Lei Andrew Fan <xuelei@openjdk.java.net><br>
<b>Gesendet:</b> Wednesday, February 23, 2022 1:12:16 AM<br>
<b>An:</b> security-dev@openjdk.java.net <security-dev@openjdk.java.net><br>
<b>Betreff:</b> RFR: 8282279: Interpret case-insensitive string locale independently</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">The String.toUpperCase() or String.toLowerCase() method is locale sensitive, and may produce unexpected results if used for strings that are intended to be interpreted locale independently. The use of the two methods had been cleaned
before, but there are a few new introduced laterly.<br>
<br>
See <a href="https://www.ivi.co/java/2011/07/07/4878.html">https://www.ivi.co/java/2011/07/07/4878.html</a> for examples.<br>
<br>
-------------<br>
<br>
Commit messages:<br>
- 8282279: Interpret case-insensitive string locale independently<br>
<br>
Changes: <a href="https://git.openjdk.java.net/jdk/pull/7583/files">https://git.openjdk.java.net/jdk/pull/7583/files</a><br>
Webrev: <a href="https://webrevs.openjdk.java.net/?repo=jdk&pr=7583&range=00">https://webrevs.openjdk.java.net/?repo=jdk&pr=7583&range=00</a><br>
Issue: <a href="https://bugs.openjdk.java.net/browse/JDK-8282279">https://bugs.openjdk.java.net/browse/JDK-8282279</a><br>
Stats: 18 lines in 5 files changed: 5 ins; 0 del; 13 mod<br>
Patch: <a href="https://git.openjdk.java.net/jdk/pull/7583.diff">https://git.openjdk.java.net/jdk/pull/7583.diff</a><br>
Fetch: git fetch <a href="https://git.openjdk.java.net/jdk">https://git.openjdk.java.net/jdk</a> pull/7583/head:pull/7583<br>
<br>
PR: <a href="https://git.openjdk.java.net/jdk/pull/7583">https://git.openjdk.java.net/jdk/pull/7583</a><br>
</div>
</span></font></div>
</body>
</html>