From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50776) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WQ3MJ-0000Op-Sd for qemu-devel@nongnu.org; Tue, 18 Mar 2014 19:23:53 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WQ3MD-0000Uf-V1 for qemu-devel@nongnu.org; Tue, 18 Mar 2014 19:23:47 -0400 Received: from mail-qc0-x235.google.com ([2607:f8b0:400d:c01::235]:55298) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WQ3MD-0000Ua-Qp for qemu-devel@nongnu.org; Tue, 18 Mar 2014 19:23:41 -0400 Received: by mail-qc0-f181.google.com with SMTP id e9so8693397qcy.40 for ; Tue, 18 Mar 2014 16:23:41 -0700 (PDT) From: "Gabriel L. Somlo" Date: Tue, 18 Mar 2014 19:23:19 -0400 Message-Id: <1395185009-26532-3-git-send-email-somlo@cmu.edu> In-Reply-To: <1395185009-26532-1-git-send-email-somlo@cmu.edu> References: <1395185009-26532-1-git-send-email-somlo@cmu.edu> Subject: [Qemu-devel] [v4 PATCH 02/12] SMBIOS: Use macro to set smbios defaults List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: alex.williamson@redhat.com, agraf@suse.de, armbru@redhat.com, gsomlo@gmail.com, kevin@koconnor.net, kraxel@redhat.com, imammedo@redhat.com, lersek@redhat.com The function smbios_set_defaults() uses a repeating code pattern for each field. This patch replaces that pattern with a macro. Signed-off-by: Gabriel Somlo --- hw/i386/smbios.c | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/hw/i386/smbios.c b/hw/i386/smbios.c index 89dc070..f4ee7b4 100644 --- a/hw/i386/smbios.c +++ b/hw/i386/smbios.c @@ -256,18 +256,17 @@ static void smbios_build_type_1_fields(void) } } +#define SMBIOS_SET_DEFAULT(field, value) \ + if (!field) { \ + field = value; \ + } + void smbios_set_defaults(const char *manufacturer, const char *product, const char *version) { - if (!type1.manufacturer) { - type1.manufacturer = manufacturer; - } - if (!type1.product) { - type1.product = product; - } - if (!type1.version) { - type1.version = version; - } + SMBIOS_SET_DEFAULT(type1.manufacturer, manufacturer); + SMBIOS_SET_DEFAULT(type1.product, product); + SMBIOS_SET_DEFAULT(type1.version, version); } uint8_t *smbios_get_table(size_t *length) -- 1.8.5.3