RFR: 8332895: Support interpolation for backgrounds and borders [v2]

Andy Goryachev angorya at openjdk.org
Fri Aug 2 16:25:37 UTC 2024


On Fri, 2 Aug 2024 15:44:41 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> I suggest we clarify this behavior in the method description, since this is a public API documentation.  
>> 
>> Simply saying "should" increases cognitive load and immediately triggers a question: why have this argument at all?, and the need to search the code for an explanation.
>> 
>> (yes, I understand we could also create a private method with the `map` argument)
>
> I've worded it more strongly such that callers _must_ pass in `null`. This is a private method, so we don't need to account for other usage scenarios.

oh, right, it is private!  never mind then.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1522#discussion_r1702066372


More information about the openjfx-dev mailing list