On Thu, 11 Jun 2026 12:46:40 GMT, Marius Hanl <[email protected]> wrote:

>> I think I mentioned that before, this is done to prevent calls to 
>> updateDisplayNode(). E.g. the method getDisplayNode() calls 
>> updateDisplayNode(), which would defeat the point of the test
>
> Yes, thats why my idea is to assign it once here and just reuse it below

I mean at this point we have 3 almost identical duplicated tests, I don't think 
that would make a big difference when it comes to code quality. Especially 
considering that one still needs to be aware of the side effect of calling that 
method. If I don't call it, I don't need to think about it

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2179#discussion_r3395982952

Reply via email to