On 20.11.2011 21:13, Jean-Marc Lasgouttes wrote:
Le Sun Nov 20 20:44:09 2011, [email protected] a écrit :
+#include "support/metahash.h"
+
+typedef boost::mpl::string<'\\end','_lay','out'>   end_layout;
+typedef boost::mpl::string<'\\end','in','set'>     end_inset;

Note how you forgot the _ above in end_inset. This kind of problem is a
typical reason why I do not like much this thing. What problem does it
solve that we cannot solve by grouping of if/else clauses? This is not
even particularly elegant IMO.


But such strings are added only one time, so being more careful than
with other stuff should avoid it.
(And if we would have unit tests it would be no problem.)

JMarc

Reply via email to