diff options
| author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2023-10-08 12:12:30 -0700 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2023-10-08 12:12:30 -0700 |
| commit | d4eef750160356c8a9dc55d5e28c3546f8ef5418 (patch) | |
| tree | 2d56615e375c88a8e52ebb5b8a481017c95e526a | |
| parent | 340dd95c6a566b01bfaf9ce2baa0b49f07e3ac22 (diff) | |
XkbCFReportError: avoid -Wformat-nonliteral warning
xkbconfig.c: In function ‘XkbCFReportError’:
xkbconfig.c:1344:5: warning: format not a string literal,
argument types not checked [-Wformat-nonliteral]
1344 | fprintf(file, msg, line);
| ^~~~~~~
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
| -rw-r--r-- | src/xkbconfig.c | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/src/xkbconfig.c b/src/xkbconfig.c index a62512d..3bbb837 100644 --- a/src/xkbconfig.c +++ b/src/xkbconfig.c @@ -1299,49 +1299,49 @@ XkbCFReportError(FILE *file, char *name, int error, int line) switch (error) { case XkbCF_BadAlloc: - msg = "allocation failed\n"; + msg = "allocation failed"; break; case XkbCF_UnterminatedString: - msg = "unterminated string on line %d"; + msg = "unterminated string"; break; case XkbCF_MissingIdent: - msg = "expected identifier on line %d"; + msg = "expected identifier"; break; case XkbCF_MissingEquals: - msg = "expected '=' on line %d"; + msg = "expected '='"; break; case XkbCF_ExpectedEOS: - msg = "expected ';' or newline on line %d"; + msg = "expected ';' or newline"; break; case XkbCF_ExpectedBoolean: - msg = "expected a boolean value on line %d"; + msg = "expected a boolean value"; break; case XkbCF_ExpectedInteger: - msg = "expected a numeric value on line %d"; + msg = "expected a numeric value"; break; case XkbCF_ExpectedString: - msg = "expected a string on line %d"; + msg = "expected a string"; break; case XkbCF_ExpectedModifier: - msg = "expected a modifier name on line %d"; + msg = "expected a modifier name"; break; case XkbCF_ExpectedControl: - msg = "expected a control name on line %d"; + msg = "expected a control name"; break; case XkbCF_ExpectedAXOption: - msg = "expected an AccessX option on line %d"; + msg = "expected an AccessX option"; break; case XkbCF_ExpectedOperator: - msg = "expected '+' or '-' on line %d"; + msg = "expected '+' or '-'"; break; case XkbCF_ExpectedOORGroupBehavior: - msg = "expected wrap, clamp or group number on line %d"; + msg = "expected wrap, clamp or group number"; break; default: - msg = "unknown error on line %d"; + msg = "unknown error"; break; } - fprintf(file, msg, line); + fprintf(file, "%s on line %d", msg, line); if (name) fprintf(file, " of %s\n", name); else |