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

Alexey Ivanov aivanov at openjdk.org
Mon Jun 30 17:09:40 UTC 2025


On Mon, 30 Jun 2025 16:24:09 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> I looked at another test case Rajat and I discussed, and using `clipScale` doesn't help there either. I prefer rendering with `clipRound`.
>
> 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 of the methods. 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`.

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

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


More information about the client-libs-dev mailing list