diff options
| author | Eric Blake | 2018-11-20 21:36:28 +0100 |
|---|---|---|
| committer | Laurent Vivier | 2018-12-11 18:28:46 +0100 |
| commit | 3381466dc95f5ee7087caa8fee77920bc03b0e62 (patch) | |
| tree | 7d6e75fb956fa7e78b697f30c848629e48aee95f /scripts/checkpatch.pl | |
| parent | qapi: Reduce Makefile boilerplate (diff) | |
| download | qemu-3381466dc95f5ee7087caa8fee77920bc03b0e62.tar.gz qemu-3381466dc95f5ee7087caa8fee77920bc03b0e62.tar.xz qemu-3381466dc95f5ee7087caa8fee77920bc03b0e62.zip | |
misc: Avoid UTF-8 in error messages
While most developers are now using UTF-8 environments, it's
harder to guarantee that error messages will be output to
a multibyte locale. Rather than risking error messages that
get corrupted into mojibake when the user runs qemu in a
non-multibyte locale, let's stick to straight ASCII error
messages, rather than assuming that our use of UTF-8 in source
code string constants will work unchanged in other locales.
Found with:
$ LC_ALL=C git grep -l $'".*[\x80-\xff].*"' origin -- '**/*.[ch]' | cat
origin:hw/misc/tmp105.c
origin:hw/misc/tmp421.c
Reported-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20181120203628.2367003-1-eblake@redhat.com>
[lv: added command line to find non ASCII characters]
Signed-off-by: Laurent Vivier <laurent@vivier.eu>
Diffstat (limited to 'scripts/checkpatch.pl')
0 files changed, 0 insertions, 0 deletions
