On Wed, 13 May 2020 12:29:03 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> I agree that these methods are better suited there, but I'm not sure the 
>> same epsilon value will be suitable for other
>> places that will want to use these. That value is somewhat arbitrary anyway 
>> I think.
>
> Either is fine with me.

> but I'm not sure the same epsilon value will be suitable for other places 
> that will want to use these

True, EPSILON is defined in some other classes with different value. Only 
Animation classes define EPSILON as 1e-12. So
it may not be reasonable to include Animation specific helper method in common 
Util class. These methods would be more
suitable for Animation only util class. I think I am good with the current way.

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

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

Reply via email to