diff options
| author |   tb <tb@openbsd.org> | 2018-07-10 17:45:52 +0000 |
|---|---|---|
| committer | tb <tb@openbsd.org> | 2018-07-10 17:45:52 +0000 |
| commit | 0971b67f68eabe84e58ba202abe73d50c878d1ac (patch) | |
| tree | b15e774a68a10234788ba9c13aac96e9df7b23cf /gnu/usr.bin/cvs/src | |
| parent | Make legacy interrupts work in acpipci(4). (diff) | |
| download | wireguard-openbsd-0971b67f68eabe84e58ba202abe73d50c878d1ac.tar.xz wireguard-openbsd-0971b67f68eabe84e58ba202abe73d50c878d1ac.zip | |
Fix a few, but not all, clang warnings: Use "%s" to print modifiable
strings, add a couple of braces, ansify a few functions, add and remove
a few extra parens.
ok jcs
Diffstat (limited to 'gnu/usr.bin/cvs/src')
| -rw-r--r-- | gnu/usr.bin/cvs/src/admin.c | 2 | ||||
| -rw-r--r-- | gnu/usr.bin/cvs/src/hash.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/gnu/usr.bin/cvs/src/admin.c b/gnu/usr.bin/cvs/src/admin.c index c5995ab831f..935e1b4c99b 100644 --- a/gnu/usr.bin/cvs/src/admin.c +++ b/gnu/usr.bin/cvs/src/admin.c @@ -776,7 +776,7 @@ admin_fileproc (callerdat, finfo) if (delta->other_delta == NULL) delta->other_delta = getlist(); - if (n = findnode (delta->other_delta, "commitid")) + if ((n = findnode (delta->other_delta, "commitid"))) { error (0, 0, "%s: revision %s already has commitid %s", rcs->path, rev, n->data); diff --git a/gnu/usr.bin/cvs/src/hash.c b/gnu/usr.bin/cvs/src/hash.c index 7e562b89f0d..66ec46a7eda 100644 --- a/gnu/usr.bin/cvs/src/hash.c +++ b/gnu/usr.bin/cvs/src/hash.c @@ -306,7 +306,7 @@ findnode (list, key) /* This probably should be "assert (list != NULL)" (or if not we should document the current behavior), but only if we check all the callers to see if any are relying on this behavior. */ - if ((list == (List *) NULL)) + if (list == (List *) NULL) return ((Node *) NULL); assert (key != NULL); |