RFR: JDK-8322964 Optimize performance of CSS selector matching [v6]
Kevin Rushforth
kcr at openjdk.org
Mon Mar 11 16:43:14 UTC 2024
On Sat, 9 Mar 2024 13:42:55 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:
>> modules/javafx.graphics/src/main/java/javafx/css/StyleClass.java line 32:
>>
>>> 30: * @deprecated for removal
>>> 31: */
>>> 32: public final class StyleClass {
>>
>> Needs the `@Deprecated(forRemoval = true)` annotation on the class, and the doc tag needs an explanation of why it's being removed ("erroneously exposed", "no longer needed", "flawed from the start"...) and what should be used instead, if at all.
>
> I added it.
>
> Technically, we could leave this class in, but it would just be a dead class that serves no purpose within FX, at least, once the `SimpleSelector` and `CompoundSelector` classes have been moved to internal packages.
>
> I'm wondering if I should add any deprecations in this PR at all, as now it requires a CSR. This could be all handled in the PR that moves these classes to internal packages.
Ideally, you would not need any deprecations in this PR.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1316#discussion_r1520046722
More information about the openjfx-dev
mailing list