On Wed, 7 Jul 2021 09:48:20 GMT, Jeanette Winzenburg
wrote:
>> The issue is about memory leaks and side-effects (like NPEs) when switching
>> skins.
>>
>> Details (copied from issue for convenience):
>>
>> memory leak in TextInputControlBehavior:
>> - listener accidentally added twice
On Mon, 28 Jun 2021 13:44:16 GMT, Marius Hanl wrote:
>> Jeanette Winzenburg has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> addressed review issues
>
> Just a formal review, I left some comments inline
@Maran23 did resolve all we
On Wed, 7 Jul 2021 09:48:20 GMT, Jeanette Winzenburg
wrote:
>> The issue is about memory leaks and side-effects (like NPEs) when switching
>> skins.
>>
>> Details (copied from issue for convenience):
>>
>> memory leak in TextInputControlBehavior:
>> - listener accidentally added twice
On Wed, 7 Jul 2021 10:02:52 GMT, Jeanette Winzenburg
wrote:
>> modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/behavior/BehaviorCleanupTest.java
>> line 404:
>>
>>> 402: }
>>> 403: if (!root.getChildren().contains(control)) {
>>> 404:
On Tue, 6 Jul 2021 20:00:16 GMT, Ambarish Rapte wrote:
>> Jeanette Winzenburg has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> addressed review issues
>
>
On Tue, 6 Jul 2021 20:21:09 GMT, Ambarish Rapte wrote:
>> I'm also interested in the opinion from others. I think we are a bit more
>> safer with weak listener and there are used often as well.
>> But as you correctly mentioned a lot of times (still) a listener is created
>> inline. But I
> The issue is about memory leaks and side-effects (like NPEs) when switching
> skins.
>
> Details (copied from issue for convenience):
>
> memory leak in TextInputControlBehavior:
> - listener accidentally added twice (removed once)
> - keyPad mappings not properly installed/disposed
>
>