On Thu, 25 Mar 2021 16:30:30 GMT, Nir Lisker <nlis...@openjdk.org> wrote:
>> Good catch. Test fails before and succeeds after the patch. > > By the way, this class could easily be converted to a `record`, and that > would take care of the `toString` and other ceremonies. I'm not sure it's a > backwards compatible change, though. Converting an existing class to a Record wouldn't be backward compatible (not to mention we can't use them yet). It wouldn't help for the Node::toString portion in any case. I'll add the comment and send out a new version. ------------- PR: https://git.openjdk.java.net/jfx/pull/443