Boost C++ Libraries: Ticket #11269: Fusion adapting an "empty" struct is no longer possible in boost 1.58

damien.buhl@… — Wed, 06 May 2015 05:26:30 GMT

We did not had this usage in our testcases and I didn't even thought about it as an interesting use of adapt struct.

I understand however now that one might want to do this. I'm looking at it, I might fix this fast.

damien.buhl@… — Wed, 06 May 2015 05:41:34 GMT

There is a workaround for your issue until I get it fixed. Compile with the following flag disabled:

g++ -DBOOST_PP_VARIADICS=0 main.cpp

This will take the code path using sequences only which works as in 1.57. I'm still fixing the other case however.

damien.buhl@… — Wed, 06 May 2015 06:33:31 GMT

So I have a fix working and passing all tests + this new test case with or without VARIADICS enabled.

Is there other use case than ADAPT_STRUCT with empty struct ? Do you think ADAPT_ADT might also be needed with empty struct ? ASSOC_STRUCT?

djowel can you remember which use cases might makes sense ?

The fix is for the moment in my branch on github : daminetreg/fusion branch : feature/fix-ticket-11269-adapt-empty-struct

I also made a PR for it in github boostorg/fusion#75

Joel de Guzman — Wed, 06 May 2015 07:37:55 GMT

We should probably support all cases of empy structs/classes. it's kind wiered with ADAPT_ADT, but i can't tell when someone will use such a thing. maybe in generic code?

damien.buhl@… — Fri, 15 May 2015 07:22:09 GMT

Happily since my PR #75 we do support all cases of empty structs/classes. I made a new PR #76 with all the testcases.

This bug is fixed. :)

status changed; resolution set

Joel de Guzman — Fri, 12 Jun 2015 06:50:51 GMT

status new → closed
resolution → fixed