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

Reply via email to