|
Boost : |
Subject: Re: [boost] [config] Boost PP and variadic macros
From: Edward Diener (eldiener_at_[hidden])
Date: 2011-04-04 11:32:51
On 4/4/2011 10:57 AM, John Maddock wrote:
>>>>> True enough, but there weren't many changes required: try SVN Trunk
>>>>> now,
>>>>> Boost.Config should compile in C mode as well as C++ mode now. Tested
>>>>> locally with msvc, gcc (all the versions I could find!), intel and
>>>>> sun.
>>>>> I've also added a C-language test case, and manually inspected the
>>>>> compiler and platform config files for C++ specific code. Lets see how
>>>>> that works out...
>>>>
>>>> I have some more patches. Do I attach them here or send them to you ?
>>>
>>> Either is fine.
>>
>> Attached are some more patches.
>
> Unless I'm missing something, those shouldn't be needed - I deliberately
> disabled std lib configuration when __cplusplus isn't set as it makes no
> real sense in C mode?
Agreed. But you do have checks for __cpluscplus in some of the other
config/stdlib header files so I made those patches to conform. Maybe you
could now remove those others, and just ignore those patches I sent.
Attached are two more, which are relevant, and one which is a cosmetic
spelling change which confused me when I first read it and means almost
the opposite of what you meant to say in a comment. These are the last I
found.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk