On Fri, 13 Oct 2023 14:50:03 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> Sounds reasonable. Should I move the cleanup of Color as well?
>
>> Should I move the cleanup of Color as well?
> 
> Removing `colorValueID` could be left here, I think: it removes an exported 
> symbol which is no longer used.
> 
> In addition to it, it makes `Java_java_awt_Color_initIDs` empty.

Ok, I'll revert the initIDs removals, and will file a follow-up PR once this 
one is merged.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13261#discussion_r1358384108

Reply via email to