15 Apr
2012
15 Apr
'12
2:53 p.m.
On 04/15/2012 04:26 PM, Mathias Gaunard wrote:
You use nested ifs, not as good as a switch.
Also, the special code to deal with visitors that return void seems entirely unnecessary.
As for any_container.hpp, the clear_and_set stuff (which I suppose you use in operator=) is very wrong. You're literally copying an object to a different location and calling the destructor there.
My implementation isn't perfect for sure. But is it fixable? Is it a good working basis? If you think so, I'll create a GIT repository right now so everyone can contribute. Or maybe we should base our work on Jarryd Beck's implementation (that Paul Fultz gave a link for).