diff options
| author |   krw <krw@openbsd.org> | 2016-12-20 21:10:29 +0000 |
|---|---|---|
| committer | krw <krw@openbsd.org> | 2016-12-20 21:10:29 +0000 |
| commit | 5378bd166dab5724a5222d3cd66e45062328d7f0 (patch) | |
| tree | f3a02ea236d97d9f31c6e098dabe6283af46a892 | |
| parent | Fix compiler warnings generated by clang. This matches upstream commit (diff) | |
| download | wireguard-openbsd-5378bd166dab5724a5222d3cd66e45062328d7f0.tar.xz wireguard-openbsd-5378bd166dab5724a5222d3cd66e45062328d7f0.zip | |
Tweak generated .c output so switch statements always have a
'default:' case.
Several hundred "not handled" warnings go away.
ok deraadt@
| -rw-r--r-- | usr.bin/rpcgen/rpc_cout.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/usr.bin/rpcgen/rpc_cout.c b/usr.bin/rpcgen/rpc_cout.c index 1a04f04fa67..0a76c95d791 100644 --- a/usr.bin/rpcgen/rpc_cout.c +++ b/usr.bin/rpcgen/rpc_cout.c @@ -1,4 +1,4 @@ -/* $OpenBSD: rpc_cout.c,v 1.25 2015/08/20 22:32:41 deraadt Exp $ */ +/* $OpenBSD: rpc_cout.c,v 1.26 2016/12/20 21:10:29 krw Exp $ */ /* $NetBSD: rpc_cout.c,v 1.6 1996/10/01 04:13:53 cgd Exp $ */ /* @@ -374,12 +374,12 @@ emit_union(def) fprintf(fout, "\t\tbreak;\n"); } dflt = def->def.un.default_decl; + fprintf(fout, "\tdefault:\n"); if (dflt != NULL) { if (!streq(dflt->type, "void")) { int len = strlen(def->def_name) + strlen(format) + strlen(dflt->name) + 1; - fprintf(fout, "\tdefault:\n"); object = malloc(len); if (object == NULL) { fprintf(stderr, "Fatal error: no memory\n"); @@ -396,10 +396,9 @@ emit_union(def) print_ifstat(2, dflt->prefix, dflt->type, dflt->rel, dflt->array_max, object, dflt->name); free(object); - fprintf(fout, "\t\tbreak;\n"); } + fprintf(fout, "\t\tbreak;\n"); } else { - fprintf(fout, "\tdefault:\n"); fprintf(fout, "\t\treturn (FALSE);\n"); } |