diff options
| author | Pat Pannuto <pat.pannuto@gmail.com> | 2017-01-15 19:12:02 -0500 |
|---|---|---|
| committer | Corinna Vinschen <corinna@vinschen.de> | 2017-01-16 10:14:28 +0100 |
| commit | e02866a1b43fae39d6842bd182de79e54d8e74cd (patch) | |
| tree | b7c85eb33419a6fd28dc10bc7c2df0710d610173 /newlib/libc/time | |
| parent | 411882e2faba331120ca9d82e4422216ddfb5e3b (diff) | |
| download | newlib-e02866a1b43fae39d6842bd182de79e54d8e74cd.zip newlib-e02866a1b43fae39d6842bd182de79e54d8e74cd.tar.gz newlib-e02866a1b43fae39d6842bd182de79e54d8e74cd.tar.bz2 | |
Add missing headers to fix implicit function defns
A few files were missing headers for memset/malloc, likely missed
because the files don't directly call the functions, rather they
come in via macros in libc/include/sys/reent.h:
#define _REENT_CHECK(var, what, type, size, init) do { \
struct _reent *_r = (var); \
if (_r->what == NULL) { \
_r->what = (type)malloc(size); \
#define _REENT_CHECK_ASCTIME_BUF(var) \
_REENT_CHECK(var, _asctime_buf, char *, _REENT_ASCTIME_SIZE, \
memset((var)->_asctime_buf, 0, _REENT_ASCTIME_SIZE))
Without these fixes, implicit function signatures are provided,
which gcc warns could cause aliasing issues down the line:
../../../../../../../newlib-2.5.0/newlib/libc/time/asctime.c:62:3: warning: type of 'memset' does not match original declaration [-Wlto-type-mismatch]
/Volumes/code/external/newlib-cygwin/newlib/libc/include/string.h:29:7: note: return value type mismatch
_PTR _EXFUN(memset,(_PTR, int, size_t));
^
/Volumes/code/external/newlib-cygwin/newlib/libc/include/string.h:29:7: note: 'memset' was previously declared here
/Volumes/code/external/newlib-cygwin/newlib/libc/include/string.h:29:7: note: code may be misoptimized unless -fno-strict-aliasing is used
../../../../../../../newlib-2.5.0/newlib/libc/time/asctime.c:62:3: warning: type of 'malloc' does not match original declaration [-Wlto-type-mismatch]
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: return value type mismatch
extern _PTR malloc _PARAMS ((size_t));
^
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: 'malloc' was previously declared here
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: code may be misoptimized unless -fno-strict-aliasing is used
../../../../../../../newlib-2.5.0/newlib/libc/time/lcltime.c:58:3: warning: type of 'malloc' does not match original declaration [-Wlto-type-mismatch]
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: return value type mismatch
extern _PTR malloc _PARAMS ((size_t));
^
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: 'malloc' was previously declared here
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: code may be misoptimized unless -fno-strict-aliasing is used
../../../../../../../newlib-2.5.0/newlib/libc/string/strsignal.c:70:3: warning: type of 'malloc' does not match original declaration [-Wlto-type-mismatch]
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: return value type mismatch
extern _PTR malloc _PARAMS ((size_t));
^
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: 'malloc' was previously declared here
/Volumes/code/external/newlib-cygwin/newlib/libc/include/malloc.h:37:13: note: code may be misoptimized unless -fno-strict-aliasing is used
Including the proper headers elminates the implicit function
signatures and these warnings.
Diffstat (limited to 'newlib/libc/time')
| -rw-r--r-- | newlib/libc/time/asctime.c | 2 | ||||
| -rw-r--r-- | newlib/libc/time/lcltime.c | 1 |
2 files changed, 3 insertions, 0 deletions
diff --git a/newlib/libc/time/asctime.c b/newlib/libc/time/asctime.c index 024310b..f56b511 100644 --- a/newlib/libc/time/asctime.c +++ b/newlib/libc/time/asctime.c @@ -47,6 +47,8 @@ ANSI C requires <<asctime>>. <<asctime>> requires no supporting OS subroutines. */ +#include <stdlib.h> +#include <string.h> #include <time.h> #include <_ansi.h> #include <reent.h> diff --git a/newlib/libc/time/lcltime.c b/newlib/libc/time/lcltime.c index 16162bf..2c9a25f 100644 --- a/newlib/libc/time/lcltime.c +++ b/newlib/libc/time/lcltime.c @@ -44,6 +44,7 @@ ANSI C requires <<localtime>>. <<localtime>> requires no supporting OS subroutines. */ +#include <stdlib.h> #include <time.h> #include <reent.h> |