Index: src/LayoutModuleList.h

+///
+typedef std::list<std::string> LayoutModuleList;
+}

Why not support/string_list.h (see docstring_list.h)?

+#ifndef TEX2LYX
+               if (!lm) {
+                       docstring const msg =
+                                               bformat(_("The module %1$s has been 
requested by\n"
+                                               "this document but has not been 
found in the list of\n"
+                                               "available modules. If you recently 
installed it, you\n"
+                                               "probably need to reconfigure 
LyX.\n"), from_utf8(modName));
+                       frontend::Alert::warning(_("Module not available"),


Having to test on TEX2LYX rings a bell. What about throwing an exception (with the error message) instead?

In general the patch makes sense, but I guess more code has to be moved to TextClass in order to
be useful to tex2lyx.

JMarc

Reply via email to