Re: [Emc-developers] [PATCH] Fix servo_mult calculation in emcmotSetCycleTime

[Jeff E. <je...@un...>]

I applied a form of this patch on v2.5_branch.

commit 42b8e518ca7c412c5e897c88c0efc12427deb429
Author: Jeff Epler <je...@un...>
Date:   Sat Dec 24 10:25:48 2011 -0600

    motion: remove useless call to ceil()
    
    This call was useless, and reportedly causes some unspecified older
    compilers to encounter internal compiler errors.  Others may warn
    about the implicit conversion from int to double (as this makes the
    code incompatible with pre-ANSI C compilers, something we all care
    deeply about).
    
    The original change may have intended to be something more like
        ceil((double)traj_period_nsec / nsec)
    but since all we really care about is that the servo multiplier is
    an exact integer, it's preferable to preserve the existing behavior
    and just remove the useless function call.

Thank you for the original report.

Jeff