summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexander Monakov <amonakov@ispras.ru>2017-07-04 22:53:12 +0300
committerRich Felker <dalias@aerifal.cx>2017-07-04 18:04:52 -0400
commit2e6e08423b95b1b2bce7568f650c2b69d8d8fb8b (patch)
treeb49d8bbf91de7423ac2f305d16b1013563610d5a
parentbc42dcbfd9466c22d8ab0f8235e5091689a95aad (diff)
downloadmusl-2e6e08423b95b1b2bce7568f650c2b69d8d8fb8b.tar.gz
remove ineffective compiler assist from printf
The switch statement has no 'default:' case and the function ends immediately following the switch, so the extra comparison did not communicate any extra information to the compiler.
-rw-r--r--src/stdio/vfprintf.c2
-rw-r--r--src/stdio/vfwprintf.c2
2 files changed, 0 insertions, 4 deletions
diff --git a/src/stdio/vfprintf.c b/src/stdio/vfprintf.c
index 8c81efc2..15356f53 100644
--- a/src/stdio/vfprintf.c
+++ b/src/stdio/vfprintf.c
@@ -107,8 +107,6 @@ union arg
static void pop_arg(union arg *arg, int type, va_list *ap)
{
- /* Give the compiler a hint for optimizing the switch. */
- if ((unsigned)type > MAXSTATE) return;
switch (type) {
case PTR: arg->p = va_arg(*ap, void *);
break; case INT: arg->i = va_arg(*ap, int);
diff --git a/src/stdio/vfwprintf.c b/src/stdio/vfwprintf.c
index 1e6e47cc..e87ad42a 100644
--- a/src/stdio/vfwprintf.c
+++ b/src/stdio/vfwprintf.c
@@ -98,8 +98,6 @@ union arg
static void pop_arg(union arg *arg, int type, va_list *ap)
{
- /* Give the compiler a hint for optimizing the switch. */
- if ((unsigned)type > MAXSTATE) return;
switch (type) {
case PTR: arg->p = va_arg(*ap, void *);
break; case INT: arg->i = va_arg(*ap, int);