<i18n dev> RFR: 8302815 Use new Math.clamp method in core libraries [v3]

Tagir F. Valeev tvaleev at openjdk.org
Tue Feb 21 20:39:53 UTC 2023


> For cleanup and dogfooding the new method, it would be nice to use Math.clamp where possible in java.base. See PR #12428.
> 
> As Math.clamp performs an additional check that min is not greater than max, I conservatively replaced only those occurrences where I can see that this invariant is always held. There are more occurrences, where clamp can be potentially used but it's unclear whether min <= max is always true.

Tagir F. Valeev has updated the pull request incrementally with one additional commit since the last revision:

  Update copyright year

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12633/files
  - new: https://git.openjdk.org/jdk/pull/12633/files/be13683b..e3fb1502

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12633&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12633&range=01-02

  Stats: 8 lines in 8 files changed: 0 ins; 0 del; 8 mod
  Patch: https://git.openjdk.org/jdk/pull/12633.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12633/head:pull/12633

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


More information about the i18n-dev mailing list