Re: RFR: JDK-8303897 ObservableValue's when binding should only invalidate when strictly needed [v3]

2023-04-11 Thread John Hendrikx
On Tue, 11 Apr 2023 01:36:48 GMT, Nir Lisker wrote: > > Well, I am just following the specification here, but JavaFX itself isn't... > > We may need to make a decision here. JavaFX is not consistent itself where > > it applies `equals` and where it uses reference equality: > > Alright, so this

Re: RFR: JDK-8224260: ChangeListener not triggered when adding a new listener in invalidated method [v2]

2023-04-11 Thread John Hendrikx
> Fixes three issues in ExpressionHelper: > > - Current Value was not retained when changing from SingleChange to Generic, > this can lead to missed changes > - Current Value was not retained when changing from Generic to SingleChange, > this can lead to missed changes > - Current Value was not

Re: RFR: JDK-8224260: ChangeListener not triggered when adding a new listener in invalidated method [v2]

2023-04-11 Thread Nir Lisker
On Tue, 11 Apr 2023 07:57:56 GMT, John Hendrikx wrote: >> Fixes three issues in ExpressionHelper: >> >> - Current Value was not retained when changing from SingleChange to Generic, >> this can lead to missed changes >> - Current Value was not retained when changing from Generic to SingleChange,

Re: RFR: 8278938: [Win] Robot can target wrong key for punctuation and symbols [v4]

2023-04-11 Thread Jose Pereda
On Mon, 10 Apr 2023 23:55:51 GMT, Martin Fox wrote: >> When processing a `WM_CHAR` event on an OEM key (punctuation, symbol, dead >> key) the glass code will dynamically query the key's unshifted character to >> determine the Java code to assign to it. This is necessary since the >> relationsh

Re: RFR: JDK-8224260: ChangeListener not triggered when adding a new listener in invalidated method [v2]

2023-04-11 Thread Kevin Rushforth
On Tue, 11 Apr 2023 07:57:56 GMT, John Hendrikx wrote: >> Fixes three issues in ExpressionHelper: >> >> - Current Value was not retained when changing from SingleChange to Generic, >> this can lead to missed changes >> - Current Value was not retained when changing from Generic to SingleChange,

Re: RFR: 8269593: Different fontname on macos [v2]

2023-04-11 Thread Kevin Rushforth
On Tue, 29 Jun 2021 14:20:30 GMT, Johan Vos wrote: >> Make sure the returned fullName of a created font matches the requested >> name. Since the name is used as a key/identifier in some cases, some >> internal code relies on this. >> Added a test to check the case of "System Font Regular" on Ma

Withdrawn: 8269593: Different fontname on macos

2023-04-11 Thread Kevin Rushforth
On Tue, 29 Jun 2021 14:00:08 GMT, Johan Vos wrote: > Make sure the returned fullName of a created font matches the requested name. > Since the name is used as a key/identifier in some cases, some internal code > relies on this. > Added a test to check the case of "System Font Regular" on MacOS,

Withdrawn: 8246104: Some complex text doesn't render correctly on macOS

2023-04-11 Thread Kevin Rushforth
On Sat, 26 Jun 2021 15:40:47 GMT, Johan Vos wrote: > [Mac only] register system fonts. > Fix for JDK-8246104 > > The list of available fonts returned by > CTFontCollectionCreateFromAvailableFonts does not contain internal fonts (at > least not by default, although this is not documented). By r

Re: RFR: 8246104: Some complex text doesn't render correctly on macOS

2023-04-11 Thread Kevin Rushforth
On Sat, 26 Jun 2021 15:40:47 GMT, Johan Vos wrote: > [Mac only] register system fonts. > Fix for JDK-8246104 > > The list of available fonts returned by > CTFontCollectionCreateFromAvailableFonts does not contain internal fonts (at > least not by default, although this is not documented). By r

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v2]

2023-04-11 Thread Ajit Ghaisas
On Mon, 10 Apr 2023 08:32:33 GMT, Karthik P K wrote: >> When custom skin was loaded, the listeners added in `ContextMenuContent` >> class while loading the default skin were not removed. This was causing the >> NPE when outdated listeners were invoked. >> >> Updated the code to dispose listene

Integrated: JDK-8224260: ChangeListener not triggered when adding a new listener in invalidated method

2023-04-11 Thread John Hendrikx
On Thu, 30 Mar 2023 21:53:48 GMT, John Hendrikx wrote: > Fixes three issues in ExpressionHelper: > > - Current Value was not retained when changing from SingleChange to Generic, > this can lead to missed changes > - Current Value was not retained when changing from Generic to SingleChange, > t

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v3]

2023-04-11 Thread Karthik P K
On Mon, 10 Apr 2023 07:21:21 GMT, Ajit Ghaisas wrote: >> Karthik P K has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Add space between if and ( >> - Address review comments > > modules/javafx.controls/src/main/java/javafx/scene/contro

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v3]

2023-04-11 Thread Karthik P K
> When custom skin was loaded, the listeners added in `ContextMenuContent` > class while loading the default skin were not removed. This was causing the > NPE when outdated listeners were invoked. > > Updated the code to dispose listeners in the `dispose` method of > `ContextMenuSkin` so that w

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v3]

2023-04-11 Thread Andy Goryachev
On Tue, 11 Apr 2023 12:18:52 GMT, Karthik P K wrote: >> When custom skin was loaded, the listeners added in `ContextMenuContent` >> class while loading the default skin were not removed. This was causing the >> NPE when outdated listeners were invoked. >> >> Updated the code to dispose listene

[jfx20u] Integrated: 8301009: Update libxml2 to 2.10.3

2023-04-11 Thread Hima Bindu Meda
Clean Backport.This pull request contains a backport of commit [935c7b79](https://github.com/openjdk/jfx/commit/935c7b797d79407d741735324313684617d1292d) from the [openjdk/jfx](https://git.openjdk.org/jfx) repository. - Commit messages: - Backport 935c7b797d79407d741735324313684617

[jfx20u] Integrated: 8301009: Update libxml2 to 2.10.3

2023-04-11 Thread Hima Bindu Meda
On Tue, 11 Apr 2023 17:03:51 GMT, Hima Bindu Meda wrote: > Clean Backport.This pull request contains a backport of commit > [935c7b79](https://github.com/openjdk/jfx/commit/935c7b797d79407d741735324313684617d1292d) > from the [openjdk/jfx](https://git.openjdk.org/jfx) repository. This pull req

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-11 Thread Andy Goryachev
On Mon, 10 Apr 2023 23:08:31 GMT, John Hendrikx wrote: >> BitSet uses the SetListenerHelper abstraction to prevent allocating the >> listener arrays. >> >> When removing listeners, the newly returned listener helper (which may be >> different from the one called) is not reassigned. This effect

Integrated: 8278938: [Win] Robot can target wrong key for punctuation and symbols

2023-04-11 Thread Martin Fox
On Thu, 23 Dec 2021 20:50:53 GMT, Martin Fox wrote: > When processing a `WM_CHAR` event on an OEM key (punctuation, symbol, dead > key) the glass code will dynamically query the key's unshifted character to > determine the Java code to assign to it. This is necessary since the > relationship b

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-11 Thread Kevin Rushforth
On Tue, 11 Apr 2023 18:27:51 GMT, Andy Goryachev wrote: > too late, but this code gives off 2 warnings: The warnings are from Eclipse? The signature of this method matches the interface method from `Set` so the warnings are surprising. If you think it's a legitimate warning, then can you file

Re: RFR: JDK-8304933: BitSet (used for CSS pseudo class states) listener management is incorrect [v2]

2023-04-11 Thread Andy Goryachev
On Mon, 10 Apr 2023 23:08:31 GMT, John Hendrikx wrote: >> BitSet uses the SetListenerHelper abstraction to prevent allocating the >> listener arrays. >> >> When removing listeners, the newly returned listener helper (which may be >> different from the one called) is not reassigned. This effect

Re: RFR: 8087370: [odroid] Monocle: Touch is still broken on Odroid

2023-04-11 Thread Johan Vos
On Wed, 13 Oct 2021 10:52:40 GMT, Fabian Wolter wrote: > There are sometimes multitouch events detected, when only a single touch > should be detected under certain conditions. This lead to touch events on > previous touch positions. > > The referenced bug is closed with "won't fix" with the j

RFR: 8305867: BitSetShim: The type parameter T is hiding the type T warning

2023-04-11 Thread Andy Goryachev
Fixed minor warnings in Eclipse. - Commit messages: - 8305867: BitSetShim: The type parameter T is hiding the type T warning Changes: https://git.openjdk.org/jfx/pull/1086/files Webrev: https://webrevs.openjdk.org/?repo=jfx&pr=1086&range=00 Issue: https://bugs.openjdk.org/browse/

Re: RFR: 8305867: BitSetShim: The type parameter T is hiding the type T warning

2023-04-11 Thread John Hendrikx
On Tue, 11 Apr 2023 20:46:37 GMT, Andy Goryachev wrote: > Fixed minor warnings in Eclipse. Marked as reviewed by jhendrikx (Committer). - PR Review: https://git.openjdk.org/jfx/pull/1086#pullrequestreview-1380481855

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v4]

2023-04-11 Thread Karthik P K
> When custom skin was loaded, the listeners added in `ContextMenuContent` > class while loading the default skin were not removed. This was causing the > NPE when outdated listeners were invoked. > > Updated the code to dispose listeners in the `dispose` method of > `ContextMenuSkin` so that w

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v3]

2023-04-11 Thread Karthik P K
On Tue, 11 Apr 2023 15:04:27 GMT, Andy Goryachev wrote: >> Karthik P K has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - Add space between if and ( >> - Address review comments > > modules/javafx.controls/src/main/java/javafx/scene/cont

Re: RFR: 8088594: NullPointerException on showing submenu of a contextmenu [v4]

2023-04-11 Thread Ajit Ghaisas
On Wed, 12 Apr 2023 05:08:46 GMT, Karthik P K wrote: >> When custom skin was loaded, the listeners added in `ContextMenuContent` >> class while loading the default skin were not removed. This was causing the >> NPE when outdated listeners were invoked. >> >> Updated the code to dispose listene