On Wed, 27 Aug 2025 18:40:07 GMT, Chen Liang <[email protected]> wrote:

> Is there a reason these fields are not static? Non-static final fields like 
> these are constant only in the class and not constant when used by other 
> classes, and require an instance to access, but I don't find these 
> differences convincing as the reason.

This looks like a mistake. In an earlier version of this change there were also 
variables to over-ride these for this instance named "stepDelay" and 
"stepLength" and they were inadvertently marked as static. 
I said they should be instance variables .. and that seems to have then been 
applied to the DEFAULT_* variables too which was not the request.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26969#discussion_r2305132231

Reply via email to