Re: [PATCH 2/2] Provide <memory.h>

[Corinna Vinschen <vinschen@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1185 bytes --]

On Nov 22 09:49, Eric Blake wrote:
> On 11/22/2016 12:59 AM, Sebastian Huber wrote:
> > Provide <memory.h> for all standard Newlib targets and remove
> > Cygwin-specific header.  Most POSIX like systems provide this historic
> > header.
> > 
> > Signed-off-by: Sebastian Huber <sebastian.huber@embedded-brains.de>
> > ---
> >  newlib/libc/include/memory.h   |  4 ++++
> >  winsup/cygwin/include/memory.h | 15 ---------------
> >  2 files changed, 4 insertions(+), 15 deletions(-)
> >  create mode 100644 newlib/libc/include/memory.h
> >  delete mode 100644 winsup/cygwin/include/memory.h
> > 
> > diff --git a/newlib/libc/include/memory.h b/newlib/libc/include/memory.h
> > new file mode 100644
> > index 0000000..f4a14fc
> > --- /dev/null
> > +++ b/newlib/libc/include/memory.h
> > @@ -0,0 +1,4 @@
> > +#ifndef _MEMORY_H
> > +#define	_MEMORY_H
> > +#include <string.h>
> > +#endif /* !_MEMORY_H */
> 
> Would it be worth adding
> 
> #warn This header is deprecated; consider using <string.h>
> 
> to get people to modernize their code?

Nice idea, but glibc is not doing that either...


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]