[PATCH] Changes the default size of time_t to 64 bit.

Hans-Bernhard Bröker HBBroeker@t-online.de
Fri Aug 11 21:07:00 GMT 2017


Am 11.08.2017 um 22:15 schrieb Jeff Johnston:

> What about: --enable-time_t-compat to indicate it is a compat-type option
> indicating old behaviour?

That would violate rule-for-naming-things #1: never distinguish between 
two stages of development of something by naming one of them "new", or 
the other one "old".  If you do, you'll end up having to explain what 
the difference between really-very-new and 
brand-spanking-really-very-new actually is, and having no idea how to do 
that.

IMHO, the name of that option absolutely has to contain explicit mention 
of "32 bits", because its entire reason to exist is to make time_t 32 
bits.  Whether the actual time_t typedef then ends up being int, long, 
int_least32_t or what-not is of no interested to the person using that 
option, so has no business appearing in the option name.

The only alternative option syntax I might find acceptable would be 
something like --enable-time_t_override=<typename>, but only if 
configure can be taught to check that <typename> is actually a viable 
choice.



More information about the Newlib mailing list