aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2014-11-26 16:34:06 +0000
committerJoseph Myers <joseph@codesourcery.com>2014-11-26 16:34:06 +0000
commitf1a5a0e3c3d32bb800a0a63a0cfcd1b25a8e3d4d (patch)
treef1a2b380ea030ed0879fc575dd821262898f0eed
parent59ef17152b1ad9a4c4d618ec085586e3e14f6e94 (diff)
downloadglibc-f1a5a0e3c3d32bb800a0a63a0cfcd1b25a8e3d4d.zip
glibc-f1a5a0e3c3d32bb800a0a63a0cfcd1b25a8e3d4d.tar.gz
glibc-f1a5a0e3c3d32bb800a0a63a0cfcd1b25a8e3d4d.tar.bz2
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--ChangeLog3
-rw-r--r--sysdeps/mips/__longjmp.c4
2 files changed, 4 insertions, 3 deletions
diff --git a/ChangeLog b/ChangeLog
index 0797a15..5ec7f0d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -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 5e6a353..c2498c3 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.