Markus Armbruster <arm...@redhat.com> writes: > See PATCH 1/3 for rationale. > > Markus Armbruster (3): > Purge the silly GLib "Basic Types", except for gboolean > Purge GLib's gboolean, it's a trap for the unwary > checkpatch: Keep out the GLib silliness we just purged
Changing QEMU code to work around checkpatch's failing is insane. When interacting with glib, use glib types. Not using those types just makes the code more difficult to understand. Regards, Anthony Liguori > > async.c | 12 +++++------- > coroutine-gthread.c | 8 ++++---- > hw/9pfs/virtio-9p-coth.c | 2 +- > hw/fw_cfg.c | 6 +++--- > hw/qdev-monitor.c | 2 +- > hw/qdev.c | 4 ++-- > hw/xen_disk.c | 6 +++--- > include/qom/object.h | 8 ++++---- > main-loop.c | 2 +- > qapi/opts-visitor.c | 10 +++++----- > qapi/qmp-input-visitor.c | 4 ++-- > qemu-img.c | 4 ++-- > qga/channel-posix.c | 25 +++++++++++++------------ > qga/channel-win32.c | 26 +++++++++++++------------- > qga/channel.h | 10 +++++----- > qga/commands-posix.c | 6 +++--- > qga/commands.c | 2 +- > qga/guest-agent-command-state.c | 4 ++-- > qga/guest-agent-core.h | 2 +- > qga/main.c | 16 ++++++++-------- > qom/container.c | 2 +- > qom/object.c | 26 +++++++++++++------------- > qtest.c | 8 ++++---- > scripts/checkpatch.pl | 29 +++++++++++++++++++++++++++++ > target-alpha/cpu.c | 4 ++-- > target-arm/cpu.c | 2 +- > target-arm/helper.c | 4 ++-- > target-i386/cpu.c | 2 +- > target-m68k/helper.c | 4 ++-- > target-openrisc/cpu.c | 4 ++-- > target-ppc/translate_init.c | 12 ++++++------ > tests/libqtest.c | 20 ++++++++++---------- > tests/test-visitor-serialization.c | 12 ++++++------ > trace/simple.c | 8 ++++---- > vl.c | 6 +++--- > 35 files changed, 165 insertions(+), 137 deletions(-) > > -- > 1.7.11.7