On Wed, 19 Aug 2026 19:13:28 GMT, Andy Goryachev <[email protected]> wrote:

>> Michael Strauß has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   remove empty tag
>
> modules/javafx.graphics/src/main/java/javafx/scene/layout/package-info.java 
> line 421:
> 
>> 419:  *         double childX = snapPositionX(snappedLeftInset() + 
>> allocatedWidth);
>> 420:  *
>> 421:  *         // Incorrect: snapSizeX can turn floating-point noise into 
>> an extra pixel
> 
> I am wondering if we should remove incorrect examples (there are many more 
> ways to produce an incorrect result).

I found them really helpful. This is one of that things where negative examples 
really help to understand the full picture IMO.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2260#discussion_r3816382438

Reply via email to