[Oorexx-rfes] [ oorexx-Feature Requests-2869733 ] C9x integer types

[SourceForge.net <no...@so...>]

Feature Requests item #2869733, was opened at 2009-09-29 08:25
Message generated for change (Settings changed) made by bigrixx
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684733&aid=2869733&group_id=119701

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: APIs
>Group: v4.2.0
>Status: Pending
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Laurent Crepel (lcrepel)
Assigned to: Rick McGuire (bigrixx)
Summary: C9x integer types

Initial Comment:
Hi all,

The header file rexxapitypes.h defines C9X Standard integer types unconditionally. Unfortunately we are using a third party library that defines these types too.
As a result the build of our project fails due to definition conflicts! (error C2371: 'int8_t' : redefinition; different basic types ) 
We brings to your awareness that these types are already present under some platforms (e.g Linux) in the header inttypes.h, for windows there is an open project by the name of msinttypes (http://code.google.com/p/msinttypes/) to add the support of this standard to Visual Studio. 
The key point is that we should at the very minimum test to see if the header file inttypes.h is available (or/and the integer types are already defined.)
As a solution we suggest the below changes to the rexxapitypes.h file:

#ifdef NATIVE_INTYPES 
/* If the plattform  and compiler supports the 
 * C9X 'proposed'  standard for interger types, 
 * use the provided inttypes.h
 * otherwise use our own defintions of the same types
 */

#include <inttypes.h> 
#else 
// portable ANSI types
typedef short int16_t;
typedef unsigned short uint16_t;
typedef int int32_t;
typedef unsigned int uint32_t;
typedef char int8_t;
typedef unsigned char uint8_t;
typedef signed __int64 int64_t;
typedef unsigned __int64 uint64_t;
#endif //NATIVE_INTYPES 


Regards,

Laurent CREPEL.


----------------------------------------------------------------------

>Comment By: Rick McGuire (bigrixx)
Date: 2011-02-03 12:33

Message:
This is an excellent suggestion.

Committed revision 6693

----------------------------------------------------------------------

Comment By: Rick McGuire (bigrixx)
Date: 2009-09-29 08:28

Message:
Changing this to an enhancement request. 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=684733&aid=2869733&group_id=119701