RFR: 8274318: Replace 'for' cycles with iterator with enhanced-for in java.management [v2]
Andrey Turbanov
github.com+741251+turbanoff at openjdk.java.net
Fri Oct 1 21:12:57 UTC 2021
> There are a few places in code, where manual `for` loop is used with Iterator to iterate over Collection.
> Instead of manual `for` cycles, it's preferred to use enhanced-for cycle instead: it's less verbose, makes code easier to read and it's less error-prone.
> It doesn't have any performance impact: javac compiler generates similar code when compiling enhanced-for cycle.
> Similar cleanups:
> 1. [JDK-8274016](https://bugs.openjdk.java.net/browse/JDK-8274016) java.desktop
> 2. [JDK-8274237](https://bugs.openjdk.java.net/browse/JDK-8274237) java.base
> 3. [JDK-8274261](https://bugs.openjdk.java.net/browse/JDK-8274261) jdk.jcmd
Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
8274318: Replace 'for' cycles with iterator with enhanced-for in java.management
small code-style fixes
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/5695/files
- new: https://git.openjdk.java.net/jdk/pull/5695/files/e3884614..263e8682
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5695&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5695&range=00-01
Stats: 7 lines in 1 file changed: 0 ins; 3 del; 4 mod
Patch: https://git.openjdk.java.net/jdk/pull/5695.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/5695/head:pull/5695
PR: https://git.openjdk.java.net/jdk/pull/5695
More information about the serviceability-dev
mailing list