diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2014-11-26 16:34:06 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2014-11-26 16:34:06 +0000 |
| commit | f1a5a0e3c3d32bb800a0a63a0cfcd1b25a8e3d4d (patch) | |
| tree | f1a2b380ea030ed0879fc575dd821262898f0eed | |
| parent | 59ef17152b1ad9a4c4d618ec085586e3e14f6e94 (diff) | |
| download | glibc-f1a5a0e3c3d32bb800a0a63a0cfcd1b25a8e3d4d.tar.gz | |
Fix sysdeps/mips/__longjmp.c warning.
This patch fixes
../sysdeps/mips/__longjmp.c:27:1: warning: function declaration isn't a prototype [-Wstrict-prototypes]
which arose I think from the MIPS16 changes (renaming the function to
____longjmp with an alias __longjmp, so a prior header prototype for
__longjmp no longer sufficed to prevent a warning). I've made the
function use a prototype definition, which is what we want for all
function definitions in glibc anyway.
Tested for MIPS.
* sysdeps/mips/__longjmp.c (____longjmp): Use prototype
definition.
| -rw-r--r-- | ChangeLog | 3 | ||||
| -rw-r--r-- | sysdeps/mips/__longjmp.c | 4 |
2 files changed, 4 insertions, 3 deletions
@@ -1,5 +1,8 @@ 2014-11-26 Joseph Myers <joseph@codesourcery.com> + * sysdeps/mips/__longjmp.c (____longjmp): Use prototype + definition. + * nptl/tst-cancel-self-cancelstate.c (do_test): Cast argument of pthread_cleanup_push to void *. diff --git a/sysdeps/mips/__longjmp.c b/sysdeps/mips/__longjmp.c index 5e6a3537b4..c2498c32c3 100644 --- a/sysdeps/mips/__longjmp.c +++ b/sysdeps/mips/__longjmp.c @@ -24,9 +24,7 @@ #endif static void __attribute__ ((nomips16)) -____longjmp (env_arg, val_arg) - __jmp_buf env_arg; - int val_arg; +____longjmp (__jmp_buf env_arg, int val_arg) { /* gcc 1.39.19 miscompiled the longjmp routine (as it did setjmp before the hack around it); force it to use $a1 for the longjmp value. |