RFR: 8349188: LineBorder does not scale correctly [v3]

Sergey Bylokhov serb at openjdk.org
Wed Jul 2 02:10:45 UTC 2025


On Mon, 30 Jun 2025 17:07:01 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> My point is to investigate the difference between clipScale and clipRound to ensure the correct one is used. There was a complex discussion before about similar issue, which is why we currently have two separate versions of this rounding logic. Simply replacing one with the other might not work and some other tweaks should be done.
>
> My point is `clipRound` works good in this use case, *it resolves the problem*, I see no reason not to accept the fix.
> 
> ---
> 
> I agree with your general concern that we should understand better the purpose of each method. Yet no one seems to remember the outcome of that complex discussion, therefore switching to `clipScale` could be postponed; if it's deemed necessary, we can easily change the method, both in the implementation and in the test.
> 
> I submitted [JDK-8361095](https://bugs.openjdk.org/browse/JDK-8361095) to ensure this current discussion isn't forgotten and we figure out the purpose and guidelines for `clipScale` and `clipRound`.

Ok sounds good.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26025#discussion_r2178841430


More information about the client-libs-dev mailing list