Am 17.11.2011 20:40, schrieb Georg Baum:

I like the principal idea (splitting parse_text() into smaller pieces), but
I don't like the way you do it, because the same limit will be quickly hit
again. I would propose something different: Move similar stuff into own
functions, e.g. all the font stuff (from textrm to noun could go into a bool
parse_font(), and then it would look like

... else if (t.cs() == "listof") {
...
} else if parse_font(...) {
        ;
} else if (t.cs() == "lyxadded" ...

That would indeed be much better.
As you are the tex2lyx expert, I would let you do this. I'm technically too limited with C++ to find a structure that suits our needs also for the future.
But if you like, I can comment on what you want to group as function.

thanks and regards
Uwe

Reply via email to