]> git.lyx.org Git - lyx.git/blobdiff - src/tex2lyx/text.cpp
Cmake build: tex2lyx
[lyx.git] / src / tex2lyx / text.cpp
index 01626f3a500892313a2afbde6d830c93171e0657..f406869153eddb30f1ca4fdf8bab2a04ecc4dfb4 100644 (file)
@@ -628,27 +628,40 @@ void output_command_layout(ostream & os, Parser & p, bool outer,
        }
        context.check_deeper(os);
        context.check_layout(os);
-       unsigned int optargs = 0;
-       while (optargs < context.layout->optargs) {
+       // FIXME: Adjust to format 446!
+       // Since format 446, layouts do not require anymore all optional
+       // arguments before the required ones. Needs to be implemented!
+       int optargs = 0;
+       while (optargs < context.layout->optArgs()) {
                eat_whitespace(p, os, context, false);
                if (p.next_token().cat() == catEscape ||
                    p.next_token().character() != '[')
                        break;
                p.get_token(); // eat '['
-               begin_inset(os, "Argument\n");
+               // FIXME: Just a workaround. InsetArgument::updateBuffer
+               //        will compute a proper ID for all "999" Arguments
+               //        (which is also what lyx2lyx produces).
+               //        However, tex2lyx should be able to output proper IDs
+               //        itself.
+               begin_inset(os, "Argument 999\n");
                os << "status collapsed\n\n";
                parse_text_in_inset(p, os, FLAG_BRACK_LAST, outer, context);
                end_inset(os);
                eat_whitespace(p, os, context, false);
                ++optargs;
        }
-       unsigned int reqargs = 0;
-       while (reqargs < context.layout->reqargs) {
+       int reqargs = 0;
+       while (reqargs < context.layout->requiredArgs()) {
                eat_whitespace(p, os, context, false);
                if (p.next_token().cat() != catBegin)
                        break;
                p.get_token(); // eat '{'
-               begin_inset(os, "Argument\n");
+               // FIXME: Just a workaround. InsetArgument::updateBuffer
+               //        will compute a proper ID for all "999" Arguments
+               //        (which is also what lyx2lyx produces).
+               //        However, tex2lyx should be able to output proper IDs
+               //        itself.
+               begin_inset(os, "Argument 999\n");
                os << "status collapsed\n\n";
                parse_text_in_inset(p, os, FLAG_BRACE_LAST, outer, context);
                end_inset(os);
@@ -1417,7 +1430,7 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                                // This hack must be removed once bug 8049 is fixed!
                                if ((it + 1 != et) && (it + 2 != et || *it2 != '\n'))
                                        os << "\n\\end_layout\n\\begin_layout Verbatim\n";
-                       } else 
+                       } else
                                os << *it;
                }
                os << "\n\\end_layout\n\n";
@@ -1458,7 +1471,7 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                                        handle_ert(os, "\\", parent_context);
                                else if (*it == '$')
                                        handle_ert(os, "$", parent_context);
-                               else 
+                               else
                                        os << *it;
                        }
                        handle_ert(os, "\\end{" + name + "}",
@@ -1597,14 +1610,15 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                }
                context.check_deeper(os);
                // handle known optional and required arguments
-               // layouts require all optional arguments before the required ones
+               // FIXME: Since format 446, layouts do not require anymore all optional
+               // arguments before the required ones. Needs to be implemented!
                // Unfortunately LyX can't handle arguments of list arguments (bug 7468):
                // It is impossible to place anything after the environment name,
                // but before the first \\item.
                if (context.layout->latextype == LATEX_ENVIRONMENT) {
                        bool need_layout = true;
-                       unsigned int optargs = 0;
-                       while (optargs < context.layout->optargs) {
+                       int optargs = 0;
+                       while (optargs < context.layout->optArgs()) {
                                eat_whitespace(p, os, context, false);
                                if (p.next_token().cat() == catEscape ||
                                    p.next_token().character() != '[')
@@ -1614,15 +1628,20 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                                        context.check_layout(os);
                                        need_layout = false;
                                }
-                               begin_inset(os, "Argument\n");
+                               // FIXME: Just a workaround. InsetArgument::updateBuffer
+                               //        will compute a proper ID for all "999" Arguments
+                               //        (which is also what lyx2lyx produces).
+                               //        However, tex2lyx should be able to output proper IDs
+                               //        itself.
+                               begin_inset(os, "Argument 999\n");
                                os << "status collapsed\n\n";
                                parse_text_in_inset(p, os, FLAG_BRACK_LAST, outer, context);
                                end_inset(os);
                                eat_whitespace(p, os, context, false);
                                ++optargs;
                        }
-                       unsigned int reqargs = 0;
-                       while (reqargs < context.layout->reqargs) {
+                       int reqargs = 0;
+                       while (reqargs < context.layout->requiredArgs()) {
                                eat_whitespace(p, os, context, false);
                                if (p.next_token().cat() != catBegin)
                                        break;
@@ -1631,7 +1650,12 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                                        context.check_layout(os);
                                        need_layout = false;
                                }
-                               begin_inset(os, "Argument\n");
+                               // FIXME: Just a workaround. InsetArgument::updateBuffer
+                               //        will compute a proper ID for all "999" Arguments
+                               //        (which is also what lyx2lyx produces).
+                               //        However, tex2lyx should be able to output proper IDs
+                               //        itself.
+                               begin_inset(os, "Argument 999\n");
                                os << "status collapsed\n\n";
                                parse_text_in_inset(p, os, FLAG_BRACE_LAST, outer, context);
                                end_inset(os);
@@ -1920,27 +1944,27 @@ void copy_file(FileName const & src, string dstname)
 }
 
 
-/// Parse a NoWeb Scrap section. The initial "<<" is already parsed.
+/// Parse a NoWeb Chunk section. The initial "<<" is already parsed.
 void parse_noweb(Parser & p, ostream & os, Context & context)
 {
        // assemble the rest of the keyword
        string name("<<");
-       bool scrap = false;
+       bool chunk = false;
        while (p.good()) {
                Token const & t = p.get_token();
                if (t.asInput() == ">" && p.next_token().asInput() == ">") {
                        name += ">>";
                        p.get_token();
-                       scrap = (p.good() && p.next_token().asInput() == "=");
-                       if (scrap)
+                       chunk = (p.good() && p.next_token().asInput() == "=");
+                       if (chunk)
                                name += p.get_token().asInput();
                        break;
                }
                name += t.asInput();
        }
 
-       if (!scrap || !context.new_layout_allowed ||
-           !context.textclass.hasLayout(from_ascii("Scrap"))) {
+       if (!chunk || !context.new_layout_allowed ||
+           !context.textclass.hasLayout(from_ascii("Chunk"))) {
                cerr << "Warning: Could not interpret '" << name
                     << "'. Ignoring it." << endl;
                return;
@@ -1954,7 +1978,7 @@ void parse_noweb(Parser & p, ostream & os, Context & context)
        // always must be in an own paragraph.
        context.new_paragraph(os);
        Context newcontext(true, context.textclass,
-               &context.textclass[from_ascii("Scrap")]);
+               &context.textclass[from_ascii("Chunk")]);
        newcontext.check_layout(os);
        os << name;
        while (p.good()) {
@@ -1966,12 +1990,12 @@ void parse_noweb(Parser & p, ostream & os, Context & context)
                else {
                        ostringstream oss;
                        Context tmp(false, context.textclass,
-                                   &context.textclass[from_ascii("Scrap")]);
+                                   &context.textclass[from_ascii("Chunk")]);
                        tmp.need_end_layout = true;
                        tmp.check_layout(oss);
                        os << subst(t.asInput(), "\n", oss.str());
                }
-               // The scrap chunk is ended by an @ at the beginning of a line.
+               // The chunk is ended by an @ at the beginning of a line.
                // After the @ the line may contain a comment and/or
                // whitespace, but nothing else.
                if (t.asInput() == "@" && p.prev_token().cat() == catNewline &&
@@ -2258,6 +2282,12 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        skip_braces(p);
                }
 
+               else if (t.asInput() == "<"
+                        && p.next_token().asInput() == "<" && noweb_mode) {
+                       p.get_token();
+                       parse_noweb(p, os, context);
+               }
+
                else if (t.asInput() == "<" && p.next_token().asInput() == "<") {
                        context.check_layout(os);
                        begin_inset(os, "Quotes ");
@@ -2267,12 +2297,6 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        skip_braces(p);
                }
 
-               else if (t.asInput() == "<"
-                        && p.next_token().asInput() == "<" && noweb_mode) {
-                       p.get_token();
-                       parse_noweb(p, os, context);
-               }
-
                else if (t.cat() == catSpace || (t.cat() == catNewline && ! p.isParagraph()))
                        check_space(p, os, context);
 
@@ -2724,10 +2748,12 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        begin_inset(os, "Caption\n");
                        Context newcontext(true, context.textclass, 0, 0, context.font);
                        newcontext.check_layout(os);
+                       // FIXME InsetArgument is now properly implemented in InsetLayout
+                       //       (for captions, but also for others)
                        if (p.next_token().cat() != catEscape &&
                            p.next_token().character() == '[') {
                                p.get_token(); // eat '['
-                               begin_inset(os, "Argument\n");
+                               begin_inset(os, "Argument 1\n");
                                os << "status collapsed\n";
                                parse_text_in_inset(p, os, FLAG_BRACK_LAST, outer, context);
                                end_inset(os);
@@ -3599,8 +3625,8 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                              context, "\\lang",
                                              context.font.language, lang);
                }
-               
-               else if (prefixIs(t.cs(), "text") 
+
+               else if (prefixIs(t.cs(), "text")
                         && is_known(t.cs().substr(4), preamble.polyglossia_languages)) {
                        // scheme is \textLANGUAGE{text} where LANGUAGE is in polyglossia_languages[]
                        string lang;