From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47660) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YtO2b-0005T6-5l for qemu-devel@nongnu.org; Fri, 15 May 2015 18:25:14 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YtO2W-00048p-1v for qemu-devel@nongnu.org; Fri, 15 May 2015 18:25:13 -0400 Received: from resqmta-po-04v.sys.comcast.net ([2001:558:fe16:19:96:114:154:163]:34425) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YtO2V-00044K-L5 for qemu-devel@nongnu.org; Fri, 15 May 2015 18:25:08 -0400 From: Eric Blake Date: Fri, 15 May 2015 16:24:58 -0600 Message-Id: <1431728700-18055-1-git-send-email-eblake@redhat.com> Subject: [Qemu-devel] [PATCH 0/2] Use bool for QBool List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, berto@igalia.com, mst@redhat.com, armbru@redhat.com, mdroth@linux.vnet.ibm.com, lcapitulino@redhat.com, mreitz@redhat.com, afaerber@suse.de Passing around an 'int' for a QBool type is weird, when we already use a C99 compiler and have a sane 'bool' that does just fine. I half-debated sending this through qemu-trivial, but think it better belongs through the QMP tree. There turned out to be few enough clients that I grouped it into two patches touching a number of files each; but I'm also okay with splitting into finer-grained patches that focus on fewer files at a time if that is desired. Eric Blake (2): qobject: Use 'bool' for qbool qobject: Use 'bool' inside qdict block/qapi.c | 2 +- block/quorum.c | 4 ++-- block/vvfat.c | 4 ++-- hmp.c | 40 ++++++++++++++++++++-------------------- hw/pci/pcie_aer.c | 4 ++-- include/qapi/qmp/qbool.h | 8 ++++---- include/qapi/qmp/qdict.h | 4 ++-- monitor.c | 12 ++++++------ qapi/qmp-input-visitor.c | 2 +- qapi/qmp-output-visitor.c | 2 +- qobject/json-parser.c | 6 +++--- qobject/qbool.c | 8 ++++---- qobject/qdict.c | 8 ++++---- qobject/qjson.c | 2 +- qom/object.c | 4 ++-- tests/check-qjson.c | 11 ++++++----- tests/test-qmp-event.c | 4 ++-- tests/test-qmp-output-visitor.c | 6 +++--- util/qemu-option.c | 2 +- 19 files changed, 67 insertions(+), 66 deletions(-) -- 2.1.0