diff options
author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2024-11-10 12:37:53 -0800 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2024-11-10 12:37:53 -0800 |
commit | 906bf4a86d174ed20fda925d8f82de023aa26d2d (patch) | |
tree | 31256335b56e2aa1c800f794c7bbd44a6e7788ae /src | |
parent | c0768ab687fa07f83d0030dd161f4aa49cbe84f8 (diff) |
PutCommandResource: add do ... while (0) to avoid -Wextra-semi-stmt warnings
Clears 4 warnings from clang of the form:
ParseCmd.c:158:43: warning: empty expression statement has no effect;
remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
PutCommandResource(options[i].value);
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Part-of: <https://gitlab.freedesktop.org/xorg/lib/libx11/-/merge_requests/272>
Diffstat (limited to 'src')
-rw-r--r-- | src/ParseCmd.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/ParseCmd.c b/src/ParseCmd.c index 67719081..4a07ac46 100644 --- a/src/ParseCmd.c +++ b/src/ParseCmd.c @@ -92,11 +92,11 @@ XrmParseCommand( char **argend; #define PutCommandResource(value_str) \ - { \ + do { \ XrmStringToBindingQuarkList( \ options[i].specifier, start_bindings, start_quarks); \ XrmQPutStringResource(pdb, bindings, quarks, value_str); \ - } /* PutCommandResource */ + } while (0) /* PutCommandResource */ myargc = (*argc); argend = argv + myargc; |