[PATCH] Add timegm POSIX call [updated]

Brian Inglis Brian.Inglis@SystematicSw.ab.ca
Thu Aug 23 06:18:00 GMT 2018


On 2018-08-22 15:02, Freddie Chopin wrote:
> On Wed, 2018-08-22 at 12:12 -0700, Andrew Russell via newlib wrote:
>> -static const int DAYS_IN_MONTH[12] =
>> +static const int_least16_t DAYS_IN_MONTH[12] =
>>  {31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31};
> 
> Shouldn't the type here be actually int_least8_t and int_least16_t used
> here ->

or uint_least{8,16}_t?

>> @@ -58,11 +61,21 @@ static const int DAYS_IN_MONTH[12] =
>>  static const int _DAYS_BEFORE_MONTH[12] =
>>  {0, 31, 59, 90, 120, 151, 181, 212, 243, 273, 304, 334};

and initialization use {U}INT{8,16}_C(value) macros?

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada



More information about the Newlib mailing list