Jean-Marc Lasgouttes wrote: > Found this while investigating 2050/2060: when creating a macro with > "math-macro aa", the type is set to "" instead of "newcommand". > > It is an obvious typo in the code, so I'll just commit.
It may be an obvious typo, but it looks to me like it should be
MathMacroTemplate itself that decides what to do with s2. Shouldn't
the whole
string const type = s2.empty() ? "newcommand" : s2;
be moved into the MathMacroTemplate constructor?
--
Angus
