RFR: 8282279: Interpret case-insensitive string locale independently
Xue-Lei Andrew Fan
xuelei at openjdk.java.net
Wed Feb 23 15:43:51 UTC 2022
On Wed, 23 Feb 2022 14:51:45 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
> Locale.ROOT is preferred for locale independent uppercase and lowercase().
Yes, as the Locale.English has been used a lot. It may worthy another enhancement to use Locale.ROOT in a separate PR.
> BTW, for the strings written to the SSLLogger, I would have kept the original case intact.
and
> Looks to me like not using toUpperCase in the trace messages would be more efficient and produces shorter code.
Agreed. I will search for similar code and see if I could have a fix in a separate PR.
> 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?
It looks like a bug to me. I would like to keep this PR focus on case issues, and may file another PR for this bug.
Thanks!
-------------
PR: https://git.openjdk.java.net/jdk/pull/7583
More information about the security-dev
mailing list