diff options
| author | Dimitry Andric <dimitry@andric.com> | 2020-02-19 21:24:59 +0100 |
|---|---|---|
| committer | Matt Turner <mattst88@gmail.com> | 2020-02-19 14:30:58 -0800 |
| commit | e200d0d41ef3158ea717206c1490e499a0c07f1b (patch) | |
| tree | 8eeeebc64ef494379c860a9214f272a8f190f99a /xkbcomp.c | |
| parent | 29fda8e50e4a4a127348e63fcf9f47600beab93c (diff) | |
| download | xorg-app-xkbcomp-e200d0d41ef3158ea717206c1490e499a0c07f1b.tar.gz | |
Don't compare with string literals
xkbcomp.c:228:37: error: result of comparison against a string literal is unspecified (use an explicit string comparison function instead) [-Werror,-Wstring-compare]
if ((argv[i][0] != '-') || (uStringEqual(argv[i], "-")))
^~~~~~~~~~~~~~~~~~~~~~~~~~
./utils.h:195:30: note: expanded from macro 'uStringEqual'
^~~~~~~~~~~~~~~~~~~~~
./utils.h:198:38: note: expanded from macro 'uStringCompare'
(s1)!=(s2):strcmp(s1,s2))
^ ~~~~
Don't attempt to do this macro trickery, and simply use strcmp instead,
where it applies.
Bug: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=244235
Diffstat (limited to 'xkbcomp.c')
| -rw-r--r-- | xkbcomp.c | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -225,7 +225,7 @@ parseArgs(int argc, char *argv[]) for (i = 1; i < argc; i++) { int itmp; - if ((argv[i][0] != '-') || (uStringEqual(argv[i], "-"))) + if ((argv[i][0] != '-') || (strcmp(argv[i], "-") == 0)) { if (!xkblist) { @@ -651,7 +651,7 @@ parseArgs(int argc, char *argv[]) ERROR("No input file specified\n"); return False; } - else if (uStringEqual(inputFile, "-")) + else if (strcmp(inputFile, "-") == 0) { inputFormat = INPUT_XKB; } @@ -755,7 +755,7 @@ parseArgs(int argc, char *argv[]) return False; } } - else if ((!outputFile) && (inputFile) && uStringEqual(inputFile, "-")) + else if ((!outputFile) && (inputFile) && (strcmp(inputFile, "-") == 0)) { int len = strlen("stdin") + strlen(fileTypeExt[outputFormat]) + 2; outputFile = uTypedCalloc(len, char); @@ -930,7 +930,7 @@ main(int argc, char *argv[]) } if (inputFile != NULL) { - if (uStringEqual(inputFile, "-")) + if (strcmp(inputFile, "-") == 0) { file = stdin; inputFile = "stdin"; @@ -1132,7 +1132,7 @@ main(int argc, char *argv[]) } if (outputFile != NULL) { - if (uStringEqual(outputFile, "-")) + if (strcmp(outputFile, "-") == 0) outputFile = "stdout"; else { |