RFR: 7902982: jcstress: Add samples for some mutex algorithms [v4]

[Aleksey Shipilev]

On Tue, 29 Jun 2021 11:57:26 GMT, Michael Mirwaldt <github.com+6693355+mmirwaldt at openjdk.org> wrote:

>> jcstress-samples/src/main/java/org/openjdk/jcstress/samples/concurreny/mutex/Mutex_02_PetersonAlgorithm.java line 64:
>> 
>>> 62:             flagForActor1.set(true);
>>> 63:             turn.set(2);
>>> 64:             while (flagForActor2.get() && turn.get() == 2) ;
>> 
>> Here and later. Excess whitespace before ` ;`?
>
> I didn't add the whitespace on purpose. IntelliJ added them when I applied code reformatting.
> I wouldn't change it because I guess most developers who use IntelliJ just use the default code format.
> And if they apply code formatting  (as I use all the time), the whitespace will accidentally added again anyway.
> What do you think about that?

That formatting rule makes little sense, IMO. Should be:

while (flagForActor2.get() && turn.get() == 2); // wait

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

PR: https://git.openjdk.java.net/jcstress/pull/85