[ofa-general] Re: [PATCH 9/8] [ib-diag] ibping: add support for WinOF

[Sasha Khapyorsky]

On 16:36 Tue 17 Feb     , Sean Hefty wrote:
> > 	signal(SIGINT, report);
> > 	signal(SIGTERM, report);
> 
> Btw - I worked around adding cdecl before main by disabling the warning.  Since
> main must be cdecl by default, the compiler fixes it, but spits out a warning.
> For some reason unknown to me, the warning only occurs when building 32-bit
> apps. 
> 
> However, signal() requires that the function be cdecl as well.

Guess it is about report() function. Why to not make everything cdecl
(by using compiler/linker flag or some super-#pragma in config.h or so)?

> The above two
> calls fail to compile on 32-bit Windows platforms, so I'm still working on this.
> The simple approach of changing the compiler options doesn't work as easily as
> it looks like it should.  The WDK build environment is 'special'.

Ugh, I really fail to understand why WinOF cannot evaluate an option of
using less "special" build tools for WDK insensitive code (such as
user-space programs ported from linux) - it would solve all those issues
just magically. And we are not entered yet a more complicated porting
areas such as pthreads...

Sasha