On Tue, 12 Aug 2025 12:16:31 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> @kevinrushforth I'm not suggesting fixing this right away—now—under the 
>> pressure.
>> 
>> I think these issues should've been resolved before integrating this fix.
>
> OK, thanks. Let's file a follow-up bug to capture this.

> The formula doesn't look right. Why do you subtract `rect.x` and `rect.y` 
> instead of `insets.left` and `insets.top`?
> 
> This would work correctly if both `rect.x` and `rect.y` are zero, but it 
> would give wrong results in other cases.

I've submitted [JDK-8365379](https://bugs.openjdk.org/browse/JDK-8365379): 
_SU3.applyInsets may produce wrong results_.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23324#discussion_r2269664033

Reply via email to