mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: "Randy.Dunlap" <rdunlap@xenotime.net>
Cc: lkml <linux-kernel@vger.kernel.org>, akpm <akpm@osdl.org>
Subject: Re: [PATCH] CodingStyle: add typedefs chapter
Date: Mon, 01 May 2006 18:06:06 +0100	[thread overview]
Message-ID: <1146503166.2885.137.camel@hades.cambridge.redhat.com> (raw)
In-Reply-To: <20060430174426.a21b4614.rdunlap@xenotime.net>

On Sun, 2006-04-30 at 17:44 -0700, Randy.Dunlap wrote:
> + (b) Clear integer types, where the abstraction _helps_ avoid confusion
> +     whether it is "int" or "long".
> +
> +     u8/u16/u32 are perfectly fine typedefs. 

No, u8/u16/u32 are fall into category (d):

 (d) New types which are identical to standard C99 types, in certain
     exceptional circumstances.

     Although it would only take a short amount of time for the eyes and
     brain to become accustomed to the standard types like 'uint32_t',
     some people object to their use anyway.

     Therefore, the gratuitous 'u8/u16/u32/u64' types and their signed
     equivalents which are identical to standard types are permitted --
     although they are not mandatory.

 (e) Types safe for use in userspace. 

     In certain structures which are visible to userspace, we cannot
     require C99 types and cannot use the 'u32' form above. Thus, we
     use __u32 and similar types in all structures which are shared
     with userspace.

-- 
dwmw2


  parent reply	other threads:[~2006-05-01 17:06 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-01  0:44 Randy.Dunlap
2006-05-01  7:28 ` Michael Buesch
2006-05-01 15:34   ` Randy.Dunlap
2006-05-01 14:00 ` Jan Engelhardt
2006-05-01 14:19   ` Alexey Dobriyan
2006-05-01 14:46     ` linux-os (Dick Johnson)
2006-05-01 15:01       ` Jiri Slaby
2006-05-01 15:33   ` Artem B. Bityutskiy
2006-05-01 16:58   ` David Woodhouse
2006-05-01 20:20     ` Jan Engelhardt
2006-05-01 20:45       ` Jiri Slaby
2006-05-01 21:01         ` Jan Engelhardt
2006-05-01 21:07           ` David Woodhouse
2006-05-01 21:38             ` Alexey Dobriyan
2006-05-02 10:40               ` Jörn Engel
2006-05-02 13:17             ` Jes Sorensen
2006-05-01 17:06 ` David Woodhouse [this message]
2006-05-01 20:48   ` Randy.Dunlap
2006-05-02  0:37   ` Johannes Stezenbach
2006-05-02 13:28     ` David Woodhouse
2006-05-02 14:20       ` Johannes Stezenbach
2006-05-02 14:31         ` David Woodhouse
2006-05-02 17:11           ` Randy.Dunlap
2006-05-02 18:41             ` Linus Torvalds
2006-05-02 18:50               ` David Woodhouse
2006-05-02 19:07                 ` Linus Torvalds
2006-05-02 19:20                   ` Marcel Siegert
2006-05-02 19:44                     ` Linus Torvalds
2006-05-02 23:22                   ` David Woodhouse
2006-05-03 19:41                     ` Randy.Dunlap
2006-05-03 21:52                       ` David Woodhouse
2006-05-03 22:09                         ` Randy.Dunlap
2006-05-03 22:10                           ` David Woodhouse
2006-05-01 21:23 ` Daniel Barkalow

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1146503166.2885.137.camel@hades.cambridge.redhat.com \
    --to=dwmw2@infradead.org \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@xenotime.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®