[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Chicken-hackers] [PATCH] Add strings.h header to runtime.c
From: |
Peter Bex |
Subject: |
[Chicken-hackers] [PATCH] Add strings.h header to runtime.c |
Date: |
Tue, 17 Jan 2017 22:05:55 +0100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
Hello all,
While working on the header & feature macro portability issues that
have been plagueing us these past few days, I also noticed a warning
on MingW regarding an implicit declaration of strcasecmp.
See also https://sourceforge.net/p/mingw/bugs/1447/
According to this, strcasecmp and strncasecmp are supposed to be
declared in <strings.h> (a POSIX C header). Most systems will in
practice include that header from <string.h> (which is an ISO C header),
but not MingW. At least, not anymore.
Attached is a trivial fix (works on both branches). I decided not to
simply push it, because it may be a trivial fix, but who knows on what
platforms it may break... Theoretically at least, all our supported
platforms implement POSIX, so they should all support this header.
Cheers,
Peter
0001-Add-strings.h-for-strn-case-cmp.patch
Description: Text Data
signature.asc
Description: Digital signature
- [Chicken-hackers] [PATCH] Add strings.h header to runtime.c,
Peter Bex <=