[ofw] [PATCH] opensm - standardize on a single Windows #define

Sasha Khapyorsky sashak at voltaire.com
Mon Oct 12 15:00:41 PDT 2009


On 10:16 Thu 08 Oct     , Smith, Stan wrote:
> Hefty, Sean wrote:
> >> Code cleanup, standardize on a single Windows #define '__WIN__';
> >> WIN32 --> __WIN__. Flip usage of ifndef WIN32 --> ifdef __GNUC__
> >
> > GNUC indicates a specific compiler, not a platform, which is what the
> > check is for.
> 
> To the best of my understanding OpenSM is built for Linux/OFED, Solaris and Windows.

Also some embedded platforms and we don't know what in a future.

'#ifdef __WIN__' and '#ifndef __GNUC__' are not equivalent.

Sasha



More information about the ofw mailing list