On Sat, 26 Nov 2022 18:19:00 GMT, Nir Lisker <[email protected]> wrote:
>> John Hendrikx has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - Revert changes to gesture recognizers to be addressed in follow up bug
>> - Fix incorrect cast removals
>
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/monocle/LinuxStatefulMultiTouchProcessor.java
> line 42:
>
>> 40:
>> 41: private final Map<Integer, Integer> slotToIDMap =
>> 42: new HashMap<>();
>
> Same line. There are more places where this formatting can be fixed, but it's
> up to you.
I'll only fix the lines that were touched, as there are many many places that
could be written more succinct.
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/win/WinHTMLCodec.java
> line 157:
>
>> 155: htmlPrefix = "<HTML>";
>> 156: htmlSuffix = "</HTML>";
>> 157: if( -1 == stUpContext.indexOf("<BODY") ) {
>
> Might as well fix the spacing for `if`.
I've left this as is (even though I agree this is ugly), as I didn't touch that
line. Fixing these issues is a slippery slope :)
-------------
PR: https://git.openjdk.org/jfx/pull/960
