RFR: 8287743: javax/swing/text/CSSBorder/6796710/bug6796710.java failed [v4]

Prasanta Sadhukhan psadhukhan at openjdk.java.net
Fri Jun 10 13:11:16 UTC 2022


On Fri, 10 Jun 2022 12:54:23 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Fix review comments
>>  - Fix review comments
>
> test/jdk/javax/swing/text/CSSBorder/6796710/bug6796710.java line 163:
> 
>> 161:                 if ((Math.abs(red1 - red2) > COLORTOLERANCE) ||
>> 162:                     (Math.abs(green1 - green2) > COLORTOLERANCE) ||
>> 163:                     (Math.abs(blue1 - blue2) > COLORTOLERANCE)) {
> 
> Are binary operators usually wrapped to the next line or left on the same line?
> 
> Which style is the preferred one?

I have seen the present style more common.

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

PR: https://git.openjdk.org/jdk/pull/9011



More information about the client-libs-dev mailing list