diff options
author | David Robillard <d@drobilla.net> | 2025-03-24 15:51:14 -0400 |
---|---|---|
committer | David Robillard <d@drobilla.net> | 2025-03-24 17:38:48 -0400 |
commit | 8785ef4b74e6a54cdd7c83e69c01a72d8e200314 (patch) | |
tree | 820fc28f0bce14ac6b4cfa1b1b963ae953a7380f /src | |
parent | ae2848aabbbd953c6c2ebf58850c04250baaacff (diff) | |
download | serd-8785ef4b74e6a54cdd7c83e69c01a72d8e200314.tar.gz serd-8785ef4b74e6a54cdd7c83e69c01a72d8e200314.tar.bz2 serd-8785ef4b74e6a54cdd7c83e69c01a72d8e200314.zip |
Remove unnecessary condition and improve error message test coverage
Not all of these cases are grammatically interesting, but the invalid
characters are significant for error reporting (for example, error messages
shouldn't contain raw control characters).
Diffstat (limited to 'src')
-rw-r--r-- | src/reader.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/reader.c b/src/reader.c index a7e83717..d5ac379b 100644 --- a/src/reader.c +++ b/src/reader.c @@ -35,8 +35,7 @@ r_err_char(SerdReader* const reader, const char* const kind, const int c) { const SerdStatus st = SERD_ERR_BAD_SYNTAX; - return (c < 0x20 || c == 0x7F || c > 0x10FFFF) - ? r_err(reader, st, "bad %s character\n", kind) + return (c < 0x20 || c == 0x7F) ? r_err(reader, st, "bad %s character\n", kind) : (c == '\'' || c >= 0x80) ? r_err(reader, st, "bad %s character U+%04X\n", kind, (uint32_t)c) : r_err(reader, st, "bad %s character '%c'\n", kind, c); |