RFR: 8274755: Replace 'while' cycles with iterator with enhanced-for in jdk.jdi [v3]
Serguei Spitsyn
sspitsyn at openjdk.java.net
Thu Oct 7 05:17:05 UTC 2021
On Wed, 6 Oct 2021 19:00:35 GMT, Andrey Turbanov <github.com+741251+turbanoff at openjdk.org> wrote:
>> There are few places in code where manual while loop is used with Iterator to iterate over Collection.
>> Instead of manual while 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: java compiler generates similar code when compiling enhanced-for cycle.
>>
>> Similar cleanups:
>> * https://bugs.openjdk.java.net/browse/JDK-8258006
>> * https://bugs.openjdk.java.net/browse/JDK-8257912
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
>
> 8274755: Replace 'while' cycles with iterator with enhanced-for in jdk.jdi
> remove empty space before casts in InvokableTypeImpl too
Marked as reviewed by sspitsyn (Reviewer).
We had HotSpot convention about it. Not sure where it is located now.
One of the important rules was to follow the file (area) local style if possible.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5577
More information about the serviceability-dev
mailing list