[ofw] Re: [PATCH] osmtest - Add OSM_CDECL to main() declaration

[Smith, Stan]

Hefty, Sean wrote:
>>> Sean maintains a separate set of patches applied to IB diags in
>>> order to
>> address issues like the x86 requirement for __cdecl on main().
>>> Since OSM_CDECL was already defined in OFED opensm, it did not seem
>>> to be a
>> major concession to prefix main() with it.
>>> Options:
>>> 1) utilize OSM_CDECL
>>> 2) throw an ugly ifdef __WIN__ or ifdef __linux__ around main()
>>> 3) Create more Windows-only patches required for the Windows
>>> implementation of OFED opensm & tools.
>>
>> Any options for windows compiler about call conventions?
>
> To clarify, I maintain a single patch to handle this for the signal()
> callback handler, not the ib-diags main() functions.
>
> Here's what I use in the sources file for building the diags:
>
> MSC_WARNING_LEVEL = /W3 /WX /wd4007
>
> - Sean

Thanks for sharing the info at your earliest convenience.