[PATCH] change CLOCKS_PER_SEC to match POSIX:XSI
Bob Wilson
bwilson@tensilica.com
Tue Oct 11 18:32:00 GMT 2005
Joel Sherrill wrote:
> Corinna Vinschen wrote:
> On Oct 6 13:19, Bob Wilson wrote:
>
> The XSI extension to POSIX requires that CLOCKS_PER_SEC be defined
> to one million. The current value in newlib, except for RTEMS and
> ARM, is 1000. Since this value seems pretty arbitrary, wouldn't it
> be better to go with the POSIX value? Here is a patch to change it.
>
> 2005-10-06 Bob Wilson <bob.wilson@acm.org>
>
> * libc/include/time.h (CLOCKS_PER_SEC): Define to 1000000.
>
>
> Careful, please. This could break newly compiled applications on
> systems using the CLOCKS_PER_SEC macro internally.
>
> I always thought this was an arbitrary and useless macro. On RTEMS
> and many other systems, the clock tick quantum is configurable.
>
> Why is 1000000 suddenly right?
I'm afraid I can't answer the "why" question. I stumbled upon this
requirement a while back when I was trying to decide upon a less
arbitrary value for this macro.
Judging from various mailing list archives, the change has been
somewhat controversial, but it appears that several other prominent C
libraries have adopted it.
The default value is easily overridden, so I'm not going to push very
hard to get the default changed. I thought it was at least worth
calling to your attention.
--Bob
More information about the Newlib
mailing list