RFR: 8373038: Interpolator factories should follow method naming convention [v2]

Kevin Rushforth kcr at openjdk.org
Thu Dec 4 21:57:32 UTC 2025


On Thu, 4 Dec 2025 21:54:17 GMT, Michael Strauß <mstrauss at openjdk.org> wrote:

>> The following interpolator factories don't follow the standard method naming convention:
>> 
>> * `Interpolator.SPLINE(double, double, double, double)`
>> * `Interpolator.TANGENT(Duration, double, Duration, double)`
>> * `Interpolator.TANGENT(Duration, double)`
>> * `Interpolator.STEPS(int, StepPosition)`
>> 
>> New methods named `ofSpline`, `ofTangent`, and `ofSteps` are added. The existing methods are deprecated (not for removal) in favor of the correctly-named methods. This change is in line with the new `ofLinear` method added with #1977.
>
> Michael Strauß has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - doc changes
>  - Merge branch 'master' into fixes/interpolator-names
>  - Interpolator factories should follow method naming convention

This looks like a reasonable thing to do. I'll take a closer look at the docs before approving.

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

PR Review: https://git.openjdk.org/jfx/pull/1996#pullrequestreview-3542174057


More information about the openjfx-dev mailing list