On Thu, 4 Dec 2025 21:40:39 GMT, Kevin Rushforth <[email protected]> wrote:
>> modules/javafx.graphics/src/main/java/javafx/animation/Interpolator.java
>> line 192:
>>
>>> 190:
>>> 191: /**
>>> 192: * Use {@link #ofSpline(double, double, double, double)}.
>>
>> maybe something like "This poorly named method is deprecated in favor of
>> ..." or words to that extent?
>>
>> or keep the original description?
>
>> maybe something like "This poorly named method is deprecated in favor of
>> ..." or words to that extent?
>
> We wouldn't do this in the description, but possibly in the `@deprecated` tag
> text (not sure).
>
>> or keep the original description?
>
> That's what we typically do, although I can see the value in what @mstr2
> proposed. I'll take a look at the built docs and see how it looks.
I've commited the suggested change without noticing your reply here. The
description is now similar to:
* This is a legacy method named inconsistently with method naming
conventions,
* use {@link #ofSpline(double, double, double, double)} instead.
I can revert the change, or keep the existing description.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1996#discussion_r2590705707
