From mboxrd@z Thu Jan 1 00:00:00 1970 From: Anthony Liguori Subject: Re: [Qemu-devel] [PATCH] Introduce short names for fixed width integer types Date: Mon, 08 Aug 2011 08:00:06 -0500 Message-ID: <4E3FDDD6.9070501@codemonkey.ws> References: <1312808190-31074-1-git-send-email-avi@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org To: Avi Kivity Return-path: Received: from mail-gx0-f174.google.com ([209.85.161.174]:45876 "EHLO mail-gx0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751140Ab1HHNAK (ORCPT ); Mon, 8 Aug 2011 09:00:10 -0400 Received: by gxk21 with SMTP id 21so638797gxk.19 for ; Mon, 08 Aug 2011 06:00:09 -0700 (PDT) In-Reply-To: <1312808190-31074-1-git-send-email-avi@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 08/08/2011 07:56 AM, Avi Kivity wrote: > QEMU deals with a lot of fixed width integer types; their names > (uint64_t etc) are clumsy to use and take up a lot of space. > > Following Linux, introduce shorter names, for example U64 for > uint64_t. Except Linux uses lower case letters. I personally think Linux style is wrong here. The int8_t types are standard types. Besides, we save lots of characters by using 4-space tabs instead of 8-space tabs. We can afford to spend some of those saved characters on using proper type names :-) Regards, Anthony Liguori > > Signed-off-by: Avi Kivity > --- > qemu-common.h | 9 +++++++++ > 1 files changed, 9 insertions(+), 0 deletions(-) > > diff --git a/qemu-common.h b/qemu-common.h > index 0fdecf1..52a2300 100644 > --- a/qemu-common.h > +++ b/qemu-common.h > @@ -112,6 +112,15 @@ static inline char *realpath(const char *path, char *resolved_path) > int qemu_main(int argc, char **argv, char **envp); > #endif > > +typedef int8_t S8; > +typedef uint8_t U8; > +typedef int16_t S16; > +typedef uint16_t U16; > +typedef int32_t S32; > +typedef uint32_t U32; > +typedef int64_t S64; > +typedef uint64_t U64; > + > /* bottom halves */ > typedef void QEMUBHFunc(void *opaque); >