On Thu, 28 May 2020 15:23:38 GMT, Nir Lisker <nlis...@openjdk.org> wrote:

>> modules/javafx.graphics/src/main/java/javafx/animation/Animation.java line 
>> 402:
>> 
>>> 401:      */
>>> 402:     private void doSetCurrentRate(double value) {
>>> 403:         if (currentRate != null || !areNearEqual(value, 
>>> DEFAULT_CURRENT_RATE)) {
>> 
>> The above javadoc comment block appears after the javadoc comments for the 
>> private `currentRate` property variable and
>> before the public get/property methods for `currentRate`. This seems fragile 
>> (although the API docs pick up the right
>> one), so I recommend making this an ordinary block comment (i.e., `/*` 
>> rather than `/**`). Either that or move it below
>> the get/property methods (and add `@param value`).
>
> Doesn't the autogenerated JavaDoc for the `getX`, `setX` and `xProperty` 
> methods work by matching the method name to
> the property name regardless of they location in the class? 
> `doSetCurrentRate` shouldn't ever be able to interfere with
> that. I can add a `@param` anyway.

So it would seem. It still seems a bit fragile, but as long as it works, I 
won't push the point (we are very
inconsistent on this).

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

PR: https://git.openjdk.java.net/jfx/pull/196

Reply via email to