RFR: 8090110: Very bad TextArea of performance in a language environment that uses the Chinese character (KANJI)

yosbits github.com+7517141+yososs at openjdk.java.net
Tue Sep 29 15:32:03 UTC 2020


On Tue, 29 Sep 2020 09:51:06 GMT, Jeanette Winzenburg <fastegal at openjdk.org> wrote:

>> Added the **Next Step** paragraph to the PR overview.
>> 
>> I think it is necessary to judge this step before fixing the resource leak. The existing code is an incomplete
>> implementation of dispose, assuming the development of MULTIPLE NODES mode.
>> @kleopatra
>> TextArea setSkin is not supported. Therefore, there is no existing application that depends on it. I can't write
>> additional tests for incomplete implementations (it always fails).
>
>> 
>> 
>> @kleopatra
>> TextArea setSkin is not supported. Therefore, there is no existing application that depends on it. I can't write
>> additional tests for incomplete implementations (it always fails).
> 
> what a fancy excuse to not write tests *chuckling
> 
> Assuming you mean [JDK-8244419](https://bugs.openjdk.java.net/browse/JDK-8244419): the way round could be to comment
> the unconditional exception throwing in dispose, write tests against your change that fail before your change and pass
> after. I'm aware that might be difficult, given that there is no cleanup whatever - but should be tried, ast least :).
> At the end, mark them with Ignore("8244419") and uncomment the exception throwing in dispose again.

I understand that some people want support for the **new features** of Skin Swiching.

-------------

PR: https://git.openjdk.java.net/jfx/pull/307


More information about the openjfx-dev mailing list