From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=45318 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Peqmm-0002kJ-DF for qemu-devel@nongnu.org; Mon, 17 Jan 2011 10:14:25 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Peqml-00087t-9L for qemu-devel@nongnu.org; Mon, 17 Jan 2011 10:14:24 -0500 Received: from mx1.redhat.com ([209.132.183.28]:46735) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Peqml-00087f-2B for qemu-devel@nongnu.org; Mon, 17 Jan 2011 10:14:23 -0500 Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id p0HFEMI4015609 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 17 Jan 2011 10:14:22 -0500 From: Jes.Sorensen@redhat.com Date: Mon, 17 Jan 2011 16:14:17 +0100 Message-Id: <1295277257-17425-3-git-send-email-Jes.Sorensen@redhat.com> In-Reply-To: <1295277257-17425-1-git-send-email-Jes.Sorensen@redhat.com> References: <1295277257-17425-1-git-send-email-Jes.Sorensen@redhat.com> Subject: [Qemu-devel] [PATCH 2/2] strtosz() use toupper() to simplify switch statement List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: kwolf@redhat.com, Alex.Williamson@redhat.com, eblake@redhat.com From: Jes Sorensen Signed-off-by: Jes Sorensen --- cutils.c | 7 +------ 1 files changed, 1 insertions(+), 6 deletions(-) diff --git a/cutils.c b/cutils.c index f97ef39..f7aa90c 100644 --- a/cutils.c +++ b/cutils.c @@ -323,16 +323,14 @@ ssize_t strtosz_suffix(const char *nptr, char **end, const char default_suffix) d = c; } } - switch (d) { + switch (toupper(d)) { case 'B': - case 'b': mul = 1; if (mul_required) { goto fail; } break; case 'K': - case 'k': mul = 1 << 10; break; case 0: @@ -340,15 +338,12 @@ ssize_t strtosz_suffix(const char *nptr, char **end, const char default_suffix) goto fail; } case 'M': - case 'm': mul = 1ULL << 20; break; case 'G': - case 'g': mul = 1ULL << 30; break; case 'T': - case 't': mul = 1ULL << 40; break; default: -- 1.7.3.4