mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Renate Meijer <kleuske@xs4all.nl>
To: Dag Arne Osvik <da@osvik.no>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
	linux-kernel@vger.kernel.org, Adrian Bunk <bunk@stusta.de>,
	Andreas Schwab <schwab@suse.de>,
	Al Viro <viro@parcelfarce.linux.theplanet.co.uk>,
	Kyle Moffett <mrmacman_g4@mac.com>,
	Grzegorz Kulewski <kangur@polcom.net>,
	Kenneth Johansson <ken@kenjo.org>
Subject: Re: Use of C99 int types
Date: Mon, 4 Apr 2005 22:30:52 +0200	[thread overview]
Message-ID: <c3057294a216d19047bdca201fc97e2f@xs4all.nl> (raw)
In-Reply-To: <42511BD8.4060608@osvik.no>


On Apr 4, 2005, at 12:50 PM, Dag Arne Osvik wrote:

> Renate Meijer wrote:
>
>>
>> On Apr 4, 2005, at 12:08 AM, Kyle Moffett wrote:
>>
>>> On Apr 03, 2005, at 16:25, Kenneth Johansson wrote:
>>>
>>>> But is this not exactly what Dag Arne Osvik was trying to do ??
>>>> uint_fast32_t means that we want at least 32 bits but it's OK with
>>>> more if that happens to be faster on this particular architecture.
>>>> The problem was that the C99 standard types are not defined anywhere
>>>> in the kernel headers so they can not be used.
>>>
>>>
>>> Uhh, so what's wrong with "int" or "long"?
>>
>
> Nothing, as long as they work as required.  And Grzegorz Kulewski 
> pointed out that unsigned long is required to be at least 32 bits, 
> fulfilling the present need for a 32-bit or wider type.

>> My point exactly, though I agree with Kenneth that adding the C99 
>> types
>> would be a Good Thing.
>
>
> If it leads to better code, then indeed it would be.

At least a 32 bit integer is guaranteed to stay an 32 bit integer 
(should one be required)
though multiple incarnations of the compiler.

>   However, Al Viro disagrees and strongly hints they would lead to 
> worse code.

When used improperly. The #define Al Viro objected to, is 
objectionable. It's highly
misleading, as Mr. Viro pointed out. I fail to see where he made 
comments on stdint.h
as such.

>> And if you don't, you imply some special requirement, which, if none 
>> really exists, is
>> misleading.
>
> And in this case there is such a requirement.

Apart from the integer having 32 bits?

>   Anyway, I've already decided to use unsigned long as a replacement 
> for uint_fast32_t in my implementation.

Ok. I can live with that.

Regards,

Renate Meijer. 


  reply	other threads:[~2005-04-04 20:50 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-03 11:55 Dag Arne Osvik
2005-04-03 12:05 ` Stephen Rothwell
2005-04-03 12:30   ` Dag Arne Osvik
2005-04-03 13:27     ` Andreas Schwab
2005-04-03 22:48       ` Dag Arne Osvik
2005-04-03 23:05         ` Al Viro
2005-04-03 23:17         ` Grzegorz Kulewski
2005-04-03 23:20           ` Dag Arne Osvik
2005-04-04  0:05         ` Adrian Bunk
2005-04-03 18:13     ` Al Viro
2005-04-03 23:03       ` Dag Arne Osvik
2005-04-04  3:08         ` Herbert Xu
2005-04-04  8:42           ` Dag Arne Osvik
2005-04-03 19:23     ` Renate Meijer
2005-04-03 20:25       ` Kenneth Johansson
2005-04-03 22:08         ` Kyle Moffett
2005-04-04 10:05           ` Renate Meijer
2005-04-04 10:50             ` Dag Arne Osvik
2005-04-04 20:30               ` Renate Meijer [this message]
2005-04-04 20:57                 ` Al Viro
2005-04-04 21:25                   ` Richard B. Johnson
2005-04-04 21:49                     ` Kyle Moffett
2005-04-05  9:23                       ` Renate Meijer
2005-04-05 11:27                         ` Kyle Moffett
     [not found]                           ` <09142f748cc6ad2bf4fffab7a5519226@xs4all.nl>
2005-04-05 22:11                             ` Kyle Moffett
     [not found]                               ` <eb65bccddde63541ae4b7b2d6c4c32d3@xs4all.nl>
2005-04-06 21:11                                 ` Kyle Moffett
2005-04-07 11:28                                   ` Renate Meijer
2005-04-05 12:18                       ` Richard B. Johnson
2005-04-05 21:47                         ` Kyle Moffett
2005-04-05  8:49                   ` Renate Meijer

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=c3057294a216d19047bdca201fc97e2f@xs4all.nl \
    --to=kleuske@xs4all.nl \
    --cc=bunk@stusta.de \
    --cc=da@osvik.no \
    --cc=kangur@polcom.net \
    --cc=ken@kenjo.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mrmacman_g4@mac.com \
    --cc=schwab@suse.de \
    --cc=sfr@canb.auug.org.au \
    --cc=viro@parcelfarce.linux.theplanet.co.uk \
    /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®