On Tue, 13 Dec 2022 08:41:55 GMT, John Hendrikx <jhendr...@openjdk.org> wrote:
>> But is it correct? The previous code computed a deltaTime in seconds as >> `((currentNanos - startNanos) / 1e9)`, and then divided the >> `scrollMagnitude` by that. I think that the equivalent to the old code is: >> >> >> scrollMagnitude / (nanosPassed * NANOS_TO_SECONDS) > > It turns out it is not correct, the parenthesis are indeed required, not sure > how I could have missed it. I'm going to see if I can add a test to this as > they're missing to ensure it does the same before/after (aside from > precision). could we create a new method, like 'nanosToSeconds'? ------------- PR: https://git.openjdk.org/jfx/pull/966
