RFR: 8264666: Change implementation of safeAdd/safeMult in the LCMSImageLayout class [v2]

Sergey Bylokhov serb at openjdk.java.net
Fri Apr 22 07:36:43 UTC 2022


> - The hand-crafted methods for addition and multiplication are replaced by the "Math" versions.
> - Cleanup: the usage of do/while(false) is removed

Sergey Bylokhov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:

 - Initial version 8264666
 - Merge branch 'master' into LCMSImageLayout
 - Revert "safeXX -> xxExact"
   
   This reverts commit a1876fa596a6831f036c04f45fa89c2caba47087.
 - Revert "delete "do{...} while (false);""
   
   This reverts commit a9e91601c355f96c82dd8b2b8564a3a3e7b96aef.
 - delete "do{...} while (false);"
 - safeXX -> xxExact

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/3333/files
  - new: https://git.openjdk.java.net/jdk/pull/3333/files/a9e91601..b183980c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3333&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3333&range=00-01

  Stats: 2331707 lines in 19244 files changed: 1311618 ins; 893703 del; 126386 mod
  Patch: https://git.openjdk.java.net/jdk/pull/3333.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/3333/head:pull/3333

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



More information about the client-libs-dev mailing list