Subject: Re: [boost] [accumulators] broken on msvc
From: Edward Diener (eldiener_at_[hidden])
Date: 2014-06-16 09:11:06
On 6/16/2014 2:38 AM, Eric Niebler wrote:
> On 6/15/2014 10:36 PM, Eric Niebler wrote:
>> On 6/15/2014 2:20 PM, Andrey Semashev wrote:
>>> On Sunday 15 June 2014 21:18:12 JÃ¼rgen Hunold wrote:
>>>> The question remains: Who fixes this where?
>>> As far as I can see, is_empty.hpp does not use BOOST_PP_CAT when it doesn't
>>> include it, so the header is correct. Accumulators should include cat.hpp
>>> whenever it uses BOOST_PP_CAT, so at least accumulators_fwd.hpp has a bug.
>> I've added the missing #include on develop. Hopefully, that fixes things.
> Looks like it doesn't fix things. If Edward's is indeed the problematic
> commit, I wonder if he'd be willing to look into this. Edward?
> This is the commit:
> *All* msvc accumulators tests are currently broken on develop by this.
I have backed out all the changes I made to preprocessor so the
accumulators problem should now be fixed. I will look at the issue of
why the macro was failing with my VC++ changes when I have some time.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk