[openib-general] [PATCH 1/3 v2] osm: Changes for windows compatability

Sasha Khapyorsky sashak at voltaire.com
Wed Dec 27 09:25:29 PST 2006


On 17:46 Wed 27 Dec     , Yevgeny Kliteynik wrote:
> Hi Hal.
> 
> Fixing windows compilation problems
> [V2 - Previous patch had an error]
> 
> Signed-off-by: Yevgeny Kliteynik <kliteyn at dev.mellanox.co.il>
> ---
>  osm/include/iba/ib_types.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/osm/include/iba/ib_types.h b/osm/include/iba/ib_types.h
> index 723e8b9..ec65b64 100644
> --- a/osm/include/iba/ib_types.h
> +++ b/osm/include/iba/ib_types.h
> @@ -59,9 +59,10 @@ BEGIN_C_DECLS
>           #define OSM_EXPORT	__declspec(dllimport)
>      #endif
>      #define OSM_API __stdcall
> +    #define OSM_CDECL __cdecl
>  #else
>      #define OSM_EXPORT	extern
>      #define OSM_API
> +    #define OSM_CDECL
>      #define __ptr64
>  #endif

Just wondering, how does lack of __cdecl hurt windows compilation (in
the context of where those __cdecl is used)?

What is the reason to have both __stdcall and __cdecl (and what is the
default)?

Sasha




More information about the general mailing list