Boost C++ Libraries: Ticket #3035: non-standard integral types not fully supported

Fri, 15 May 2009 08:22:45 GMT

Well, __int64 is a bad example being an exceptional case, I actually meant the whole list of __int8, __int16 etc

nasonov — Fri, 15 May 2009 11:39:55 GMT

I added a TODO to the integral_promotion.hpp some time ago which says "common macro for this #if. Or better yet, PP SEQ of non-standard types."

So, if we define BOOST_TT_AUX_NON_STANDARD_INTS:

#if (defined(BOOST_MSVC) && (BOOST_MSVC < 1300)) \
    || (defined(BOOST_INTEL_CXX_VERSION) && defined(_MSC_VER) && (BOOST_INTEL_CXX_VERSION <= 600)))
# define BOOST_TT_AUX_NON_STANDARD_INTS \
    (__int8) (unsigned __int8)  \
    (__int16)(unsigned __int16) \
    (__int32)(unsigned __int32)
#elif defined(__BORLANDC__) && (__BORLANDC__ == 0x600) && (_MSC_VER < 1300)
# define BOOST_TT_AUX_NON_STANDARD_INTS \
    (__int8) (unsigned __int8)  \
    (__int16)(unsigned __int16) \
    (__int32)(unsigned __int32) \
    (__int64)(unsigned __int64)
#endif

we should be able to change code in is_integral.hpp to something like:

#ifdef BOOST_TT_AUX_NON_STANDARD_INTS
#define BOOST_TT_AUX_DECL(r, data, type) BOOST_TT_AUX_BOOL_TRAIT_CV_SPEC1(is_integral, type, true)
BOOST_PP_SEQ_FOR_EACH(BOOST_TT_AUX_DECL, _, BOOST_TT_AUX_NON_STANDARD_INTS)
#undef BOOST_TT_AUX_DECL
#endif

and similarly in integral_promotion.hpp:

#ifdef BOOST_TT_AUX_NON_STANDARD_INTS
#define BOOST_TT_AUX_DECL(r, data, type) BOOST_TT_AUX_PROMOTE_NONSTANDARD_TYPE(type)
BOOST_PP_SEQ_FOR_EACH(BOOST_TT_AUX_DECL, _, BOOST_TT_AUX_NON_STANDARD_INTS)
#undef BOOST_TT_AUX_DECL
#endif

It's not as easy for make_signed and make_unsigned but they convert a non-standard type to a corrensponding standard type, e.g. make_unsigned<int8>::type -> unsigned char.

This macro can be also used in tests.

Alex

John Maddock — Fri, 15 May 2009 16:52:44 GMT

A couple of points here:

1) This is only an issue for obsolete MSVC versions that treat int8 etc as distinct types right? 2) I'm fairly sure looking at the code that is_unsigned/is_signed/make_signed/make_unsigned should all support these types already. ie no special code is needed as long as is_integral recognises the type as an integral type. 3) I don't currently have a compiler installed that can test the above :-(

So... can someone please tese the above?

Or if the submitter is unable to confirm this is an issue, I suggest we just close the issue down with an "invalid" tag.

Regards, John.

status changed; resolution set

John Maddock — Tue, 04 Aug 2009 16:16:08 GMT

status new → closed
resolution → invalid

Closing as invalid: I believe this is a non-issue for all except obsolete compilers.