]> git.lyx.org Git - features.git/commitdiff
parse_text (tex2lyx): get rid of the monstrous if-then-else clause
authorJuergen Spitzmueller <spitz@lyx.org>
Mon, 5 Mar 2018 08:08:14 +0000 (09:08 +0100)
committerJuergen Spitzmueller <spitz@lyx.org>
Mon, 5 Mar 2018 08:08:14 +0000 (09:08 +0100)
Rather than that, continue in the loop if a condition is met.

This fixes tex2lyx for MINGW, which has a cluase-nesting threshold.

Fixes: #9552
src/tex2lyx/text.cpp

index d1d368cb2a4eb6937d9b6f375993bd8484ce7618..327659545d1f740df3c749d5c6dce30a89191411 100644 (file)
@@ -2524,7 +2524,14 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                is_nonCJKJapanese = false;
        }
 
+       bool have_cycled = false;
        while (p.good()) {
+               // Leave hear only after at least one cycle
+               if (have_cycled && flags & FLAG_LEAVE) {
+                       flags &= ~FLAG_LEAVE;
+                       break;
+               }
+
                Token const & t = p.get_token();
 #ifdef FILEDEBUG
                debugToken(cerr, t, flags);
@@ -2561,6 +2568,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                //
                // cat codes
                //
+               have_cycled = true;
                bool const starred = p.next_token().asInput() == "*";
                string const starredname(starred ? (t.cs() + '*') : t.cs());
                if (t.cat() == catMath) {
@@ -2589,42 +2597,48 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                // output, but looks ugly in LyX.
                                eat_whitespace(p, os, context, false);
                        }
+                       continue;
                }
 
-               else if (t.cat() == catSuper || t.cat() == catSub)
+               if (t.cat() == catSuper || t.cat() == catSub) {
                        cerr << "catcode " << t << " illegal in text mode\n";
+                       continue;
+               }
 
                // Basic support for english quotes. This should be
                // extended to other quotes, but is not so easy (a
                // left english quote is the same as a right german
                // quote...)
-               else if (t.asInput() == "`" && p.next_token().asInput() == "`") {
+               if (t.asInput() == "`" && p.next_token().asInput() == "`") {
                        context.check_layout(os);
                        begin_inset(os, "Quotes ");
                        os << "eld";
                        end_inset(os);
                        p.get_token();
                        skip_braces(p);
+                       continue;
                }
-               else if (t.asInput() == "'" && p.next_token().asInput() == "'") {
+               if (t.asInput() == "'" && p.next_token().asInput() == "'") {
                        context.check_layout(os);
                        begin_inset(os, "Quotes ");
                        os << "erd";
                        end_inset(os);
                        p.get_token();
                        skip_braces(p);
+                       continue;
                }
 
-               else if (t.asInput() == ">" && p.next_token().asInput() == ">") {
+               if (t.asInput() == ">" && p.next_token().asInput() == ">") {
                        context.check_layout(os);
                        begin_inset(os, "Quotes ");
                        os << "ald";
                        end_inset(os);
                        p.get_token();
                        skip_braces(p);
+                       continue;
                }
 
-               else if (t.asInput() == "<"
+               if (t.asInput() == "<"
                         && p.next_token().asInput() == "<") {
                        bool has_chunk = false;
                        if (noweb_mode) {
@@ -2645,16 +2659,19 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                p.get_token();
                                skip_braces(p);
                        }
+                       continue;
                }
 
-               else if (t.cat() == catSpace || (t.cat() == catNewline && ! p.isParagraph()))
+               if (t.cat() == catSpace || (t.cat() == catNewline && ! p.isParagraph())) {
                        check_space(p, os, context);
+                       continue;
+               }
 
                // babel shorthands (also used by polyglossia)
                // Since these can have different meanings for different languages
                // we import them as ERT (but they must be put in ERT to get output
                // verbatim).
-               else if (t.asInput() == "\"") {
+               if (t.asInput() == "\"") {
                        string s = "\"";
                        // These are known pairs. We put them together in
                        // one ERT inset. In other cases (such as "a), only
@@ -2674,9 +2691,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                p.get_token();
                        }
                        output_ert_inset(os, s, context);
+                       continue;
                }
 
-               else if (t.character() == '[' && noweb_mode &&
+               if (t.character() == '[' && noweb_mode &&
                         p.next_token().character() == '[') {
                        // These can contain underscores
                        p.putback();
@@ -2687,14 +2705,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                cerr << "Warning: Inserting missing ']' in '"
                                     << s << "'." << endl;
                        output_ert_inset(os, s, context);
+                       continue;
                }
 
-               else if (t.cat() == catLetter) {
+               if (t.cat() == catLetter) {
                        context.check_layout(os);
                        os << t.cs();
+                       continue;
                }
 
-               else if (t.cat() == catOther ||
+               if (t.cat() == catOther ||
                               t.cat() == catAlign ||
                               t.cat() == catParameter) {
                        context.check_layout(os);
@@ -2713,9 +2733,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else
                                // This translates "&" to "\\&" which may be wrong...
                                os << t.cs();
+                       continue;
                }
 
-               else if (p.isParagraph()) {
+               if (p.isParagraph()) {
                        // In minted floating listings we will collect
                        // everything into the caption, where multiple
                        // paragraphs are forbidden.
@@ -2727,9 +2748,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else
                                os << ' ';
                        eat_whitespace(p, os, context, true);
+                       continue;
                }
 
-               else if (t.cat() == catActive) {
+               if (t.cat() == catActive) {
                        context.check_layout(os);
                        if (t.character() == '~') {
                                if (context.layout->free_spacing)
@@ -2740,9 +2762,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                }
                        } else
                                os << t.cs();
+                       continue;
                }
 
-               else if (t.cat() == catBegin) {
+               if (t.cat() == catBegin) {
                        Token const next = p.next_token();
                        Token const end = p.next_next_token();
                        if (next.cat() == catEnd) {
@@ -2866,24 +2889,28 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                output_ert_inset(os, "}", context);
                                }
                        }
+                       continue;
                }
 
-               else if (t.cat() == catEnd) {
+               if (t.cat() == catEnd) {
                        if (flags & FLAG_BRACE_LAST) {
                                return;
                        }
                        cerr << "stray '}' in text\n";
                        output_ert_inset(os, "}", context);
+                       continue;
                }
 
-               else if (t.cat() == catComment)
+               if (t.cat() == catComment) {
                        parse_comment(p, os, t, context);
+                       continue;
+               }
 
                //
                // control sequences
                //
 
-               else if (t.cs() == "(" || t.cs() == "[") {
+               if (t.cs() == "(" || t.cs() == "[") {
                        bool const simple = t.cs() == "(";
                        context.check_layout(os);
                        begin_inset(os, "Formula");
@@ -2897,13 +2924,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                // output, but looks ugly in LyX.
                                eat_whitespace(p, os, context, false);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "begin")
+               if (t.cs() == "begin") {
                        parse_environment(p, os, outer, last_env,
                                          context);
+                       continue;
+               }
 
-               else if (t.cs() == "end") {
+               if (t.cs() == "end") {
                        if (flags & FLAG_END) {
                                // eat environment name
                                string const name = p.getArg('{', '}');
@@ -2913,9 +2943,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                return;
                        }
                        p.error("found 'end' unexpectedly");
+                       continue;
                }
 
-               else if (t.cs() == "item") {
+               if (t.cs() == "item") {
                        string s;
                        bool const optarg = p.hasOpt();
                        if (optarg) {
@@ -2974,9 +3005,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                        eat_whitespace(p, os, context, false);
                                }
                        }
+                       continue;
                }
 
-               else if (t.cs() == "bibitem") {
+               if (t.cs() == "bibitem") {
                        context.set_item();
                        context.check_layout(os);
                        eat_whitespace(p, os, context, false);
@@ -2994,9 +3026,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                   << "literal \"true\"\n";
                                end_inset(os);
                        }
+                       continue;
                }
 
-               else if (is_macro(p)) {
+               if (is_macro(p)) {
                        // catch the case of \def\inputGnumericTable
                        bool macro = true;
                        if (t.cs() == "def") {
@@ -3049,14 +3082,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        }
                        if (macro)
                                parse_macro(p, os, context);
+                       continue;
                }
 
-               else if (t.cs() == "noindent") {
+               if (t.cs() == "noindent") {
                        p.skip_spaces();
                        context.add_par_extra_stuff("\\noindent\n");
+                       continue;
                }
 
-               else if (t.cs() == "appendix") {
+               if (t.cs() == "appendix") {
                        context.add_par_extra_stuff("\\start_of_appendix\n");
                        // We need to start a new paragraph. Otherwise the
                        // appendix in 'bla\appendix\chapter{' would start
@@ -3077,10 +3112,11 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // empty paragraph, but that does not hurt, because
                        // whitespace does not matter here.
                        eat_whitespace(p, os, context, true);
+                       continue;
                }
 
                // Must catch empty dates before findLayout is called below
-               else if (t.cs() == "date") {
+               if (t.cs() == "date") {
                        eat_whitespace(p, os, context, false);
                        p.pushPosition();
                        string const date = p.verbatim_item();
@@ -3109,11 +3145,12 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                                "\\date{" + p.verbatim_item() + '}',
                                                context);
                        }
+                       continue;
                }
 
                // Starred section headings
                // Must attempt to parse "Section*" before "Section".
-               else if ((p.next_token().asInput() == "*") &&
+               if ((p.next_token().asInput() == "*") &&
                         context.new_layout_allowed &&
                         (newlayout = findLayout(context.textclass, t.cs() + '*', true))) {
                        // write the layout
@@ -3125,10 +3162,11 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        set<string> const & req = newlayout->requires();
                        for (set<string>::const_iterator it = req.begin(); it != req.end(); ++it)
                                preamble.registerAutomaticallyLoadedPackage(*it);
+                       continue;
                }
 
                // Section headings and the like
-               else if (context.new_layout_allowed &&
+               if (context.new_layout_allowed &&
                         (newlayout = findLayout(context.textclass, t.cs(), true))) {
                        // write the layout
                        output_command_layout(os, p, outer, context, newlayout);
@@ -3138,9 +3176,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        set<string> const & req = newlayout->requires();
                        for (set<string>::const_iterator it = req.begin(); it != req.end(); ++it)
                                preamble.registerAutomaticallyLoadedPackage(*it);
+                       continue;
                }
 
-               else if (t.cs() == "subfloat") {
+               if (t.cs() == "subfloat") {
                        // the syntax is \subfloat[list entry][sub caption]{content}
                        // if it is a table of figure depends on the surrounding float
                        // FIXME: second optional argument is not parsed
@@ -3201,9 +3240,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                } else
                                        output_ert_inset(os, t.asInput() + "{" + p.verbatim_item() + '}', context);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "includegraphics") {
+               if (t.cs() == "includegraphics") {
                        bool const clip = p.next_token().asInput() == "*";
                        if (clip)
                                p.get_token();
@@ -3363,9 +3403,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // Check whether some option was given twice.
                        end_inset(os);
                        preamble.registerAutomaticallyLoadedPackage("graphicx");
+                       continue;
                }
 
-               else if (t.cs() == "footnote" ||
+               if (t.cs() == "footnote" ||
                         (t.cs() == "thanks" && context.layout->intitle)) {
                        p.skip_spaces();
                        context.check_layout(os);
@@ -3373,24 +3414,27 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        os << "status collapsed\n\n";
                        parse_text_in_inset(p, os, FLAG_ITEM, false, context);
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "marginpar") {
+               if (t.cs() == "marginpar") {
                        p.skip_spaces();
                        context.check_layout(os);
                        begin_inset(os, "Marginal\n");
                        os << "status collapsed\n\n";
                        parse_text_in_inset(p, os, FLAG_ITEM, false, context);
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "lstinline" || t.cs() == "mintinline") {
+               if (t.cs() == "lstinline" || t.cs() == "mintinline") {
                        bool const use_minted = t.cs() == "mintinline";
                        p.skip_spaces();
                        parse_listings(p, os, context, true, use_minted);
+                       continue;
                }
 
-               else if (t.cs() == "ensuremath") {
+               if (t.cs() == "ensuremath") {
                        p.skip_spaces();
                        context.check_layout(os);
                        string const s = p.verbatim_item();
@@ -3400,6 +3444,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        else
                                output_ert_inset(os, "\\ensuremath{" + s + "}",
                                           context);
+                       continue;
                }
 
                else if (t.cs() == "makeindex" || t.cs() == "maketitle") {
@@ -3408,9 +3453,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                skip_spaces_braces(p);
                        } else
                                output_ert_inset(os, t.asInput(), context);
+                       continue;
                }
 
-               else if (t.cs() == "tableofcontents"
+               if (t.cs() == "tableofcontents"
                                || t.cs() == "lstlistoflistings"
                                || t.cs() == "listoflistings") {
                        string name = t.cs();
@@ -3426,9 +3472,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                else
                                        preamble.registerAutomaticallyLoadedPackage("listings");
                        }
+                       continue;
                }
 
-               else if (t.cs() == "listoffigures" || t.cs() == "listoftables") {
+               if (t.cs() == "listoffigures" || t.cs() == "listoftables") {
                        context.check_layout(os);
                        if (t.cs() == "listoffigures")
                                begin_inset(os, "FloatList figure\n");
@@ -3436,9 +3483,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                begin_inset(os, "FloatList table\n");
                        end_inset(os);
                        skip_spaces_braces(p);
+                       continue;
                }
 
-               else if (t.cs() == "listof") {
+               if (t.cs() == "listof") {
                        p.skip_spaces(true);
                        string const name = p.get_token().cs();
                        if (context.textclass.floats().typeExist(name)) {
@@ -3449,24 +3497,31 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                p.get_token(); // swallow second arg
                        } else
                                output_ert_inset(os, "\\listof{" + name + "}", context);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_text_font_families)))
+               if ((where = is_known(t.cs(), known_text_font_families))) {
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
                                context, "\\family", context.font.family,
                                known_coded_font_families[where - known_text_font_families]);
+                       continue;
+               }
 
-               else if ((where = is_known(t.cs(), known_text_font_series)))
+               if ((where = is_known(t.cs(), known_text_font_series))) {
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
                                context, "\\series", context.font.series,
                                known_coded_font_series[where - known_text_font_series]);
+                       continue;
+               }
 
-               else if ((where = is_known(t.cs(), known_text_font_shapes)))
+               if ((where = is_known(t.cs(), known_text_font_shapes))) {
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
                                context, "\\shape", context.font.shape,
                                known_coded_font_shapes[where - known_text_font_shapes]);
+                       continue;
+               }
 
-               else if (t.cs() == "textnormal" || t.cs() == "normalfont") {
+               if (t.cs() == "textnormal" || t.cs() == "normalfont") {
                        context.check_layout(os);
                        TeXFont oldFont = context.font;
                        context.font.init();
@@ -3480,9 +3535,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                context.font = oldFont;
                        } else
                                eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if (t.cs() == "textcolor") {
+               if (t.cs() == "textcolor") {
                        // scheme is \textcolor{color name}{text}
                        string const color = p.verbatim_item();
                        // we support the predefined colors of the color  and the xcolor package
@@ -3508,9 +3564,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else
                                // for custom defined colors
                                output_ert_inset(os, t.asInput() + "{" + color + "}", context);
+                       continue;
                }
 
-               else if (t.cs() == "underbar" || t.cs() == "uline") {
+               if (t.cs() == "underbar" || t.cs() == "uline") {
                        // \underbar is not 100% correct (LyX outputs \uline
                        // of ulem.sty). The difference is that \ulem allows
                        // line breaks, and \underbar does not.
@@ -3523,18 +3580,20 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        context.check_layout(os);
                        os << "\n\\bar default\n";
                        preamble.registerAutomaticallyLoadedPackage("ulem");
+                       continue;
                }
 
-               else if (t.cs() == "sout") {
+               if (t.cs() == "sout") {
                        context.check_layout(os);
                        os << "\n\\strikeout on\n";
                        parse_text_snippet(p, os, FLAG_ITEM, outer, context);
                        context.check_layout(os);
                        os << "\n\\strikeout default\n";
                        preamble.registerAutomaticallyLoadedPackage("ulem");
+                       continue;
                }
 
-               else if (t.cs() == "uuline" || t.cs() == "uwave"
+               if (t.cs() == "uuline" || t.cs() == "uwave"
                        || t.cs() == "emph" || t.cs() == "noun"
                        || t.cs() == "xout") {
                        context.check_layout(os);
@@ -3544,9 +3603,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        os << "\n\\" << t.cs() << " default\n";
                        if (t.cs() == "uuline" || t.cs() == "uwave" || t.cs() == "xout")
                                preamble.registerAutomaticallyLoadedPackage("ulem");
+                       continue;
                }
 
-               else if (t.cs() == "lyxadded" || t.cs() == "lyxdeleted") {
+               if (t.cs() == "lyxadded" || t.cs() == "lyxdeleted") {
                        context.check_layout(os);
                        string name = p.getArg('{', '}');
                        string localtime = p.getArg('{', '}');
@@ -3586,9 +3646,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                        preamble.registerAutomaticallyLoadedPackage("xcolor");
                                }
                        }
+                       continue;
                }
 
-               else if (t.cs() == "textipa") {
+               if (t.cs() == "textipa") {
                        context.check_layout(os);
                        begin_inset(os, "IPA\n");
                        bool merging_hyphens_allowed = context.merging_hyphens_allowed;
@@ -3598,25 +3659,27 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        end_inset(os);
                        preamble.registerAutomaticallyLoadedPackage("tipa");
                        preamble.registerAutomaticallyLoadedPackage("tipx");
+                       continue;
                }
 
-               else if (t.cs() == "texttoptiebar" || t.cs() == "textbottomtiebar") {
+               if (t.cs() == "texttoptiebar" || t.cs() == "textbottomtiebar") {
                        context.check_layout(os);
                        begin_inset(os, "IPADeco " + t.cs().substr(4) + "\n");
                        os << "status open\n";
                        parse_text_in_inset(p, os, FLAG_ITEM, outer, context);
                        end_inset(os);
                        p.skip_spaces();
+                       continue;
                }
 
-               else if (t.cs() == "textvertline") {
+               if (t.cs() == "textvertline") {
                        // FIXME: This is not correct, \textvertline is higher than |
                        os << "|";
                        skip_braces(p);
                        continue;
                }
 
-               else if (t.cs() == "tone" ) {
+               if (t.cs() == "tone" ) {
                        context.check_layout(os);
                        // register the tone package
                        preamble.registerAutomaticallyLoadedPackage("tone");
@@ -3644,9 +3707,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else
                                // we did not find a non-ert version
                                output_ert_inset(os, command, context);
+                       continue;
                }
 
-               else if (t.cs() == "phantom" || t.cs() == "hphantom" ||
+               if (t.cs() == "phantom" || t.cs() == "hphantom" ||
                             t.cs() == "vphantom") {
                        context.check_layout(os);
                        if (t.cs() == "phantom")
@@ -3659,9 +3723,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        parse_text_in_inset(p, os, FLAG_ITEM, outer, context,
                                            "Phantom");
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "href") {
+               if (t.cs() == "href") {
                        context.check_layout(os);
                        string target = convert_command_inset_arg(p.verbatim_item());
                        string name = convert_command_inset_arg(p.verbatim_item());
@@ -3684,9 +3749,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        os << "literal \"true\"\n";
                        end_inset(os);
                        skip_spaces_braces(p);
+                       continue;
                }
 
-               else if (t.cs() == "lyxline") {
+               if (t.cs() == "lyxline") {
                        // swallow size argument (it is not used anyway)
                        p.getArg('{', '}');
                        if (!context.atParagraphStart()) {
@@ -3711,9 +3777,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                              "width \"100line%\"\n"
                              "height \"1pt\"\n";
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "rule") {
+               if (t.cs() == "rule") {
                        string const offset = (p.hasOpt() ? p.getArg('[', ']') : string());
                        string const width = p.getArg('{', '}');
                        string const thickness = p.getArg('{', '}');
@@ -3724,14 +3791,15 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        os << "width \"" << translate_len(width) << "\"\n"
                                  "height \"" << translate_len(thickness) << "\"\n";
                        end_inset(os);
+                       continue;
                }
 
                // handle refstyle first to catch \eqref which can also occur
                // without refstyle. Only recognize these commands if
                // refstyle.sty was found in the preamble (otherwise \eqref
                // and user defined ref commands could be misdetected).
-               else if ((where = is_known(t.cs(), known_refstyle_commands)) &&
-                        preamble.refstyle()) {
+               if ((where = is_known(t.cs(), known_refstyle_commands))
+                    && preamble.refstyle()) {
                        context.check_layout(os);
                        begin_command_inset(os, "ref", "formatted");
                        os << "reference \"";
@@ -3744,11 +3812,12 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        os << "noprefix \"false\"\n";
                        end_inset(os);
                        preamble.registerAutomaticallyLoadedPackage("refstyle");
+                       continue;
                }
 
                // if refstyle is used, we must not convert \prettyref to a
                // formatted reference, since that would result in a refstyle command.
-               else if ((where = is_known(t.cs(), known_ref_commands)) &&
+               if ((where = is_known(t.cs(), known_ref_commands)) &&
                         (t.cs() != "prettyref" || !preamble.refstyle())) {
                        string const opt = p.getOpt();
                        if (opt.empty()) {
@@ -3771,9 +3840,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                output_ert_inset(os, t.asInput() + '[' + opt + "]{" +
                                       p.verbatim_item() + '}', context);
                        }
+                       continue;
                }
 
-               else if (use_natbib &&
+               if (use_natbib &&
                         is_known(t.cs(), known_natbib_commands) &&
                         ((t.cs() != "citefullauthor" &&
                           t.cs() != "citeyear" &&
@@ -3835,9 +3905,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // the document class directly
                        if (preamble.citeEngine() == "basic")
                                preamble.citeEngine("natbib");
+                       continue;
                }
 
-               else if ((use_biblatex
+               if ((use_biblatex
                         && is_known(t.cs(), known_biblatex_commands)
                         && ((t.cs() == "cite"
                             || t.cs() == "citeauthor"
@@ -3983,9 +4054,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                use_biblatex_natbib ?
                                          preamble.citeEngine("biblatex-natbib")
                                        : preamble.citeEngine("biblatex");
+                       continue;
                }
 
-               else if (use_jurabib &&
+               if (use_jurabib &&
                         is_known(t.cs(), known_jurabib_commands) &&
                         (t.cs() == "cite" || p.next_token().asInput() != "*")) {
                        context.check_layout(os);
@@ -4038,9 +4110,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // the document class directly
                        if (preamble.citeEngine() == "basic")
                                preamble.citeEngine("jurabib");
+                       continue;
                }
 
-               else if (t.cs() == "cite"
+               if (t.cs() == "cite"
                        || t.cs() == "nocite") {
                        context.check_layout(os);
                        string after = convert_command_inset_arg(p.getArg('[', ']'));
@@ -4055,10 +4128,11 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                end_inset(os);
                        } else if (t.cs() == "nocite")
                                btprint = key;
+                       continue;
                }
 
-               else if (t.cs() == "index" ||
-                        (t.cs() == "sindex" && preamble.use_indices() == "true")) {
+               if (t.cs() == "index" ||
+                   (t.cs() == "sindex" && preamble.use_indices() == "true")) {
                        context.check_layout(os);
                        string const arg = (t.cs() == "sindex" && p.hasOpt()) ?
                                p.getArg('[', ']') : "";
@@ -4069,9 +4143,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        end_inset(os);
                        if (kind != "idx")
                                preamble.registerAutomaticallyLoadedPackage("splitidx");
+                       continue;
                }
 
-               else if (t.cs() == "nomenclature") {
+               if (t.cs() == "nomenclature") {
                        context.check_layout(os);
                        begin_command_inset(os, "nomenclature", "nomenclature");
                        string prefix = convert_command_inset_arg(p.getArg('[', ']'));
@@ -4085,18 +4160,20 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                           << "literal \"true\"\n";
                        end_inset(os);
                        preamble.registerAutomaticallyLoadedPackage("nomencl");
+                       continue;
                }
 
-               else if (t.cs() == "label") {
+               if (t.cs() == "label") {
                        context.check_layout(os);
                        begin_command_inset(os, "label", "label");
                        os << "name \""
                           << convert_command_inset_arg(p.verbatim_item())
                           << "\"\n";
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "lyxmintcaption") {
+               if (t.cs() == "lyxmintcaption") {
                        string const pos = p.getArg('[', ']');
                        if (pos == "t") {
                                string const caption =
@@ -4108,9 +4185,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                // so simply skip it.
                                parse_text_snippet(p, FLAG_ITEM, false, context);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "printindex" || t.cs() == "printsubindex") {
+               if (t.cs() == "printindex" || t.cs() == "printsubindex") {
                        context.check_layout(os);
                        string commandname = t.cs();
                        bool star = false;
@@ -4133,9 +4211,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        preamble.registerAutomaticallyLoadedPackage("makeidx");
                        if (preamble.use_indices() == "true")
                                preamble.registerAutomaticallyLoadedPackage("splitidx");
+                       continue;
                }
 
-               else if (t.cs() == "printnomenclature") {
+               if (t.cs() == "printnomenclature") {
                        string width = "";
                        string width_type = "";
                        context.check_layout(os);
@@ -4160,9 +4239,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        end_inset(os);
                        skip_spaces_braces(p);
                        preamble.registerAutomaticallyLoadedPackage("nomencl");
+                       continue;
                }
 
-               else if ((t.cs() == "textsuperscript" || t.cs() == "textsubscript")) {
+               if ((t.cs() == "textsuperscript" || t.cs() == "textsubscript")) {
                        context.check_layout(os);
                        begin_inset(os, "script ");
                        os << t.cs().substr(4) << '\n';
@@ -4171,9 +4251,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        end_inset(os);
                        if (t.cs() == "textsubscript")
                                preamble.registerAutomaticallyLoadedPackage("subscript");
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_quotes))) {
+               if ((where = is_known(t.cs(), known_quotes))) {
                        context.check_layout(os);
                        begin_inset(os, "Quotes ");
                        os << known_coded_quotes[where - known_quotes];
@@ -4183,18 +4264,20 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // {} pair.
                        eat_whitespace(p, os, context, false);
                        skip_braces(p);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_sizes)) &&
+               if ((where = is_known(t.cs(), known_sizes)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
                        context.font.size = known_coded_sizes[where - known_sizes];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_font_families)) &&
+               if ((where = is_known(t.cs(), known_font_families)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
@@ -4202,9 +4285,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                known_coded_font_families[where - known_font_families];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_font_series)) &&
+               if ((where = is_known(t.cs(), known_font_series)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
@@ -4212,9 +4296,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                known_coded_font_series[where - known_font_series];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_font_shapes)) &&
+               if ((where = is_known(t.cs(), known_font_shapes)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
@@ -4222,8 +4307,9 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                known_coded_font_shapes[where - known_font_shapes];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
-               else if ((where = is_known(t.cs(), known_old_font_families)) &&
+               if ((where = is_known(t.cs(), known_old_font_families)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
@@ -4233,9 +4319,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                known_coded_font_families[where - known_old_font_families];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_old_font_series)) &&
+               if ((where = is_known(t.cs(), known_old_font_series)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
@@ -4245,9 +4332,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                known_coded_font_series[where - known_old_font_series];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_old_font_shapes)) &&
+               if ((where = is_known(t.cs(), known_old_font_shapes)) &&
                         context.new_layout_allowed) {
                        context.check_layout(os);
                        TeXFont const oldFont = context.font;
@@ -4257,24 +4345,27 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                known_coded_font_shapes[where - known_old_font_shapes];
                        output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
+                       continue;
                }
 
-               else if (t.cs() == "selectlanguage") {
+               if (t.cs() == "selectlanguage") {
                        context.check_layout(os);
                        // save the language for the case that a
                        // \foreignlanguage is used
                        context.font.language = babel2lyx(p.verbatim_item());
                        os << "\n\\lang " << context.font.language << "\n";
+                       continue;
                }
 
-               else if (t.cs() == "foreignlanguage") {
+               if (t.cs() == "foreignlanguage") {
                        string const lang = babel2lyx(p.verbatim_item());
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
                                              context, "\\lang",
                                              context.font.language, lang);
+                       continue;
                }
 
-               else if (prefixIs(t.cs(), "text") && preamble.usePolyglossia()
+               if (prefixIs(t.cs(), "text") && preamble.usePolyglossia()
                         && is_known(t.cs().substr(4), preamble.polyglossia_languages)) {
                        // scheme is \textLANGUAGE{text} where LANGUAGE is in polyglossia_languages[]
                        string lang;
@@ -4302,18 +4393,20 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                                          context, "\\lang",
                                                          context.font.language, lang);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "inputencoding") {
+               if (t.cs() == "inputencoding") {
                        // nothing to write here
                        string const enc = subst(p.verbatim_item(), "\n", " ");
                        p.setEncoding(enc, Encoding::inputenc);
+                       continue;
                }
 
-               else if (is_known(t.cs(), known_special_chars) ||
-                        (t.cs() == "protect" &&
-                         p.next_token().cat() == catEscape &&
-                         is_known(p.next_token().cs(), known_special_protect_chars))) {
+               if (is_known(t.cs(), known_special_chars) ||
+                   (t.cs() == "protect" &&
+                    p.next_token().cat() == catEscape &&
+                    is_known(p.next_token().cs(), known_special_protect_chars))) {
                        // LyX sometimes puts a \protect in front, so we have to ignore it
                        where = is_known(
                                t.cs() == "protect" ? p.get_token().cs() : t.cs(),
@@ -4321,9 +4414,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        context.check_layout(os);
                        os << known_coded_special_chars[where - known_special_chars];
                        skip_spaces_braces(p);
+                       continue;
                }
 
-               else if ((t.cs() == "nobreakdash" && p.next_token().asInput() == "-") ||
+               if ((t.cs() == "nobreakdash" && p.next_token().asInput() == "-") ||
                         (t.cs() == "protect" && p.next_token().asInput() == "\\nobreakdash" &&
                          p.next_next_token().asInput() == "-") ||
                         (t.cs() == "@" && p.next_token().asInput() == ".")) {
@@ -4336,15 +4430,17 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        else
                                os << "\\SpecialChar endofsentence\n";
                        p.get_token();
+                       continue;
                }
 
-               else if (t.cs() == "textquotedbl") {
+               if (t.cs() == "textquotedbl") {
                        context.check_layout(os);
                        os << "\"";
                        skip_braces(p);
+                       continue;
                }
 
-               else if (t.cs() == "_" || t.cs() == "&" || t.cs() == "#"
+               if (t.cs() == "_" || t.cs() == "&" || t.cs() == "#"
                            || t.cs() == "$" || t.cs() == "{" || t.cs() == "}"
                            || t.cs() == "%" || t.cs() == "-") {
                        context.check_layout(os);
@@ -4352,9 +4448,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                os << "\\SpecialChar softhyphen\n";
                        else
                                os << t.cs();
+                       continue;
                }
 
-               else if (t.cs() == "char") {
+               if (t.cs() == "char") {
                        context.check_layout(os);
                        if (p.next_token().character() == '`') {
                                p.get_token();
@@ -4368,9 +4465,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else {
                                output_ert_inset(os, "\\char", context);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "verb") {
+               if (t.cs() == "verb") {
                        context.check_layout(os);
                        // set catcodes to verbatim early, just in case.
                        p.setCatcodes(VERBATIM_CATCODES);
@@ -4381,15 +4479,18 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                                 + arg.second + delim, context);
                        else
                                cerr << "invalid \\verb command. Skipping" << endl;
+                       continue;
                }
 
                // Problem: \= creates a tabstop inside the tabbing environment
                // and else an accent. In the latter case we really would want
                // \={o} instead of \= o.
-               else if (t.cs() == "=" && (flags & FLAG_TABBING))
+               if (t.cs() == "=" && (flags & FLAG_TABBING)) {
                        output_ert_inset(os, t.asInput(), context);
+                       continue;
+               }
 
-               else if (t.cs() == "\\") {
+               if (t.cs() == "\\") {
                        context.check_layout(os);
                        if (p.hasOpt())
                                output_ert_inset(os, "\\\\" + p.getOpt(), context);
@@ -4404,19 +4505,21 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                begin_inset(os, "Newline newline");
                                end_inset(os);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "newline" ||
-                        (t.cs() == "linebreak" && !p.hasOpt())) {
+               if (t.cs() == "newline" ||
+                   (t.cs() == "linebreak" && !p.hasOpt())) {
                        context.check_layout(os);
                        begin_inset(os, "Newline ");
                        os << t.cs();
                        end_inset(os);
                        skip_spaces_braces(p);
+                       continue;
                }
 
-               else if (t.cs() == "input" || t.cs() == "include"
-                        || t.cs() == "verbatiminput") {
+               if (t.cs() == "input" || t.cs() == "include"
+                   || t.cs() == "verbatiminput") {
                        string name = t.cs();
                        if (t.cs() == "verbatiminput"
                            && p.next_token().asInput() == "*")
@@ -4533,9 +4636,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                        preamble.registerAutomaticallyLoadedPackage("verbatim");
                        }
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "bibliographystyle") {
+               if (t.cs() == "bibliographystyle") {
                        // store new bibliographystyle
                        bibliographystyle = p.verbatim_item();
                        // If any other command than \bibliography, \addcontentsline
@@ -4575,16 +4679,18 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                        "\\bibliographystyle{" + bibliographystyle + '}',
                                        context);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "phantomsection") {
+               if (t.cs() == "phantomsection") {
                        // we only support this if it occurs between
                        // \bibliographystyle and \bibliography
                        if (bibliographystyle.empty())
                                output_ert_inset(os, "\\phantomsection", context);
+                       continue;
                }
 
-               else if (t.cs() == "addcontentsline") {
+               if (t.cs() == "addcontentsline") {
                        context.check_layout(os);
                        // get the 3 arguments of \addcontentsline
                        string const one = p.getArg('{', '}');
@@ -4596,6 +4702,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                        "\\addcontentsline{" + one + "}{" + two + "}{"+ three + '}',
                                        context);
                        }
+                       continue;
                }
 
                else if (t.cs() == "bibliography") {
@@ -4627,9 +4734,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        }
                        os << "options " << '"' << BibOpts << '"' << "\n";
                        end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "printbibliography") {
+               if (t.cs() == "printbibliography") {
                        context.check_layout(os);
                        string BibOpts;
                        string bbloptions = p.hasOpt() ? p.getArg('[', ']') : string();
@@ -4667,9 +4775,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                os << "biblatexopts " << '"' << bbloptions << '"' << "\n";
                        end_inset(os);
                        need_commentbib = false;
+                       continue;
                }
 
-               else if (t.cs() == "bibbysection") {
+               if (t.cs() == "bibbysection") {
                        context.check_layout(os);
                        string BibOpts;
                        string bbloptions = p.hasOpt() ? p.getArg('[', ']') : string();
@@ -4696,9 +4805,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                os << "biblatexopts " << '"' << bbloptions << '"' << "\n";
                        end_inset(os);
                        need_commentbib = false;
+                       continue;
                }
 
-               else if (t.cs() == "parbox") {
+               if (t.cs() == "parbox") {
                        // Test whether this is an outer box of a shaded box
                        p.pushPosition();
                        // swallow arguments
@@ -4726,14 +4836,17 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else
                                parse_box(p, os, 0, FLAG_ITEM, outer, context,
                                          "", "", t.cs(), "", "");
+                       continue;
                }
 
-               else if (t.cs() == "fbox" || t.cs() == "mbox" ||
-                            t.cs() == "ovalbox" || t.cs() == "Ovalbox" ||
-                        t.cs() == "shadowbox" || t.cs() == "doublebox")
+               if (t.cs() == "fbox" || t.cs() == "mbox" ||
+                   t.cs() == "ovalbox" || t.cs() == "Ovalbox" ||
+                   t.cs() == "shadowbox" || t.cs() == "doublebox") {
                        parse_outer_box(p, os, FLAG_ITEM, outer, context, t.cs(), "");
+                       continue;
+               }
 
-               else if (t.cs() == "fcolorbox" || t.cs() == "colorbox") {
+               if (t.cs() == "fcolorbox" || t.cs() == "colorbox") {
                        string backgroundcolor;
                        preamble.registerAutomaticallyLoadedPackage("xcolor");
                        if (t.cs() == "fcolorbox") {
@@ -4744,15 +4857,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                backgroundcolor = p.getArg('{', '}');
                                parse_box(p, os, 0, 0, outer, context, "", "", "", "", backgroundcolor);
                        }
+                       continue;
                }
 
                // FIXME: due to the compiler limit of "if" nestings
                // the code for the alignment was put here
                // put them in their own if if this is fixed
-               else if (t.cs() == "fboxrule" || t.cs() == "fboxsep"
-                            || t.cs() == "shadowsize"
-                                || t.cs() == "raggedleft" || t.cs() == "centering"
-                        || t.cs() == "raggedright") {
+               if (t.cs() == "fboxrule" || t.cs() == "fboxsep"
+                   || t.cs() == "shadowsize"
+                   || t.cs() == "raggedleft" || t.cs() == "centering"
+                   || t.cs() == "raggedright") {
                        if (t.cs() == "fboxrule")
                                fboxrule = "";
                        if (t.cs() == "fboxsep")
@@ -4775,11 +4889,12 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        } else {
                                output_ert_inset(os, t.asInput(), context);
                        }
+                       continue;
                }
 
                //\framebox() is part of the picture environment and different from \framebox{}
                //\framebox{} will be parsed by parse_outer_box
-               else if (t.cs() == "framebox") {
+               if (t.cs() == "framebox") {
                        if (p.next_token().character() == '(') {
                                //the syntax is: \framebox(x,y)[position]{content}
                                string arg = t.asInput();
@@ -4796,11 +4911,12 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                parse_outer_box(p, os, FLAG_ITEM, outer,
                                                    context, t.cs(), special);
                        }
+                       continue;
                }
 
                //\makebox() is part of the picture environment and different from \makebox{}
                //\makebox{} will be parsed by parse_box
-               else if (t.cs() == "makebox") {
+               if (t.cs() == "makebox") {
                        if (p.next_token().character() == '(') {
                                //the syntax is: \makebox(x,y)[position]{content}
                                string arg = t.asInput();
@@ -4814,20 +4930,22 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                //the syntax is: \makebox[width][position]{content}
                                parse_box(p, os, 0, FLAG_ITEM, outer, context,
                                          "", "", t.cs(), "", "");
+                       continue;
                }
 
-               else if (t.cs() == "smallskip" ||
-                        t.cs() == "medskip" ||
-                        t.cs() == "bigskip" ||
-                        t.cs() == "vfill") {
+               if (t.cs() == "smallskip" ||
+                   t.cs() == "medskip" ||
+                   t.cs() == "bigskip" ||
+                   t.cs() == "vfill") {
                        context.check_layout(os);
                        begin_inset(os, "VSpace ");
                        os << t.cs();
                        end_inset(os);
                        skip_spaces_braces(p);
+                       continue;
                }
 
-               else if ((where = is_known(t.cs(), known_spaces))) {
+               if ((where = is_known(t.cs(), known_spaces))) {
                        context.check_layout(os);
                        begin_inset(os, "space ");
                        os << '\\' << known_coded_spaces[where - known_spaces]
@@ -4844,20 +4962,22 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // remove the braces after "\\,", too.
                        if (t.cs() != " ")
                                skip_braces(p);
+                       continue;
                }
 
-               else if (t.cs() == "newpage" ||
-                        (t.cs() == "pagebreak" && !p.hasOpt()) ||
-                        t.cs() == "clearpage" ||
-                        t.cs() == "cleardoublepage") {
+               if (t.cs() == "newpage" ||
+                   (t.cs() == "pagebreak" && !p.hasOpt()) ||
+                   t.cs() == "clearpage" ||
+                   t.cs() == "cleardoublepage") {
                        context.check_layout(os);
                        begin_inset(os, "Newpage ");
                        os << t.cs();
                        end_inset(os);
                        skip_spaces_braces(p);
+                       continue;
                }
 
-               else if (t.cs() == "DeclareRobustCommand" ||
+               if (t.cs() == "DeclareRobustCommand" ||
                         t.cs() == "DeclareRobustCommandx" ||
                         t.cs() == "newcommand" ||
                         t.cs() == "newcommandx" ||
@@ -4895,9 +5015,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                os << "\n" << ert;
                                end_inset(os);
                        }
+                       continue;
                }
 
-               else if (t.cs() == "let" && p.next_token().asInput() != "*") {
+               if (t.cs() == "let" && p.next_token().asInput() != "*") {
                        // let could be handled by parse_command(),
                        // but we need to call add_known_command() here.
                        string ert = t.asInput();
@@ -4926,9 +5047,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        if (it != known_commands.end())
                                known_commands[t.asInput()] = it->second;
                        output_ert_inset(os, ert, context);
+                       continue;
                }
 
-               else if (t.cs() == "hspace" || t.cs() == "vspace") {
+               if (t.cs() == "hspace" || t.cs() == "vspace") {
                        if (starred)
                                p.get_token();
                        string name = t.asInput();
@@ -5062,10 +5184,11 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                } else
                                        output_ert_inset(os, name + '{' + length + '}', context);
                        }
+                       continue;
                }
 
                // The single '=' is meant here.
-               else if ((newinsetlayout = findInsetLayout(context.textclass, starredname, true))) {
+               if ((newinsetlayout = findInsetLayout(context.textclass, starredname, true))) {
                        if (starred)
                                p.get_token();
                        p.skip_spaces();
@@ -5108,9 +5231,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // we collect everything into the caption.
                        if (minted_float.empty())
                                end_inset(os);
+                       continue;
                }
 
-               else if (t.cs() == "includepdf") {
+               if (t.cs() == "includepdf") {
                        p.skip_spaces();
                        string const arg = p.getArg('[', ']');
                        map<string, string> opts;
@@ -5180,9 +5304,10 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        end_inset(os);
                        context.check_layout(os);
                        registerExternalTemplatePackages("PDFPages");
+                       continue;
                }
 
-               else if (t.cs() == "loadgame") {
+               if (t.cs() == "loadgame") {
                        p.skip_spaces();
                        string name = normalize_filename(p.verbatim_item());
                        string const path = getMasterFilePath(true);
@@ -5215,149 +5340,143 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        if (p.get_token().asInput() == "showboard")
                                p.get_token();
                        registerExternalTemplatePackages("ChessDiagram");
+                       continue;
                }
 
-               else {
-                       // try to see whether the string is in unicodesymbols
-                       // Only use text mode commands, since we are in text mode here,
-                       // and math commands may be invalid (bug 6797)
-                       string name = t.asInput();
-                       // handle the dingbats, cyrillic and greek
-                       if (name == "\\ding" || name == "\\textcyr" ||
-                           (name == "\\textgreek" && !preamble.usePolyglossia()))
-                               name = name + '{' + p.getArg('{', '}') + '}';
-                       // handle the ifsym characters
-                       else if (name == "\\textifsymbol") {
-                               string const optif = p.getFullOpt();
-                               string const argif = p.getArg('{', '}');
-                               name = name + optif + '{' + argif + '}';
-                       }
-                       // handle the \ascii characters
-                       // the case of \ascii within braces, as LyX outputs it, is already
-                       // handled for t.cat() == catBegin
-                       else if (name == "\\ascii") {
-                               // the code is "\asci\xxx"
-                               name = "{" + name + p.get_token().asInput() + "}";
+               // try to see whether the string is in unicodesymbols
+               // Only use text mode commands, since we are in text mode here,
+               // and math commands may be invalid (bug 6797)
+               string name = t.asInput();
+               // handle the dingbats, cyrillic and greek
+               if (name == "\\ding" || name == "\\textcyr" ||
+                   (name == "\\textgreek" && !preamble.usePolyglossia()))
+                       name = name + '{' + p.getArg('{', '}') + '}';
+               // handle the ifsym characters
+               else if (name == "\\textifsymbol") {
+                       string const optif = p.getFullOpt();
+                       string const argif = p.getArg('{', '}');
+                       name = name + optif + '{' + argif + '}';
+               }
+               // handle the \ascii characters
+               // the case of \ascii within braces, as LyX outputs it, is already
+               // handled for t.cat() == catBegin
+               else if (name == "\\ascii") {
+                       // the code is "\asci\xxx"
+                       name = "{" + name + p.get_token().asInput() + "}";
+                       skip_braces(p);
+               }
+               // handle some TIPA special characters
+               else if (preamble.isPackageUsed("tipa")) {
+                       if (name == "\\s") {
+                               // fromLaTeXCommand() does not yet
+                               // recognize tipa short cuts
+                               name = "\\textsyllabic";
+                       } else if (name == "\\=" &&
+                                  p.next_token().asInput() == "*") {
+                               // fromLaTeXCommand() does not yet
+                               // recognize tipa short cuts
+                               p.get_token();
+                               name = "\\textsubbar";
+                       } else if (name == "\\textdoublevertline") {
+                               // FIXME: This is not correct,
+                               // \textvertline is higher than \textbardbl
+                               name = "\\textbardbl";
                                skip_braces(p);
-                       }
-                       // handle some TIPA special characters
-                       else if (preamble.isPackageUsed("tipa")) {
-                               if (name == "\\s") {
-                                       // fromLaTeXCommand() does not yet
-                                       // recognize tipa short cuts
-                                       name = "\\textsyllabic";
-                               } else if (name == "\\=" &&
-                                          p.next_token().asInput() == "*") {
-                                       // fromLaTeXCommand() does not yet
-                                       // recognize tipa short cuts
+                       } else if (name == "\\!" ) {
+                               if (p.next_token().asInput() == "b") {
+                                       p.get_token();  // eat 'b'
+                                       name = "\\texthtb";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "d") {
                                        p.get_token();
-                                       name = "\\textsubbar";
-                               } else if (name == "\\textdoublevertline") {
-                                       // FIXME: This is not correct,
-                                       // \textvertline is higher than \textbardbl
-                                       name = "\\textbardbl";
+                                       name = "\\texthtd";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "g") {
+                                       p.get_token();
+                                       name = "\\texthtg";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "G") {
+                                       p.get_token();
+                                       name = "\\texthtscg";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "j") {
+                                       p.get_token();
+                                       name = "\\texthtbardotlessj";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "o") {
+                                       p.get_token();
+                                       name = "\\textbullseye";
                                        skip_braces(p);
-                               } else if (name == "\\!" ) {
-                                       if (p.next_token().asInput() == "b") {
-                                               p.get_token();  // eat 'b'
-                                               name = "\\texthtb";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "d") {
-                                               p.get_token();
-                                               name = "\\texthtd";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "g") {
-                                               p.get_token();
-                                               name = "\\texthtg";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "G") {
-                                               p.get_token();
-                                               name = "\\texthtscg";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "j") {
-                                               p.get_token();
-                                               name = "\\texthtbardotlessj";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "o") {
-                                               p.get_token();
-                                               name = "\\textbullseye";
-                                               skip_braces(p);
-                                       }
-                               } else if (name == "\\*" ) {
-                                       if (p.next_token().asInput() == "k") {
-                                               p.get_token();
-                                               name = "\\textturnk";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "r") {
-                                               p.get_token();  // eat 'b'
-                                               name = "\\textturnr";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "t") {
-                                               p.get_token();
-                                               name = "\\textturnt";
-                                               skip_braces(p);
-                                       } else if (p.next_token().asInput() == "w") {
-                                               p.get_token();
-                                               name = "\\textturnw";
-                                               skip_braces(p);
-                                       }
                                }
-                       }
-                       if ((name.size() == 2 &&
-                            contains("\"'.=^`bcdHkrtuv~", name[1]) &&
-                            p.next_token().asInput() != "*") ||
-                           is_known(name.substr(1), known_tipa_marks)) {
-                               // name is a command that corresponds to a
-                               // combining character in unicodesymbols.
-                               // Append the argument, fromLaTeXCommand()
-                               // will either convert it to a single
-                               // character or a combining sequence.
-                               name += '{' + p.verbatim_item() + '}';
-                       }
-                       // now get the character from unicodesymbols
-                       bool termination;
-                       docstring rem;
-                       set<string> req;
-                       docstring s = normalize_c(encodings.fromLaTeXCommand(from_utf8(name),
-                                       Encodings::TEXT_CMD, termination, rem, &req));
-                       if (!s.empty()) {
-                               context.check_layout(os);
-                               os << to_utf8(s);
-                               if (!rem.empty())
-                                       output_ert_inset(os, to_utf8(rem), context);
-                               if (termination)
-                                       skip_spaces_braces(p);
-                               for (set<string>::const_iterator it = req.begin(); it != req.end(); ++it)
-                                       preamble.registerAutomaticallyLoadedPackage(*it);
-                       }
-                       //cerr << "#: " << t << " mode: " << mode << endl;
-                       // heuristic: read up to next non-nested space
-                       /*
-                       string s = t.asInput();
-                       string z = p.verbatim_item();
-                       while (p.good() && z != " " && !z.empty()) {
-                               //cerr << "read: " << z << endl;
-                               s += z;
-                               z = p.verbatim_item();
-                       }
-                       cerr << "found ERT: " << s << endl;
-                       output_ert_inset(os, s + ' ', context);
-                       */
-                       else {
-                               if (t.asInput() == name &&
-                                   p.next_token().asInput() == "*") {
-                                       // Starred commands like \vspace*{}
-                                       p.get_token();  // Eat '*'
-                                       name += '*';
+                       } else if (name == "\\*" ) {
+                               if (p.next_token().asInput() == "k") {
+                                       p.get_token();
+                                       name = "\\textturnk";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "r") {
+                                       p.get_token();  // eat 'b'
+                                       name = "\\textturnr";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "t") {
+                                       p.get_token();
+                                       name = "\\textturnt";
+                                       skip_braces(p);
+                               } else if (p.next_token().asInput() == "w") {
+                                       p.get_token();
+                                       name = "\\textturnw";
+                                       skip_braces(p);
                                }
-                               if (!parse_command(name, p, os, outer, context))
-                                       output_ert_inset(os, name, context);
                        }
                }
-
-               if (flags & FLAG_LEAVE) {
-                       flags &= ~FLAG_LEAVE;
-                       break;
+               if ((name.size() == 2 &&
+                    contains("\"'.=^`bcdHkrtuv~", name[1]) &&
+                    p.next_token().asInput() != "*") ||
+                   is_known(name.substr(1), known_tipa_marks)) {
+                       // name is a command that corresponds to a
+                       // combining character in unicodesymbols.
+                       // Append the argument, fromLaTeXCommand()
+                       // will either convert it to a single
+                       // character or a combining sequence.
+                       name += '{' + p.verbatim_item() + '}';
+               }
+               // now get the character from unicodesymbols
+               bool termination;
+               docstring rem;
+               set<string> req;
+               docstring s = normalize_c(encodings.fromLaTeXCommand(from_utf8(name),
+                               Encodings::TEXT_CMD, termination, rem, &req));
+               if (!s.empty()) {
+                       context.check_layout(os);
+                       os << to_utf8(s);
+                       if (!rem.empty())
+                               output_ert_inset(os, to_utf8(rem), context);
+                       if (termination)
+                               skip_spaces_braces(p);
+                       for (set<string>::const_iterator it = req.begin(); it != req.end(); ++it)
+                               preamble.registerAutomaticallyLoadedPackage(*it);
+               }
+               //cerr << "#: " << t << " mode: " << mode << endl;
+               // heuristic: read up to next non-nested space
+               /*
+               string s = t.asInput();
+               string z = p.verbatim_item();
+               while (p.good() && z != " " && !z.empty()) {
+                       //cerr << "read: " << z << endl;
+                       s += z;
+                       z = p.verbatim_item();
+               }
+               cerr << "found ERT: " << s << endl;
+               output_ert_inset(os, s + ' ', context);
+               */
+               else {
+                       if (t.asInput() == name &&
+                           p.next_token().asInput() == "*") {
+                               // Starred commands like \vspace*{}
+                               p.get_token();  // Eat '*'
+                               name += '*';
+                       }
+                       if (!parse_command(name, p, os, outer, context))
+                               output_ert_inset(os, name, context);
                }
        }
 }