Re: Patch - Add restrict to time.h

[Joel Sherrill <joel dot sherrill at oarcorp dot com>]

On 11/25/2013 5:20 AM, Corinna Vinschen wrote:
> On Nov 25 11:46, Corinna Vinschen wrote:
>> On Nov 24 11:45, Joel Sherrill wrote:
>>> Hi
>>> 
>>> This should be the next to the last patch in the Google Code-In
>>> add restrict keyword series. wchar.h should be the last.
>>> 
>>> This patch has one oddity -- strftime and wcsftime are built 
>>> from the same source as best I can tell. And there are multiple
>>> versions of them in the same file to support different build
>>> options.
>>> 
>>> Please review this one a little closer than the others.
>> 
>> I did and it looks good to me.  There are only two functions, 
>> strftime and wcsftime, and two variations, with and without C99 
>> extensions, to get either completeness or smaller codesize.
>> This does not affect the definition of the entry points.
>> 
>> Please apply.

Done. Thanks.

> It looks like your patches miss a few restricts in the header 
> files.  I was just having another look and I found at least
> ctime_r im time.h and renameat in stdio.h.  Yes, the latter is
> Cygwin-only, but that doesn't mean we should leave them out, does
> it?  Would you mind to have another sweep at this to pick up the
> missed ones?

I followed up in another email but there was no intention to miss
anything on any target. If POSIX 2013 has restrict, it should have
been added. These do not have restrict in POSIX 2013.

> 
> Thanks, Corinna
> 



-- 
Joel Sherrill, Ph.D.             Director of Research & Development
joel.sherrill@OARcorp.com        On-Line Applications Research
Ask me about RTEMS: a free RTOS  Huntsville AL 35805
Support Available                (256) 722-9985