Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v8]

2022-06-09 Thread Florian Kirmaier
On Thu, 14 Oct 2021 17:14:11 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8273485 >> Added check for null when calling initScreens > > mod

Re: RFR: [WIP] 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed [v6]

2022-05-24 Thread Florian Kirmaier
On Mon, 23 May 2022 20:44:47 GMT, Johan Vos wrote: >> When the size of a ListCell is changed and a scrollTo method is invoked >> without having a layout calculation in between, the old (wrong) size is used >> to calculcate the total estimate. This happens e.g. when the size is changed >> in th

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed [v4]

2022-05-16 Thread Florian Kirmaier
On Sun, 8 May 2022 18:56:45 GMT, Johan Vos wrote: >> @johanvos >> As requested, we've made a unit test, which tests this bug. >> It's based on your test and our original test class. It can be added to the >> ListViewTest. >> You can find it, in the JDK ticket. >> >> Btw, adding the cells incre

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed [v4]

2022-05-09 Thread Florian Kirmaier
On Sun, 8 May 2022 18:56:45 GMT, Johan Vos wrote: >> @johanvos >> As requested, we've made a unit test, which tests this bug. >> It's based on your test and our original test class. It can be added to the >> ListViewTest. >> You can find it, in the JDK ticket. >> >> Btw, adding the cells incre

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed [v4]

2022-04-25 Thread Florian Kirmaier
On Thu, 14 Apr 2022 08:52:27 GMT, Johan Vos wrote: >> Johan Vos has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Don't shift cells if we are already showing the lowest index cell. > > I agree with that observation. The mathematical perfec

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed [v4]

2022-04-12 Thread Florian Kirmaier
On Wed, 30 Mar 2022 13:27:40 GMT, Johan Vos wrote: >> When the size of a ListCell is changed and a scrollTo method is invoked >> without having a layout calculation in between, the old (wrong) size is used >> to calculcate the total estimate. This happens e.g. when the size is changed >> in th

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed [v4]

2022-04-12 Thread Florian Kirmaier
On Wed, 30 Mar 2022 13:27:40 GMT, Johan Vos wrote: >> When the size of a ListCell is changed and a scrollTo method is invoked >> without having a layout calculation in between, the old (wrong) size is used >> to calculcate the total estimate. This happens e.g. when the size is changed >> in th

Integrated: 8279228 Leak in ScrollPaneSkin, related to touch events

2022-02-23 Thread Florian Kirmaier
On Thu, 23 Dec 2021 17:43:19 GMT, Florian Kirmaier wrote: > Fixing memoryleak, related to touch events in ScrollPaneWhen touchDetected or > mouseDown is true, the sbTouch animation is running, > and the node is removed from the Scene, then the animation will never stop, > caus

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v5]

2022-01-27 Thread Florian Kirmaier
"reshown", but it > restores sanity about memory behaviour, which should be considered more > important. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8269907 Removed the sync methods for the scene, bec

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed

2022-01-13 Thread Florian Kirmaier
On Sun, 9 Jan 2022 15:20:36 GMT, Johan Vos wrote: > When the size of a ListCell is changed and a scrollTo method is invoked > without having a layout calculation in between, the old (wrong) size is used > to calculcate the total estimate. This happens e.g. when the size is changed > in the `up

Re: RFR: 8277848 Binding and Unbinding to List leads to memory leak [v3]

2022-01-11 Thread Florian Kirmaier
On Wed, 5 Jan 2022 18:14:44 GMT, Florian Kirmaier wrote: >> Making the initial listener of the ListProperty weak fixes the problem. >> The same is fixed for Set and Map. >> Due to a smart implementation, this is done without any performance drawback. >> (The trick is to

Re: RFR: 8277848 Binding and Unbinding to List leads to memory leak [v4]

2022-01-11 Thread Florian Kirmaier
) > By implying the same trick to the InvalidationListener, this should even > improve the performance of the collection properties. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8277848 Added the 3

Re: RFR: 8256397: MultipleSelectionModel throws IndexOutOfBoundException [v3]

2022-01-10 Thread Florian Kirmaier
On Mon, 10 Jan 2022 12:29:04 GMT, Florian Kirmaier wrote: >> Fixing IndexOutOfBoundsException in the MultipleSelectionModelBase and added >> a unit-test for it. >> ticket: https://bugs.openjdk.java.net/browse/JDK-8256397 >> run test: `./gradlew --continue -PFULL_TEST=tr

Re: RFR: 8256397: MultipleSelectionModel throws IndexOutOfBoundException [v3]

2022-01-10 Thread Florian Kirmaier
> Fixing IndexOutOfBoundsException in the MultipleSelectionModelBase and added > a unit-test for it. > ticket: https://bugs.openjdk.java.net/browse/JDK-8256397 > run test: `./gradlew --continue -PFULL_TEST=true controls:test --tests > "*MultipleSelectionModelImplTest*&quo

Re: RFR: 8256397: MultipleSelectionModel throws IndexOutOfBoundException [v2]

2022-01-10 Thread Florian Kirmaier
> Fixing IndexOutOfBoundsException in the MultipleSelectionModelBase and added > a unit-test for it. > ticket: https://bugs.openjdk.java.net/browse/JDK-8256397 > run test: `./gradlew --continue -PFULL_TEST=true controls:test --tests > "*MultipleSelectionModelImplTest*&quo

Re: RFR: 8277785: ListView scrollTo jumps to wrong location when CellHeight is changed

2022-01-10 Thread Florian Kirmaier
On Sun, 9 Jan 2022 15:20:36 GMT, Johan Vos wrote: > When the size of a ListCell is changed and a scrollTo method is invoked > without having a layout calculation in between, the old (wrong) size is used > to calculcate the total estimate. This happens e.g. when the size is changed > in the `up

Re: RFR: 8279228 Leak in ScrollPaneSkin, related to touch events

2022-01-07 Thread Florian Kirmaier
On Thu, 23 Dec 2021 17:43:19 GMT, Florian Kirmaier wrote: > Fixing memoryleak, related to touch events in ScrollPaneWhen touchDetected or > mouseDown is true, the sbTouch animation is running, > and the node is removed from the Scene, then the animation will never stop, > caus

Re: RFR: 8277848 Binding and Unbinding to List leads to memory leak [v2]

2022-01-05 Thread Florian Kirmaier
On Fri, 31 Dec 2021 12:53:54 GMT, Michael Strauß wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8277848 >> Added missing change > > Why are the new listener imlem

Re: RFR: 8277848 Binding and Unbinding to List leads to memory leak [v3]

2022-01-05 Thread Florian Kirmaier
) > By implying the same trick to the InvalidationListener, this should even > improve the performance of the collection properties. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8277848 Further optimization base

RFR: 8279228 Leak in ScrollPaneSkin, related to touch events

2021-12-23 Thread Florian Kirmaier
Fixing memoryleak, related to touch events in ScrollPaneWhen touchDetected or mouseDown is true, the sbTouch animation is running, and the node is removed from the Scene, then the animation will never stop, causing a memory leak. A simple fix is to also check, whether the Node is visible, by che

Integrated: 8278905 JavaFX: EnumConverter has a typo in the toString method

2021-12-16 Thread Florian Kirmaier
On Thu, 16 Dec 2021 14:29:21 GMT, Florian Kirmaier wrote: > Fixing typo in EnumConverter This pull request has now been integrated. Changeset: 4c5bf44e Author: Florian Kirmaier Committer: Kevin Rushforth URL: https://git.openjdk.java.net/jfx/com

RFR: 8278905 JavaFX: EnumConverter has a typo in the toString method

2021-12-16 Thread Florian Kirmaier
Fixing typo in EnumConverter - Commit messages: - 8278905 Changes: https://git.openjdk.java.net/jfx/pull/697/files Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=697&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8278905 Stats: 1 line in 1 file changed: 0 ins;

Re: RFR: 8277848 Binding and Unbinding to List leads to memory leak [v2]

2021-12-10 Thread Florian Kirmaier
) > By implying the same trick to the InvalidationListener, this should even > improve the performance of the collection properties. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8277848 Added missing change

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v4]

2021-12-10 Thread Florian Kirmaier
On Thu, 9 Dec 2021 12:45:32 GMT, Florian Kirmaier wrote: >> After thinking about this issue for some time, I've now got a solution. >> I know put the scene in the state it is, before is was shown, when the >> dirtyNodes are unset, the whole scene is basically considered

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v4]

2021-12-10 Thread Florian Kirmaier
"reshown", but it > restores sanity about memory behaviour, which should be considered more > important. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8269907 Fixed rare bug, causing bounds to be out of syn

RFR: 8277848 Binding and Unbinding to List leads to memory leak

2021-12-07 Thread Florian Kirmaier
Making the initial listener of the ListProperty weak fixes the problem. The same is fixed for Set and Map. Due to a smart implementation, this is done without any performance drawback. (The trick is to have an object, which is both the WeakReference and the Changelistener) By implying the same tri

Re: RFR: 8276170: Create Sources when publishing to Maven

2021-12-02 Thread Florian Kirmaier
On Fri, 29 Oct 2021 12:36:13 GMT, eduardsdv wrote: > Create sources.jars and attach they to the publish task, so that they can be > uploaded to the (e.g. maven) repository automatically. I wonder how the sources for the official maven release are generated? https://repo1.maven.org/maven2/org/o

Integrated: 8274022 Additional Memory Leak in ControlAcceleratorSupport

2021-11-15 Thread Florian Kirmaier
On Sat, 30 Oct 2021 10:56:40 GMT, Florian Kirmaier wrote: > This fixes the new ControlAcceleratorBug which was Introduced in JavaFX17. > To fix it, I've made the Value of the WeakHashMap also weak. > We only keep this value to remove it as a listener later on. Therefore there

Re: RFR: 8274022 Additional Memory Leak in ControlAcceleratorSupport [v5]

2021-11-15 Thread Florian Kirmaier
On Mon, 15 Nov 2021 08:24:04 GMT, Florian Kirmaier wrote: >> This fixes the new ControlAcceleratorBug which was Introduced in JavaFX17. >> To fix it, I've made the Value of the WeakHashMap also weak. >> We only keep this value to remove it as a listener later on. Theref

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v7]

2021-11-15 Thread Florian Kirmaier
On Mon, 18 Oct 2021 08:20:22 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v8]

2021-11-15 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 Added check for null when calling initScreen

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v3]

2021-11-15 Thread Florian Kirmaier
On Fri, 12 Nov 2021 23:42:30 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8274022 >> Simplified code related to WeakHashMaps > > modules/javafx.c

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v5]

2021-11-15 Thread Florian Kirmaier
t's especially annoying > because it takes some time until the application gets unstable. > > Therefore **I would recommend** after this fix is approved, **to make a new > version for JavaFX17** with this fix because this bug is so severe. Florian Kirmaier has updated the pull req

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v4]

2021-11-15 Thread Florian Kirmaier
t's especially annoying > because it takes some time until the application gets unstable. > > Therefore **I would recommend** after this fix is approved, **to make a new > version for JavaFX17** with this fix because this bug is so severe. Florian Kirmaier has updated the pull req

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v3]

2021-11-15 Thread Florian Kirmaier
On Fri, 12 Nov 2021 22:42:35 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8274022 >> Simplified code related to WeakHashMaps > > modules/javafx.c

Integrated: 8276179 PrismFontFile.isInstalledFont is dead code and should be removed

2021-11-02 Thread Florian Kirmaier
On Fri, 29 Oct 2021 15:10:35 GMT, Florian Kirmaier wrote: > removing dead code. > When looking into the font code, I've noticed that this code is no longer > used, so I thought I should make a PR with a minor cleanup. This pull request has now been integrated. Changeset: c

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v7]

2021-11-02 Thread Florian Kirmaier
On Mon, 18 Oct 2021 08:20:22 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v7]

2021-11-02 Thread Florian Kirmaier
On Sat, 30 Oct 2021 13:35:39 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8273485 >> Fixing toggle fullscreen! > > modules/javaf

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v3]

2021-11-02 Thread Florian Kirmaier
On Tue, 2 Nov 2021 10:15:41 GMT, Florian Kirmaier wrote: >> After thinking about this issue for some time, I've now got a solution. >> I know put the scene in the state it is, before is was shown, when the >> dirtyNodes are unset, the whole scene is basically considered

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v3]

2021-11-02 Thread Florian Kirmaier
"reshown", but it > restores sanity about memory behaviour, which should be considered more > important. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8269907 We now require the rendering lock when clea

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v3]

2021-11-02 Thread Florian Kirmaier
t's especially annoying > because it takes some time until the application gets unstable. > > Therefore **I would recommend** after this fix is approved, **to make a new > version for JavaFX17** with this fix because this bug is so severe. Florian Kirmaier has updated the pull req

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v3]

2021-11-02 Thread Florian Kirmaier
On Sun, 31 Oct 2021 16:32:49 GMT, Michael Strauß wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8274022 >> Simplified code related to WeakHashMaps > > modules/javafx.c

Re: RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport [v2]

2021-11-02 Thread Florian Kirmaier
t's especially annoying > because it takes some time until the application gets unstable. > > Therefore **I would recommend** after this fix is approved, **to make a new > version for JavaFX17** with this fix because this bug is so severe. Florian Kirmaier has updated the pull req

RFR: 8274022 fixing critical memory leak in the ControlAcceleratorSupport

2021-10-30 Thread Florian Kirmaier
This fixes the new ControlAcceleratorBug which was Introduced in JavaFX17. To fix it, I've made the Value of the WeakHashMap also weak. We only keep this value to remove it as a listener later on. Therefore there shouldn't be issues by making this value weak. I've seen this Bug very very often,

Re: RFR: 8276179 removing unused font code - isInstalledFont

2021-10-29 Thread Florian Kirmaier
On Fri, 29 Oct 2021 15:10:35 GMT, Florian Kirmaier wrote: > removing dead code. > When looking into the font code, I've noticed that this code is no longer > used, so I thought I should make a PR with a minor cleanup. I've also seen many places in the font code, which ju

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v7]

2021-10-29 Thread Florian Kirmaier
On Mon, 18 Oct 2021 08:20:22 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

RFR: 8276179 removing unused font code - isInstalledFont

2021-10-29 Thread Florian Kirmaier
removing dead code. When looking into the font code, I've noticed that this code is no longer used, so I thought I should make a PR with a minor cleanup. - Commit messages: - JDK-8276179 Changes: https://git.openjdk.java.net/jfx/pull/658/files Webrev: https://webrevs.openjdk.java.

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v2]

2021-10-29 Thread Florian Kirmaier
On Thu, 14 Oct 2021 14:29:55 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8269907 >> The bug is now fixed in a new way. Toolkit now supports re

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v7]

2021-10-29 Thread Florian Kirmaier
On Mon, 18 Oct 2021 08:20:22 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v7]

2021-10-18 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 Fixing toggle fullscreen! - Chang

How to publish sources and javadoc?

2021-10-04 Thread Florian Kirmaier
and sources be generated and added to the Maven release? Florian Kirmaier

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v6]

2021-09-28 Thread Florian Kirmaier
On Tue, 28 Sep 2021 12:07:36 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v6]

2021-09-28 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 removed the toggle fullscreen before closing

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v4]

2021-09-28 Thread Florian Kirmaier
On Fri, 17 Sep 2021 12:51:53 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8273485 >> small cleanup of the changes. > > You can also see the

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v5]

2021-09-28 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 Fixed Beep sound when closing a fullscreen windo

Integrated: 8273969: Memory Leak on the Runnable provided to Platform.startup

2021-09-23 Thread Florian Kirmaier
On Sun, 19 Sep 2021 15:33:46 GMT, Florian Kirmaier wrote: > Probably my most boring PR. ;) > Setting the lambda to null, after it has been used, fixes the leak. This pull request has now been integrated. Changeset: 4b9cb210 Author:Florian Kirmaier Committer: Kevin Rushfort

Re: RFR: 8273969: Memory Leak on the Runnable provided to Platform.startup [v2]

2021-09-22 Thread Florian Kirmaier
> Probably my most boring PR. ;) > Setting the lambda to null, after it has been used, fixes the leak. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273969 Some changes based on code review - C

Re: RFR: 8273969: Memory Leak on the Runnable provided to Platform.startup [v2]

2021-09-22 Thread Florian Kirmaier
On Mon, 20 Sep 2021 12:56:39 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8273969 >> Some changes based on code review > > tests/sy

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v4]

2021-09-21 Thread Florian Kirmaier
On Thu, 16 Sep 2021 13:22:32 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

Re: RFR: 8273969: Memory Leak on the Lambda provided to Platform.startup

2021-09-20 Thread Florian Kirmaier
On Sun, 19 Sep 2021 15:33:46 GMT, Florian Kirmaier wrote: > Probably my most boring PR. ;) > Setting the lambda to null, after it has been used, fixes the leak. I guess that makes sense - but I can think of a million ways to create nearly unsolvable problems with it. - PR:

RFR: 8273969: Memory Leak on the Lambda provided to Platform.startup

2021-09-19 Thread Florian Kirmaier
Probably my most boring PR. ;) Setting the lambda to null, after it has been used, fixes the leak. - Commit messages: - JDK-8273969 Changes: https://git.openjdk.java.net/jfx/pull/626/files Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=626&range=00 Issue: https://bugs.open

Re: RFR: 8273969: Memory Leak on the Lambda provided to Platform.startup

2021-09-19 Thread Florian Kirmaier
On Sun, 19 Sep 2021 15:33:46 GMT, Florian Kirmaier wrote: > Probably my most boring PR. ;) > Setting the lambda to null, after it has been used, fixes the leak. For the strange reason, why i haven't used a lambda for the test, I've created another ticket: https://bugs.openjdk

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v4]

2021-09-16 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 small cleanup of the changes

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v3]

2021-09-13 Thread Florian Kirmaier
On Mon, 13 Sep 2021 13:35:52 GMT, Florian Kirmaier wrote: >> When using Swing it's possible to generate a Deadlock. >> It's related to the nested eventloop started in enterFullScreenExitingLoop >> - and the RenderLock aquired when using setView in Scene. >>

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v3]

2021-09-13 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 Removed the enter/leave ne

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac [v2]

2021-09-13 Thread Florian Kirmaier
ng the nested loop fixes the Problem. > I hope this doesn't have any side effect - so far i don't know of any. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8273485 Added unit-test - Changes: - a

Re: RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac

2021-09-09 Thread Florian Kirmaier
On Wed, 8 Sep 2021 10:37:40 GMT, Florian Kirmaier wrote: > When using Swing it's possible to generate a Deadlock. > It's related to the nested eventloop started in enterFullScreenExitingLoop - > and the RenderLock aquired when using setView in Scene. > Sample Programm an

RFR: 8273485: Deadlock when also using Swing and exiting Fullscreen on Mac

2021-09-08 Thread Florian Kirmaier
When using Swing it's possible to generate a Deadlock. It's related to the nested eventloop started in enterFullScreenExitingLoop - and the RenderLock aquired when using setView in Scene. Sample Programm and Threaddump are added to the ticket. Removing the nested loop fixes the Problem. I hope

Re: RFR: 8271395: Fixing crash at printing [v2]

2021-09-08 Thread Florian Kirmaier
a new one is created for the Printing Thread. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8271395 QuantumRenderer is no longer public - Changes: - all: https://git.openjdk.java.net/jfx/pull/618/

Re: RFR: 8271395: Fixing crash at printing [v2]

2021-09-08 Thread Florian Kirmaier
On Mon, 6 Sep 2021 13:33:11 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8271395 >> QuantumRenderer is no longer public > > modules/javafx.grap

RFR: 8271395: Fixing crash at printing

2021-09-06 Thread Florian Kirmaier
This PR switches the Thread to the QuantumRenderer, in the case the Disposer is called from another Thread - the printing Thread. I'm open for better solutions on how to fix this Issue. Initially i thought there is also a Race Condition in the resource pool, but a new one is created for the Print

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v2]

2021-07-29 Thread Florian Kirmaier
On Thu, 22 Jul 2021 11:50:11 GMT, Florian Kirmaier wrote: >> After thinking about this issue for some time, I've now got a solution. >> I know put the scene in the state it is, before is was shown, when the >> dirtyNodes are unset, the whole scene is basically conside

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed [v2]

2021-07-22 Thread Florian Kirmaier
"reshown", but it > restores sanity about memory behaviour, which should be considered more > important. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: JDK-8269907 The bug is now fixed in a new way.

Re: RFR: 8269907 memory leak - Dirty Nodes / Parent removed

2021-07-22 Thread Florian Kirmaier
On Wed, 21 Jul 2021 11:29:38 GMT, Florian Kirmaier wrote: > After thinking about this issue for some time, I've now got a solution. > I know put the scene in the state it is, before is was shown, when the > dirtyNodes are unset, the whole scene is basically considered dirty.

RFR: MemoryLeak: Scene.dirtyNodes and Parent.removed

2021-07-21 Thread Florian Kirmaier
Hi everyone, I've now found a possible fix for a memory leak I discovered several weeks ago. Can someone take a look into it? issue: https://bugs.openjdk.java.net/browse/JDK-8269907 PR: https://github.com/openjdk/jfx/pull/584 Greetings Florian

Re: RFR: JDK-8269921 Text in Textflow and listeners on bounds can cause endless loop/crash and other performance issues [v2]

2021-07-06 Thread Florian Kirmaier
On Tue, 6 Jul 2021 15:07:26 GMT, Florian Kirmaier wrote: >> modules/javafx.graphics/src/main/java/javafx/scene/text/Text.java line 386: >> >>> 384: private GlyphList[] getRuns() { >>> 385: if (textRuns != null) return textRuns; >>> 386:

Re: RFR: JDK-8269921 Text in Textflow and listeners on bounds can cause endless loop/crash and other performance issues [v2]

2021-07-06 Thread Florian Kirmaier
On Tue, 6 Jul 2021 14:58:49 GMT, Kevin Rushforth wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> JDK-8269921 >> Added a copyright header > > modules/javafx.graphics/src/m

Re: RFR: JDK-8269921 Text in Textflow and listeners on bounds can cause endless loop/crash and other performance issues [v2]

2021-07-06 Thread Florian Kirmaier
s without issues in TextFlow Heavy applications. > > Benefits: > * Better Tooling Support For ScenicView etc. > * Fixes complicated but reproducible crashes > * Might fix some rare crashes, which are hard to reproduce > * Likely improves performance - might fix some edge

RFR: JDK-8269921 Text in Textflow and listeners on bounds can cause endless loop/crash and other performance issues

2021-07-06 Thread Florian Kirmaier
It's "a bit" complicated. In some situations, getRuns get's called because listeners on bounds are set. This causes TextFlow to layout to compute the runs. Afterward, the bounds of the parents get updated. This triggers a call to compute bounds - which cascades up to the children. When the geometr

RFR: JDK-8269921 Text in Textflow and listeners on bounds can cause endless loop/crash and other performance issues

2021-07-06 Thread Florian Kirmaier
Hi everyone, I've developed a fix for an exception in the MultipleSelectionModel. Can someone look into it? https://github.com/openjdk/jfx/pull/564 Greetings Florian Kirmaier

MemoryLeak: Scene.dirtyNodes and Parent.removed

2021-07-06 Thread Florian Kirmaier
Hi everyone, I was hunting for some memory leaks, but I think I need some help/feedback/thoughts on this one. ticket: https://bugs.openjdk.java.net/browse/JDK-8269907 A unit test can be found in the ticket. Florian Kirmaier

Integrated: 8264127: ListCell editing status is true, when index changes while editing

2021-05-22 Thread Florian Kirmaier
On Wed, 24 Mar 2021 14:58:40 GMT, Florian Kirmaier wrote: > Fixing ListCell editing status is true, when index changes while editing. This pull request has now been integrated. Changeset: 240d28ff Author: Florian Kirmaier Committer: Jeanette Winzenburg URL: ht

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v14]

2021-05-20 Thread Florian Kirmaier
On Tue, 18 May 2021 10:14:11 GMT, Florian Kirmaier wrote: >> Fixing ListCell editing status is true, when index changes while editing. > > Florian Kirmaier has updated the pull request incrementally with two > additional commits since the last revision: > > - 8264127

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v15]

2021-05-20 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: 8264127 small changes based on CR - Changes: - all: https://git.openjdk.java.net/

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v14]

2021-05-18 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with two additional commits since the last revision: - 8264127 fixed assertion text, exception handler is now installed/uninstalled like

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v13]

2021-05-18 Thread Florian Kirmaier
On Mon, 17 May 2021 08:40:44 GMT, Jeanette Winzenburg wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8264127 >> More changes, simplifications and tests based on CR > > m

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v12]

2021-05-14 Thread Florian Kirmaier
On Wed, 12 May 2021 08:13:32 GMT, Florian Kirmaier wrote: >> Fixing ListCell editing status is true, when index changes while editing. > > Florian Kirmaier has updated the pull request incrementally with one > additional commit since the last revision: > > 8264127:

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v12]

2021-05-14 Thread Florian Kirmaier
On Thu, 13 May 2021 12:29:46 GMT, Jeanette Winzenburg wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8264127: >> we now use a try finally statement, to make sure u

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v13]

2021-05-14 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: 8264127 More changes, simplifications and tests based on CR - Changes: - all: ht

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v11]

2021-05-12 Thread Florian Kirmaier
On Tue, 27 Apr 2021 11:52:58 GMT, Florian Kirmaier wrote: >> Fixing ListCell editing status is true, when index changes while editing. > > Florian Kirmaier has updated the pull request incrementally with one > additional commit since the last revision: > > 8264127: >

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v12]

2021-05-12 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: 8264127: we now use a try finally statement, to make sure updateEditingIndex is re

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v11]

2021-04-27 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: 8264127: removed accidentally commited print statements. - Changes: - all: ht

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v8]

2021-04-27 Thread Florian Kirmaier
On Tue, 27 Apr 2021 10:38:07 GMT, Jeanette Winzenburg wrote: >> I've now rewritten the code. It's now way simpler and avoids duplicate code. >> I guess the different implementation in the different cells all have some >> subtle differences. >> >> Then i guess we wait for the other review to f

Re: RFR: 8263402: MemoryLeak: Node hardreferences it's previous Parent after csslayout and getting removed from the scene [v5]

2021-04-27 Thread Florian Kirmaier
On Sun, 11 Apr 2021 15:48:11 GMT, Florian Kirmaier wrote: >> Fixing a memory leak. >> A node hard references its old parent after CSS layout and getting removed. >> This shouldn't be the case, this is very counterintuitive. >> >> The fix uses a

Integrated: 8263402: MemoryLeak: Node hardreferences it's previous Parent after csslayout and getting removed from the scene

2021-04-27 Thread Florian Kirmaier
On Wed, 10 Mar 2021 22:25:32 GMT, Florian Kirmaier wrote: > Fixing a memory leak. > A node hard references its old parent after CSS layout and getting removed. > This shouldn't be the case, this is very counterintuitive. > > The fix uses a WeakReference i

Integrated: 8264677 MemoryLeak: Progressindicator leaks, when treeShowing is false

2021-04-27 Thread Florian Kirmaier
On Sat, 3 Apr 2021 15:35:19 GMT, Florian Kirmaier wrote: > Fixing leak in ProgressIndicator when treeShowing is false This pull request has now been integrated. Changeset: 483f171a Author: Florian Kirmaier Committer: Kevin Rushforth URL: https://git.openjdk.java.net/jfx/com

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v8]

2021-04-27 Thread Florian Kirmaier
On Mon, 26 Apr 2021 11:46:57 GMT, Jeanette Winzenburg wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8264127: >> Added missing test case > > modules/javafx.control

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v10]

2021-04-27 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with one additional commit since the last revision: 8264127: restructured the if/else cascade in updating index. It's now simpler an less nested a

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v9]

2021-04-27 Thread Florian Kirmaier
> Fixing ListCell editing status is true, when index changes while editing. Florian Kirmaier has updated the pull request incrementally with two additional commits since the last revision: - 8264127: Added more asserts to the ListCellTest, to make it more symmetric to the other t

Re: RFR: 8264127: ListCell editing status is true, when index changes while editing [v8]

2021-04-27 Thread Florian Kirmaier
On Mon, 26 Apr 2021 11:34:34 GMT, Jeanette Winzenburg wrote: >> Florian Kirmaier has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8264127: >> Added missing test case > > modules/javafx.controls/src

  1   2   3   >