[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-openjfx-dev
Subject: Re: [Rev 02] RFR: 8236840: Memory leak when switching ButtonSkin
From: Ambarish Rapte <arapte () openjdk ! java ! net>
Date: 2020-03-30 7:21:32
Message-ID: a3UbEHffec0OwUlHFpqYqxgk9CaMPxv_TekAc8lb6Q4=.bf7e6dbe-1a4a-452e-b061-8b8dc7f207dd () github ! com
[Download RAW message or body]
> ButtonSkin adds a `ChangeListener` to `Control.sceneProperty()` which results in \
> leaking the `ButtonSkin` itself when the `Button`'s skin is changed to a new \
> `ButtonSkin`. Using a `WeakChangeListener` instead of `ChangeListener` solves the \
> issue. Please take a look.
Ambarish Rapte has updated the pull request incrementally with one additional commit \
since the last revision:
Fix review comment, add test
-------------
Changes:
- all: https://git.openjdk.java.net/jfx/pull/147/files
- new: https://git.openjdk.java.net/jfx/pull/147/files/43624547..e81f175a
Webrevs:
- full: https://webrevs.openjdk.java.net/jfx/147/webrev.02
- incr: https://webrevs.openjdk.java.net/jfx/147/webrev.01-02
Stats: 27 lines in 2 files changed: 24 ins; 0 del; 3 mod
Patch: https://git.openjdk.java.net/jfx/pull/147.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/147/head:pull/147
PR: https://git.openjdk.java.net/jfx/pull/147
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic