diff options
| author |   tb <tb@openbsd.org> | 2017-12-15 13:04:11 +0000 |
|---|---|---|
| committer | tb <tb@openbsd.org> | 2017-12-15 13:04:11 +0000 |
| commit | 588305a03d43a55c3db29a065ff88f6420f90423 (patch) | |
| tree | fca64876daced96bf7c119acc7cfa865f368550c | |
| parent | enable the 'wDn' test now that jot has been fixed. (diff) | |
| download | wireguard-openbsd-588305a03d43a55c3db29a065ff88f6420f90423.tar.xz wireguard-openbsd-588305a03d43a55c3db29a065ff88f6420f90423.zip | |
Instead of pointlessly falling through other cases and ending up doing
goto fmt_broken, do it explicitly. Makes following the code path through
the switch a bit easier.
Prompted by a request by martijn
| -rw-r--r-- | usr.bin/jot/jot.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/usr.bin/jot/jot.c b/usr.bin/jot/jot.c index 63c77bbb6fb..c01ebb0fb3a 100644 --- a/usr.bin/jot/jot.c +++ b/usr.bin/jot/jot.c @@ -1,4 +1,4 @@ -/* $OpenBSD: jot.c,v 1.37 2017/12/15 12:17:49 tb Exp $ */ +/* $OpenBSD: jot.c,v 1.38 2017/12/15 13:04:11 tb Exp $ */ /* $NetBSD: jot.c,v 1.3 1994/12/02 20:29:43 pk Exp $ */ /*- @@ -426,6 +426,7 @@ getformat(void) intdata = true; break; } + goto fmt_broken; case 'O': case 'U': /* %lO and %lU are undefined */ if (!longdata) { @@ -433,11 +434,13 @@ getformat(void) intdata = nosign = true; break; } + goto fmt_broken; case 'c': if (!(intdata | longdata)) { chardata = true; break; } + /* FALLTHROUGH */ case 'h': case 'n': case 'p': case 'q': case 's': case 'L': case '$': case '*': goto fmt_broken; |