[ofa-general] Re: [PATCH] management: uint -> unsigned replacement
Michael S. Tsirkin
mst at dev.mellanox.co.il
Tue Jun 26 06:24:57 PDT 2007
> Quoting Hal Rosenstock <halr at voltaire.com>:
> Subject: Re: [PATCH] management: uint -> unsigned replacement
>
> On Tue, 2007-06-26 at 06:20, Michael S. Tsirkin wrote:
> > Some management headers use uint type which (on my system)
>
> What's your system ?
SLES10.
> > is described as "old
> > compatibility name for C type". This type might not defined e.g. if
> > __STRICT_ANSI__ is set,
>
> Is strict ANSI a requirement ?
Not sure. The app in question does
#define _XOPEN_SOURCE 600
> > so it is best to avoid its usage at least in headers.
> > Replace by unsigned in all headers.
> >
> > Signed-off-by: Michael S. Tsirkin <mst at dev.mellanox.co.il>
> >
> > ---
> >
> > Hal can you apply this please? As a separate question:
> > I didn't go over .c files (we don't build them with strict ansi now),
> > but maybe removing uint there is a good idea, too?
>
> Yes but it will take more than this to make them strict ANSI.
>
> Is this as an OFED 1.2 follow on or just for master ?
You decide.
--
MST
More information about the general
mailing list