Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Andy Goryachev
On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Kevin Rushforth
On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Kevin Rushforth
On Fri, 29 Jul 2022 07:40:42 GMT, Marius Hanl wrote: >> i think *now* the code is clear without the comment! >> (and the comment does not explain *why* it is intentional or what is the >> intention) > > I would probably leave it as before. I found that code already when working > on the OpenJFX

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Phil Race
On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Nir Lisker
On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Andy Goryachev
On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-29 Thread Marius Hanl
On Tue, 26 Jul 2022 22:59:26 GMT, Andy Goryachev wrote: >> modules/javafx.graphics/src/main/java/com/sun/javafx/util/Utils.java line >> 875: >> >>> 873: public static boolean assertionEnabled() { >>> 874: boolean assertsEnabled = false; >>> 875: assert (assertsEnabled = true

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-26 Thread Nir Lisker
On Tue, 26 Jul 2022 22:54:56 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-26 Thread Andy Goryachev
On Tue, 26 Jul 2022 22:43:36 GMT, Kevin Rushforth wrote: >> Andy Goryachev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - 8289397: review comments >> - Revert "8289397: functional equivalent" >> >>This reverts commit 6ae0ef60

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-26 Thread Kevin Rushforth
On Tue, 26 Jul 2022 22:50:49 GMT, Andy Goryachev wrote: >> - replaced with exact functional equivalent (in the presence of exceptions, >> for example) > > Andy Goryachev has updated the pull request incrementally with two additional > commits since the last revision: > > - 8289397: review com

Re: RFR: 8289397: Fix warnings: Possible accidental assignment in place of a comparison. A condition expression should not be reduced to an assignment [v2]

2022-07-26 Thread Andy Goryachev
> - replaced with exact functional equivalent (in the presence of exceptions, > for example) Andy Goryachev has updated the pull request incrementally with two additional commits since the last revision: - 8289397: review comments - Revert "8289397: functional equivalent" This reverts