Re: roken strptime addition to cygwin

[Corinna Vinschen <cygwin-patches at cygwin dot com>]

On Fri, Jan 18, 2002 at 08:17:44AM -0500, Mark Bradshaw wrote:
> BTW, the last was CC'd to the newlib-patches group without being subscribed,
> and I got a bounce.  I didn't see a signup for it on the newlib site, so I
> just hoped that it would allow emails in without a subscription.  Guess not.
> Could someone with appropriate uber powers please check that one in.

While the newlib change is alredy checked in I have a question
to the function itself.

Wouldn't it make more sense to apply that function to newlib
itself?

What bugs me is that the newlib/libc/time/strftime.c source
already defines the needed static arrays with day names and
month names.  It looks to me as if it would be useful to
put strptime() into that strftime.c file since otherwise
we're ending up with these arrays twice.  The next advantage
would be that not only Cygwin has an advantage of that but
all newlib targets.

So I would like to suggest that you move the function implementaton
into newlib/libc/time/strftime.c, Mark.  The extern declaration in
libc/include/time.h could be moved out of the `#ifdef __CYGWIN__'
brackets and the cygwin.din patch would stay as it is.

Is that ok?

Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Developer                                mailto:cygwin@cygwin.com
Red Hat, Inc.