[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] Purge the silly GLib Basic Types
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] Purge the silly GLib Basic Types |
Date: |
Fri, 18 Jan 2013 10:17:11 -0600 |
User-agent: |
Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Markus Armbruster <address@hidden> 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