]> git.lyx.org Git - lyx.git/blobdiff - src/tex2lyx/text.C
* support/qstring_helpers.h: erase ucs4_to_qstring() method.
[lyx.git] / src / tex2lyx / text.C
index 6a6bf7d7f79aa3f8196d098db3f9bb2117babc9e..d28fdea83bc4c58cec6b3edf4510df10cff40df2 100644 (file)
 #include "context.h"
 #include "FloatList.h"
 #include "lengthcommon.h"
-#include "support/FileInfo.h"
 #include "support/lstrings.h"
-#include "support/tostr.h"
+#include "support/convert.h"
 #include "support/filetools.h"
 
+#include <boost/filesystem/operations.hpp>
 #include <boost/tuple/tuple.hpp>
 
 #include <iostream>
 #include <sstream>
 #include <vector>
 
-using lyx::support::FileInfo;
-using lyx::support::MakeAbsPath;
-using lyx::support::rtrim;
-using lyx::support::suffixIs;
-using lyx::support::contains;
-using lyx::support::subst;
+
+namespace lyx {
+
+using support::addExtension;
+using support::changeExtension;
+using support::FileName;
+using support::makeAbsPath;
+using support::makeRelPath;
+using support::rtrim;
+using support::suffixIs;
+using support::contains;
+using support::subst;
 
 using std::cerr;
 using std::endl;
@@ -46,19 +52,11 @@ using std::istringstream;
 using std::string;
 using std::vector;
 
-
-/// thin wrapper around parse_text using a string
-string parse_text(Parser & p, unsigned flags, const bool outer,
-                 Context & context)
-{
-       ostringstream os;
-       parse_text(p, os, flags, outer, context);
-       return os.str();
-}
+namespace fs = boost::filesystem;
 
 
 void parse_text_in_inset(Parser & p, ostream & os, unsigned flags, bool outer,
-               Context & context)
+               Context const & context)
 {
        Context newcontext(true, context.textclass);
        newcontext.font = context.font;
@@ -67,19 +65,45 @@ void parse_text_in_inset(Parser & p, ostream & os, unsigned flags, bool outer,
 }
 
 
-/// parses a paragraph snippet, useful for example for \emph{...}
+namespace {
+
+/// parses a paragraph snippet, useful for example for \\emph{...}
 void parse_text_snippet(Parser & p, ostream & os, unsigned flags, bool outer,
                Context & context)
 {
-       Context newcontext(false, context.textclass);
-       newcontext.font = context.font;
+       Context newcontext(context);
+       // Don't inherit the extra stuff
+       newcontext.extra_stuff.clear();
        parse_text(p, os, flags, outer, newcontext);
-       // should not be needed
-       newcontext.check_end_layout(os);
+       // Make sure that we don't create invalid .lyx files
+       context.need_layout = newcontext.need_layout;
+       context.need_end_layout = newcontext.need_end_layout;
 }
 
 
-namespace {
+/*!
+ * Thin wrapper around parse_text_snippet() using a string.
+ *
+ * We completely ignore \c context.need_layout and \c context.need_end_layout,
+ * because our return value is not used directly (otherwise the stream version
+ * of parse_text_snippet() could be used). That means that the caller needs
+ * to do layout management manually.
+ * This is intended to parse text that does not create any layout changes.
+ */
+string parse_text_snippet(Parser & p, unsigned flags, const bool outer,
+                 Context & context)
+{
+       Context newcontext(context);
+       newcontext.need_layout = false;
+       newcontext.need_end_layout = false;
+       newcontext.new_layout_allowed = false;
+       // Avoid warning by Context::~Context()
+       newcontext.extra_stuff.clear();
+       ostringstream os;
+       parse_text_snippet(p, os, flags, outer, newcontext);
+       return os.str();
+}
+
 
 char const * const known_latex_commands[] = { "ref", "cite", "label", "index",
 "printindex", "pageref", "url", "vref", "vpageref", "prettyref", "eqref", 0 };
@@ -101,8 +125,9 @@ char const * const known_natbib_commands[] = { "cite", "citet", "citep",
  * No starred form other than "cite*" known.
  */
 char const * const known_jurabib_commands[] = { "cite", "citet", "citep",
-"citealt", "citealp", "citeauthor", "citeyear", "citeyearpar", "fullcite",
+"citealt", "citealp", "citeauthor", "citeyear", "citeyearpar",
 // jurabib commands not (yet) supported by LyX:
+// "fullcite",
 // "footcite", "footcitet", "footcitep", "footcitealt", "footcitealp",
 // "footciteauthor", "footciteyear", "footciteyearpar",
 "citefield", "citetitle", "cite*", 0 };
@@ -113,7 +138,7 @@ char const * const known_quotes[] = { "glqq", "grqq", "quotedblbase",
 
 /// the same as known_quotes with .lyx names
 char const * const known_coded_quotes[] = { "gld", "grd", "gld",
-"grd", "gls", "fls", "frd", 0};
+"grd", "gls", "fls", "frs", 0};
 
 /// LaTeX names for font sizes
 char const * const known_sizes[] = { "tiny", "scriptsize", "footnotesize",
@@ -169,11 +194,25 @@ char const * const known_dvips_graphics_formats[] = {"eps", "ps", "eps.gz",
 
 /*!
  * Graphics file extensions known by the pdftex driver of the graphics package.
- * \see known_dvips_graphics_formats
+ * \sa known_dvips_graphics_formats
  */
 char const * const known_pdftex_graphics_formats[] = {"png", "pdf", "jpg",
 "mps", "tif", 0};
 
+/*!
+ * Known file extensions for TeX files as used by \\include.
+ */
+char const * const known_tex_extensions[] = {"tex", 0};
+
+/// spaces known by InsetSpace
+char const * const known_spaces[] = { " ", "space", ",", "thinspace", "quad",
+"qquad", "enspace", "enskip", "negthinspace", 0};
+
+/// the same as known_spaces with .lyx names
+char const * const known_coded_spaces[] = { "space{}", "space{}",
+"thinspace{}", "thinspace{}", "quad{}", "qquad{}", "enspace{}", "enskip{}",
+"negthinspace{}", 0};
+
 
 /// splits "x=z, y=b" into a map
 map<string, string> split_map(string const & s)
@@ -196,7 +235,7 @@ map<string, string> split_map(string const & s)
  * The latter can be a real unit like "pt", or a latex length variable
  * like "\textwidth". The unit may contain additional stuff like glue
  * lengths, but we don't care, because such lengths are ERT anyway.
- * \return true if \param value and \param unit are valid.
+ * \returns true if \p value and \p unit are valid.
  */
 bool splitLatexLength(string const & len, string & value, string & unit)
 {
@@ -269,6 +308,8 @@ bool translate_len(string const & length, string & valstring, string & unit)
        return true;
 }
 
+}
+
 
 string translate_len(string const & length)
 {
@@ -281,11 +322,13 @@ string translate_len(string const & length)
 }
 
 
+namespace {
+
 /*!
- * Translates a LaTeX length into \param value, \param unit and
- * \param special parts suitable for a box inset.
+ * Translates a LaTeX length into \p value, \p unit and
+ * \p special parts suitable for a box inset.
  * The difference from translate_len() is that a box inset knows about
- * some special "units" that are stored in \param special.
+ * some special "units" that are stored in \p special.
  */
 void translate_box_len(string const & length, string & value, string & unit, string & special)
 {
@@ -312,11 +355,11 @@ void translate_box_len(string const & length, string & value, string & unit, str
 string find_file(string const & name, string const & path,
                 char const * const * extensions)
 {
+       // FIXME UNICODE encoding of name and path may be wrong (makeAbsPath
+       // expects utf8)
        for (char const * const * what = extensions; *what; ++what) {
-               // We don't use ChangeExtension() because it does the wrong
-               // thing if name contains a dot.
-               string const trial = name + '.' + (*what);
-               if (FileInfo(MakeAbsPath(trial, path)).exist())
+               string const trial = addExtension(name, *what);
+               if (fs::exists(makeAbsPath(trial, path).toFilesystemEncoding()))
                        return trial;
        }
        return string();
@@ -348,12 +391,10 @@ void skip_braces(Parser & p)
 }
 
 
-void handle_ert(ostream & os, string const & s, Context & context, bool check_layout = true)
+void handle_ert(ostream & os, string const & s, Context & context)
 {
-       if (check_layout) {
-               // We must have a valid layout before outputting the ERT inset.
-               context.check_layout(os);
-       }
+       // We must have a valid layout before outputting the ERT inset.
+       context.check_layout(os);
        Context newcontext(true, context.textclass);
        begin_inset(os, "ERT");
        os << "\nstatus collapsed\n";
@@ -361,9 +402,10 @@ void handle_ert(ostream & os, string const & s, Context & context, bool check_la
        for (string::const_iterator it = s.begin(), et = s.end(); it != et; ++it) {
                if (*it == '\\')
                        os << "\n\\backslash\n";
-               else if (*it == '\n')
-                       os << "\n\\newline\n";
-               else
+               else if (*it == '\n') {
+                       newcontext.new_paragraph(os);
+                       newcontext.check_layout(os);
+               } else
                        os << *it;
        }
        newcontext.check_end_layout(os);
@@ -385,7 +427,8 @@ void handle_comment(ostream & os, string const & s, Context & context)
                        os << *it;
        }
        // make sure that our comment is the last thing on the line
-       os << "\n\\newline";
+       newcontext.new_paragraph(os);
+       newcontext.check_layout(os);
        newcontext.check_end_layout(os);
        end_inset(os);
 }
@@ -405,7 +448,7 @@ private:
 LyXLayout_ptr findLayout(LyXTextClass const & textclass,
                         string const & name)
 {
-       LyXTextClass::const_iterator beg  = textclass.begin();
+       LyXTextClass::const_iterator beg = textclass.begin();
        LyXTextClass::const_iterator end = textclass.end();
 
        LyXTextClass::const_iterator
@@ -444,7 +487,7 @@ void output_command_layout(ostream & os, Parser & p, bool outer,
                        eat_whitespace(p, os, context, false);
                }
        }
-       parse_text_snippet(p, os, FLAG_ITEM, outer, context);
+       parse_text(p, os, FLAG_ITEM, outer, context);
        context.check_end_layout(os);
        if (parent_context.deeper_paragraph) {
                // We must suppress the "end deeper" because we
@@ -494,6 +537,37 @@ void check_space(Parser const & p, ostream & os, Context & context)
 }
 
 
+/*!
+ * Parse all arguments of \p command
+ */
+void parse_arguments(string const & command,
+                    vector<ArgumentType> const & template_arguments,
+                    Parser & p, ostream & os, bool outer, Context & context)
+{
+       string ert = command;
+       size_t no_arguments = template_arguments.size();
+       for (size_t i = 0; i < no_arguments; ++i) {
+               switch (template_arguments[i]) {
+               case required:
+                       // This argument contains regular LaTeX
+                       handle_ert(os, ert + '{', context);
+                       eat_whitespace(p, os, context, false);
+                       parse_text(p, os, FLAG_ITEM, outer, context);
+                       ert = "}";
+                       break;
+               case verbatim:
+                       // This argument may contain special characters
+                       ert += '{' + p.verbatim_item() + '}';
+                       break;
+               case optional:
+                       ert += p.getOpt();
+                       break;
+               }
+       }
+       handle_ert(os, ert, context);
+}
+
+
 /*!
  * Check whether \p command is a known command. If yes,
  * handle the command with all arguments.
@@ -503,27 +577,8 @@ bool parse_command(string const & command, Parser & p, ostream & os,
                   bool outer, Context & context)
 {
        if (known_commands.find(command) != known_commands.end()) {
-               vector<ArgumentType> const & template_arguments = known_commands[command];
-               string ert = command;
-               size_t no_arguments = template_arguments.size();
-               for (size_t i = 0; i < no_arguments; ++i) {
-                       switch (template_arguments[i]) {
-                       case required:
-                               // This argument contains regular LaTeX
-                               handle_ert(os, ert + '{', context);
-                               parse_text(p, os, FLAG_ITEM, outer, context);
-                               ert = "}";
-                               break;
-                       case verbatim:
-                               // This argument may contain special characters
-                               ert += '{' + p.verbatim_item() + '}';
-                               break;
-                       case optional:
-                               ert += p.getOpt();
-                               break;
-                       }
-               }
-               handle_ert(os, ert, context);
+               parse_arguments(command, known_commands[command], p, os,
+                               outer, context);
                return true;
        }
        return false;
@@ -632,6 +687,33 @@ void parse_box(Parser & p, ostream & os, unsigned flags, bool outer,
 }
 
 
+/// parse an unknown environment
+void parse_unknown_environment(Parser & p, string const & name, ostream & os,
+                              unsigned flags, bool outer,
+                              Context & parent_context)
+{
+       if (name == "tabbing")
+               // We need to remember that we have to handle '\=' specially
+               flags |= FLAG_TABBING;
+
+       // We need to translate font changes and paragraphs inside the
+       // environment to ERT if we have a non standard font.
+       // Otherwise things like
+       // \large\begin{foo}\huge bar\end{foo}
+       // will not work.
+       bool const specialfont =
+               (parent_context.font != parent_context.normalfont);
+       bool const new_layout_allowed = parent_context.new_layout_allowed;
+       if (specialfont)
+               parent_context.new_layout_allowed = false;
+       handle_ert(os, "\\begin{" + name + "}", parent_context);
+       parse_text_snippet(p, os, flags, outer, parent_context);
+       handle_ert(os, "\\end{" + name + "}", parent_context);
+       if (specialfont)
+               parent_context.new_layout_allowed = new_layout_allowed;
+}
+
+
 void parse_environment(Parser & p, ostream & os, bool outer,
                       Context & parent_context)
 {
@@ -639,7 +721,6 @@ void parse_environment(Parser & p, ostream & os, bool outer,
        string const name = p.getArg('{', '}');
        const bool is_starred = suffixIs(name, '*');
        string const unstarred_name = rtrim(name, "*");
-       eat_whitespace(p, os, parent_context, false);
        active_environments.push_back(name);
 
        if (is_math_env(name)) {
@@ -651,20 +732,23 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                end_inset(os);
        }
 
-       else if (name == "tabular") {
+       else if (name == "tabular" || name == "longtable") {
+               eat_whitespace(p, os, parent_context, false);
                parent_context.check_layout(os);
                begin_inset(os, "Tabular ");
-               handle_tabular(p, os, parent_context);
+               handle_tabular(p, os, name == "longtable", parent_context);
                end_inset(os);
+               p.skip_spaces();
        }
 
        else if (parent_context.textclass.floats().typeExist(unstarred_name)) {
+               eat_whitespace(p, os, parent_context, false);
                parent_context.check_layout(os);
                begin_inset(os, "Float " + unstarred_name + "\n");
                if (p.next_token().asInput() == "[") {
                        os << "placement " << p.getArg('[', ']') << '\n';
                }
-               os << "wide " << tostr(is_starred)
+               os << "wide " << convert<string>(is_starred)
                   << "\nsideways false"
                   << "\nstatus open\n\n";
                parse_text_in_inset(p, os, FLAG_END, outer, parent_context);
@@ -672,35 +756,66 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                // We don't need really a new paragraph, but
                // we must make sure that the next item gets a \begin_layout.
                parent_context.new_paragraph(os);
+               p.skip_spaces();
        }
 
-       else if (name == "minipage")
+       else if (name == "minipage") {
+               eat_whitespace(p, os, parent_context, false);
                parse_box(p, os, FLAG_END, outer, parent_context, false);
+               p.skip_spaces();
+       }
+
+       else if (name == "comment") {
+               eat_whitespace(p, os, parent_context, false);
+               parent_context.check_layout(os);
+               begin_inset(os, "Note Comment\n");
+               os << "status open\n";
+               parse_text_in_inset(p, os, FLAG_END, outer, parent_context);
+               end_inset(os);
+               p.skip_spaces();
+       }
+
+       else if (name == "lyxgreyedout") {
+               eat_whitespace(p, os, parent_context, false);
+               parent_context.check_layout(os);
+               begin_inset(os, "Note Greyedout\n");
+               os << "status open\n";
+               parse_text_in_inset(p, os, FLAG_END, outer, parent_context);
+               end_inset(os);
+               p.skip_spaces();
+       }
+
+       else if (!parent_context.new_layout_allowed)
+               parse_unknown_environment(p, name, os, FLAG_END, outer,
+                                         parent_context);
 
        // Alignment settings
        else if (name == "center" || name == "flushleft" || name == "flushright" ||
-                name == "centering" || name == "raggedright" || name == "raggedleft") {
+                name == "centering" || name == "raggedright" || name == "raggedleft") {
+               eat_whitespace(p, os, parent_context, false);
                // We must begin a new paragraph if not already done
                if (! parent_context.atParagraphStart()) {
                        parent_context.check_end_layout(os);
                        parent_context.new_paragraph(os);
                }
                if (name == "flushleft" || name == "raggedright")
-                       parent_context.add_extra_stuff("\\align left ");
+                       parent_context.add_extra_stuff("\\align left\n");
                else if (name == "flushright" || name == "raggedleft")
-                       parent_context.add_extra_stuff("\\align right ");
+                       parent_context.add_extra_stuff("\\align right\n");
                else
-                       parent_context.add_extra_stuff("\\align center ");
+                       parent_context.add_extra_stuff("\\align center\n");
                parse_text(p, os, FLAG_END, outer, parent_context);
                // Just in case the environment is empty ..
                parent_context.extra_stuff.erase();
                // We must begin a new paragraph to reset the alignment
                parent_context.new_paragraph(os);
+               p.skip_spaces();
        }
 
        // The single '=' is meant here.
        else if ((newlayout = findLayout(parent_context.textclass, name)).get() &&
-                  newlayout->isEnvironment()) {
+                 newlayout->isEnvironment()) {
+               eat_whitespace(p, os, parent_context, false);
                Context context(true, parent_context.textclass, newlayout,
                                parent_context.layout, parent_context.font);
                if (parent_context.deeper_paragraph) {
@@ -733,10 +848,12 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                }
                context.check_end_deeper(os);
                parent_context.new_paragraph(os);
+               p.skip_spaces();
        }
 
        else if (name == "appendix") {
                // This is no good latex style, but it works and is used in some documents...
+               eat_whitespace(p, os, parent_context, false);
                parent_context.check_end_layout(os);
                Context context(true, parent_context.textclass, parent_context.layout,
                                parent_context.layout, parent_context.font);
@@ -744,57 +861,65 @@ void parse_environment(Parser & p, ostream & os, bool outer,
                os << "\\start_of_appendix\n";
                parse_text(p, os, FLAG_END, outer, context);
                context.check_end_layout(os);
+               p.skip_spaces();
        }
 
-       else if (name == "comment") {
-               parent_context.check_layout(os);
-               begin_inset(os, "Note Comment\n");
-               os << "status open\n";
-               parse_text_in_inset(p, os, FLAG_END, outer, parent_context);
-               end_inset(os);
-       }
-
-       else if (name == "lyxgreyedout") {
-               parent_context.check_layout(os);
-               begin_inset(os, "Note Greyedout\n");
-               os << "status open\n";
-               parse_text_in_inset(p, os, FLAG_END, outer, parent_context);
-               end_inset(os);
-       }
-
-       else if (name == "tabbing") {
-               // We need to remember that we have to handle '\=' specially
-               handle_ert(os, "\\begin{" + name + "}", parent_context);
-               parse_text_snippet(p, os, FLAG_END | FLAG_TABBING, outer, parent_context);
+       else if (known_environments.find(name) != known_environments.end()) {
+               vector<ArgumentType> arguments = known_environments[name];
+               // The last "argument" denotes wether we may translate the
+               // environment contents to LyX
+               // The default required if no argument is given makes us
+               // compatible with the reLyXre environment.
+               ArgumentType contents = arguments.empty() ?
+                       required :
+                       arguments.back();
+               if (!arguments.empty())
+                       arguments.pop_back();
+               // See comment in parse_unknown_environment()
+               bool const specialfont =
+                       (parent_context.font != parent_context.normalfont);
+               bool const new_layout_allowed =
+                       parent_context.new_layout_allowed;
+               if (specialfont)
+                       parent_context.new_layout_allowed = false;
+               parse_arguments("\\begin{" + name + "}", arguments, p, os,
+                               outer, parent_context);
+               if (contents == verbatim)
+                       handle_ert(os, p.verbatimEnvironment(name),
+                                  parent_context);
+               else
+                       parse_text_snippet(p, os, FLAG_END, outer,
+                                          parent_context);
                handle_ert(os, "\\end{" + name + "}", parent_context);
+               if (specialfont)
+                       parent_context.new_layout_allowed = new_layout_allowed;
        }
 
-       else {
-               handle_ert(os, "\\begin{" + name + "}", parent_context);
-               parse_text_snippet(p, os, FLAG_END, outer, parent_context);
-               handle_ert(os, "\\end{" + name + "}", parent_context);
-       }
+       else
+               parse_unknown_environment(p, name, os, FLAG_END, outer,
+                                         parent_context);
 
        active_environments.pop_back();
-       if (name != "math")
-               p.skip_spaces();
 }
 
+
 /// parses a comment and outputs it to \p os.
 void parse_comment(Parser & p, ostream & os, Token const & t, Context & context)
 {
        BOOST_ASSERT(t.cat() == catComment);
-       context.check_layout(os);
        if (!t.cs().empty()) {
+               context.check_layout(os);
                handle_comment(os, '%' + t.cs(), context);
                if (p.next_token().cat() == catNewline) {
                        // A newline after a comment line starts a new
                        // paragraph
-                       if(!context.atParagraphStart()) {
-                               // Only start a new paragraph if not already
-                               // done (we might get called recursively)
-                               context.new_paragraph(os);
-                       }
+                       if (context.new_layout_allowed) {
+                               if(!context.atParagraphStart())
+                                       // Only start a new paragraph if not already
+                                       // done (we might get called recursively)
+                                       context.new_paragraph(os);
+                       } else
+                               handle_ert(os, "\n", context);
                        eat_whitespace(p, os, context, true);
                }
        } else {
@@ -818,7 +943,7 @@ void eat_whitespace(Parser & p, ostream & os, Context & context,
                if (t.cat() == catComment)
                        parse_comment(p, os, t, context);
                else if ((! eatParagraph && p.isParagraph()) ||
-                        (t.cat() != catSpace && t.cat() != catNewline)) {
+                        (t.cat() != catSpace && t.cat() != catNewline)) {
                        p.putback();
                        return;
                }
@@ -838,41 +963,137 @@ void parse_text_attributes(Parser & p, ostream & os, unsigned flags, bool outer,
                           string & currentvalue, string const & newvalue)
 {
        context.check_layout(os);
-       string oldvalue = currentvalue;
+       string const oldvalue = currentvalue;
        currentvalue = newvalue;
        os << '\n' << attribute << ' ' << newvalue << "\n";
        parse_text_snippet(p, os, flags, outer, context);
-       currentvalue = oldvalue;
+       context.check_layout(os);
        os << '\n' << attribute << ' ' << oldvalue << "\n";
+       currentvalue = oldvalue;
 }
 
 
 /// get the arguments of a natbib or jurabib citation command
-std::pair<string, string> getCiteArguments(Parser & p, ostream & os,
-                                          Context & context, bool natbibOrder)
+std::pair<string, string> getCiteArguments(Parser & p, bool natbibOrder)
 {
        // We need to distinguish "" and "[]", so we can't use p.getOpt().
 
        // text before the citation
        string before;
        // text after the citation
-       string after;
+       string after = p.getFullOpt();
 
-       eat_whitespace(p, os, context, false);
-       if (p.next_token().asInput() == "[") {
-               after = '[' + p.getArg('[', ']') + ']';
-               eat_whitespace(p, os, context, false);
-               if (natbibOrder) {
-                       if (p.next_token().asInput() == "[") {
-                               before = after;
-                               after = '[' + p.getArg('[', ']') + ']';
+       if (!after.empty()) {
+               before = p.getFullOpt();
+               if (natbibOrder && !before.empty())
+                       std::swap(before, after);
+       }
+       return std::make_pair(before, after);
+}
+
+
+/// Convert filenames with TeX macros and/or quotes to something LyX can understand
+string const normalize_filename(string const & name)
+{
+       Parser p(trim(name, "\""));
+       ostringstream os;
+       while (p.good()) {
+               Token const & t = p.get_token();
+               if (t.cat() != catEscape)
+                       os << t.asInput();
+               else if (t.cs() == "lyxdot") {
+                       // This is used by LyX for simple dots in relative
+                       // names
+                       os << '.';
+                       p.skip_spaces();
+               } else if (t.cs() == "space") {
+                       os << ' ';
+                       p.skip_spaces();
+               } else
+                       os << t.asInput();
+       }
+       return os.str();
+}
+
+
+/// Convert \p name from TeX convention (relative to master file) to LyX
+/// convention (relative to .lyx file) if it is relative
+void fix_relative_filename(string & name)
+{
+       if (lyx::support::absolutePath(name))
+               return;
+       // FIXME UNICODE encoding of name may be wrong (makeAbsPath expects
+       // utf8)
+       name = makeRelPath(makeAbsPath(name, getMasterFilePath()).absFilename(),
+                          getParentFilePath());
+}
+
+
+/// Parse a NoWeb Scrap 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;
+       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)
+                               name += p.get_token().asInput();
+                       break;
+               }
+               name += t.asInput();
+       }
+
+       if (!scrap || !context.new_layout_allowed ||
+           !context.textclass.hasLayout("Scrap")) {
+               cerr << "Warning: Could not interpret '" << name
+                    << "'. Ignoring it." << endl;
+               return;
+       }
+
+       // We use new_paragraph instead of check_end_layout because the stuff
+       // following the noweb chunk needs to start with a \begin_layout.
+       // This may create a new paragraph even if there was none in the
+       // noweb file, but the alternative is an invalid LyX file. Since
+       // noweb code chunks are implemented with a layout style in LyX they
+       // always must be in an own paragraph.
+       context.new_paragraph(os);
+       Context newcontext(true, context.textclass, context.textclass["Scrap"]);
+       newcontext.check_layout(os);
+       os << name;
+       while (p.good()) {
+               Token const & t = p.get_token();
+               // We abuse the parser a bit, because this is no TeX syntax
+               // at all.
+               if (t.cat() == catEscape)
+                       os << subst(t.asInput(), "\\", "\n\\backslash\n");
+               else
+                       os << subst(t.asInput(), "\n", "\n\\newline\n");
+               // The scrap 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 &&
+                   (p.next_token().cat() == catSpace ||
+                    p.next_token().cat() == catNewline ||
+                    p.next_token().cat() == catComment)) {
+                       while (p.good() && p.next_token().cat() == catSpace)
+                               os << p.get_token().asInput();
+                       if (p.next_token().cat() == catComment)
+                               // The comment includes a final '\n'
+                               os << p.get_token().asInput();
+                       else {
+                               if (p.next_token().cat() == catNewline)
+                                       p.get_token();
+                               os << '\n';
                        }
-               } else {
-                       if (p.next_token().asInput() == "[")
-                               before = '[' + p.getArg('[', ']') + ']';
+                       break;
                }
        }
-       return std::make_pair(before, after);
+       newcontext.check_end_layout(os);
 }
 
 } // anonymous namespace
@@ -962,19 +1183,42 @@ 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);
 
+               else if (t.character() == '[' && noweb_mode &&
+                        p.next_token().character() == '[') {
+                       // These can contain underscores
+                       p.putback();
+                       string const s = p.getFullOpt() + ']';
+                       if (p.next_token().character() == ']')
+                               p.get_token();
+                       else
+                               cerr << "Warning: Inserting missing ']' in '"
+                                    << s << "'." << endl;
+                       handle_ert(os, s, context);
+               }
+
                else if (t.cat() == catLetter ||
                               t.cat() == catOther ||
                               t.cat() == catAlign ||
                               t.cat() == catParameter) {
+                       // This translates "&" to "\\&" which may be wrong...
                        context.check_layout(os);
                        os << t.character();
                }
 
                else if (p.isParagraph()) {
-                       context.new_paragraph(os);
+                       if (context.new_layout_allowed)
+                               context.new_paragraph(os);
+                       else
+                               handle_ert(os, "\\par ", context);
                        eat_whitespace(p, os, context, true);
                }
 
@@ -989,56 +1233,96 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                os << t.character();
                }
 
+               else if (t.cat() == catBegin &&
+                        p.next_token().cat() == catEnd) {
+                       // {}
+                       Token const prev = p.prev_token();
+                       p.get_token();
+                       if (p.next_token().character() == '`' ||
+                           (prev.character() == '-' &&
+                            p.next_token().character() == '-'))
+                               ; // ignore it in {}`` or -{}-
+                       else
+                               handle_ert(os, "{}", context);
+
+               }
+
                else if (t.cat() == catBegin) {
                        context.check_layout(os);
                        // special handling of font attribute changes
                        Token const prev = p.prev_token();
                        Token const next = p.next_token();
                        Font const oldFont = context.font;
-                       string const s = parse_text(p, FLAG_BRACE_LAST, outer,
-                                                   context);
-                       context.font = oldFont;
-                       if (s.empty() && (p.next_token().character() == '`' ||
-                                         (prev.character() == '-' &&
-                                          p.next_token().character() == '-')))
-                               ; // ignore it in {}`` or -{}-
-                       else if (s == "[" || s == "]" || s == "*")
-                               os << s;
-                       else if (is_known(next.cs(), known_sizes))
-                               // s will change the size, so we must reset
-                               // it here
-                               os << s << "\n\\size " << context.font.size
-                                  << "\n";
-                       else if (is_known(next.cs(), known_font_families))
-                               // s will change the font family, so we must
-                               // reset it here
-                               os << s << "\n\\family "
-                                  << context.font.family << "\n";
-                       else if (is_known(next.cs(), known_font_series))
-                               // s will change the font series, so we must
-                               // reset it here
-                               os << s << "\n\\series "
-                                  << context.font.series << "\n";
-                       else if (is_known(next.cs(), known_font_shapes))
-                               // s will change the font shape, so we must
+                       if (next.character() == '[' ||
+                           next.character() == ']' ||
+                           next.character() == '*') {
+                               p.get_token();
+                               if (p.next_token().cat() == catEnd) {
+                                       os << next.character();
+                                       p.get_token();
+                               } else {
+                                       p.putback();
+                                       handle_ert(os, "{", context);
+                                       parse_text_snippet(p, os,
+                                                       FLAG_BRACE_LAST,
+                                                       outer, context);
+                                       handle_ert(os, "}", context);
+                               }
+                       } else if (! context.new_layout_allowed) {
+                               handle_ert(os, "{", context);
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
+                               handle_ert(os, "}", context);
+                       } else if (is_known(next.cs(), known_sizes)) {
+                               // next will change the size, so we must
                                // reset it here
-                               os << s << "\n\\shape "
-                                  << context.font.shape << "\n";
-                       else if (is_known(next.cs(), known_old_font_families) ||
-                                is_known(next.cs(), known_old_font_series) ||
-                                is_known(next.cs(), known_old_font_shapes))
-                               // s will change the font family, series
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
+                               if (!context.atParagraphStart())
+                                       os << "\n\\size "
+                                          << context.font.size << "\n";
+                       } else if (is_known(next.cs(), known_font_families)) {
+                               // next will change the font family, so we
+                               // must reset it here
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
+                               if (!context.atParagraphStart())
+                                       os << "\n\\family "
+                                          << context.font.family << "\n";
+                       } else if (is_known(next.cs(), known_font_series)) {
+                               // next will change the font series, so we
+                               // must reset it here
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
+                               if (!context.atParagraphStart())
+                                       os << "\n\\series "
+                                          << context.font.series << "\n";
+                       } else if (is_known(next.cs(), known_font_shapes)) {
+                               // next will change the font shape, so we
+                               // must reset it here
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
+                               if (!context.atParagraphStart())
+                                       os << "\n\\shape "
+                                          << context.font.shape << "\n";
+                       } else if (is_known(next.cs(), known_old_font_families) ||
+                                  is_known(next.cs(), known_old_font_series) ||
+                                  is_known(next.cs(), known_old_font_shapes)) {
+                               // next will change the font family, series
                                // and shape, so we must reset it here
-                               os << s
-                                  <<  "\n\\family " << context.font.family
-                                  << "\n\\series " << context.font.series
-                                  << "\n\\shape "  << context.font.shape
-                                  << "\n";
-                       else {
-                               handle_ert(os, "{", context, false);
-                               // s will end the current layout and begin a
-                               // new one if necessary
-                               os << s;
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
+                               if (!context.atParagraphStart())
+                                       os <<  "\n\\family "
+                                          << context.font.family
+                                          << "\n\\series "
+                                          << context.font.series
+                                          << "\n\\shape "
+                                          << context.font.shape << "\n";
+                       } else {
+                               handle_ert(os, "{", context);
+                               parse_text_snippet(p, os, FLAG_BRACE_LAST,
+                                                  outer, context);
                                handle_ert(os, "}", context);
                        }
                }
@@ -1097,13 +1381,21 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        bool optarg = false;
                        if (p.next_token().character() == '[') {
                                p.get_token(); // eat '['
-                               Context newcontext(false, context.textclass);
-                               newcontext.font = context.font;
-                               s = parse_text(p, FLAG_BRACK_LAST, outer, newcontext);
+                               s = parse_text_snippet(p, FLAG_BRACK_LAST,
+                                                      outer, context);
                                optarg = true;
                        }
                        context.set_item();
                        context.check_layout(os);
+                       if (context.has_item) {
+                               // An item in an unknown list-like environment
+                               // FIXME: Do this in check_layout()!
+                               context.has_item = false;
+                               if (optarg)
+                                       handle_ert(os, "\\item", context);
+                               else
+                                       handle_ert(os, "\\item ", context);
+                       }
                        if (optarg) {
                                if (context.layout->labeltype != LABEL_MANUAL) {
                                        // lyx does not support \item[\mybullet]
@@ -1139,16 +1431,35 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
 
                else if (t.cs() == "noindent") {
                        p.skip_spaces();
-                       context.add_extra_stuff("\\noindent ");
+                       context.add_extra_stuff("\\noindent\n");
                }
 
                else if (t.cs() == "appendix") {
-                       p.skip_spaces();
-                       context.add_extra_stuff("\\start_of_appendix ");
+                       context.add_extra_stuff("\\start_of_appendix\n");
+                       // We need to start a new paragraph. Otherwise the
+                       // appendix in 'bla\appendix\chapter{' would start
+                       // too late.
+                       context.new_paragraph(os);
+                       // We need to make sure that the paragraph is
+                       // generated even if it is empty. Otherwise the
+                       // appendix in '\par\appendix\par\chapter{' would
+                       // start too late.
+                       context.check_layout(os);
+                       // FIXME: This is a hack to prevent paragraph
+                       // deletion if it is empty. Handle this better!
+                       handle_comment(os,
+                               "%dummy comment inserted by tex2lyx to "
+                               "ensure that this paragraph is not empty",
+                               context);
+                       // Both measures above may generate an additional
+                       // empty paragraph, but that does not hurt, because
+                       // whitespace does not matter here.
+                       eat_whitespace(p, os, context, true);
                }
 
                // Must attempt to parse "Section*" before "Section".
                else if ((p.next_token().asInput() == "*") &&
+                        context.new_layout_allowed &&
                         // The single '=' is meant here.
                         (newlayout = findLayout(context.textclass,
                                                 t.cs() + '*')).get() &&
@@ -1159,53 +1470,65 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                }
 
                // The single '=' is meant here.
-               else if ((newlayout = findLayout(context.textclass, t.cs())).get() &&
+               else if (context.new_layout_allowed &&
+                        (newlayout = findLayout(context.textclass, t.cs())).get() &&
                         newlayout->isCommand()) {
                        output_command_layout(os, p, outer, context, newlayout);
                        p.skip_spaces();
                }
 
                else if (t.cs() == "includegraphics") {
+                       bool const clip = p.next_token().asInput() == "*";
+                       if (clip)
+                               p.get_token();
                        map<string, string> opts = split_map(p.getArg('[', ']'));
-                       string name = subst(p.verbatim_item(), "\\lyxdot ", ".");
+                       if (clip)
+                               opts["clip"] = string();
+                       string name = normalize_filename(p.verbatim_item());
 
                        string const path = getMasterFilePath();
                        // We want to preserve relative / absolute filenames,
                        // therefore path is only used for testing
-                       if (!FileInfo(MakeAbsPath(name, path)).exist()) {
+                       // FIXME UNICODE encoding of name and path may be
+                       // wrong (makeAbsPath expects utf8)
+                       if (!fs::exists(makeAbsPath(name, path).toFilesystemEncoding())) {
                                // The file extension is probably missing.
                                // Now try to find it out.
                                string const dvips_name =
                                        find_file(name, path,
-                                                 known_dvips_graphics_formats);
+                                                 known_dvips_graphics_formats);
                                string const pdftex_name =
                                        find_file(name, path,
-                                                 known_pdftex_graphics_formats);
+                                                 known_pdftex_graphics_formats);
                                if (!dvips_name.empty()) {
                                        if (!pdftex_name.empty()) {
                                                cerr << "This file contains the "
-                                                       "latex snippet\n"
-                                                       "\"\\includegraphics{"
+                                                       "latex snippet\n"
+                                                       "\"\\includegraphics{"
                                                     << name << "}\".\n"
-                                                       "However, files\n\""
+                                                       "However, files\n\""
                                                     << dvips_name << "\" and\n\""
                                                     << pdftex_name << "\"\n"
-                                                       "both exist, so I had to make a "
-                                                       "choice and took the first one.\n"
-                                                       "Please move the unwanted one "
-                                                       "someplace else and try again\n"
-                                                       "if my choice was wrong."
+                                                       "both exist, so I had to make a "
+                                                       "choice and took the first one.\n"
+                                                       "Please move the unwanted one "
+                                                       "someplace else and try again\n"
+                                                       "if my choice was wrong."
                                                     << endl;
                                        }
                                        name = dvips_name;
                                } else if (!pdftex_name.empty())
                                        name = pdftex_name;
-
-                               if (!FileInfo(MakeAbsPath(name, path)).exist())
-                                       cerr << "Warning: Could not find graphics file '"
-                                            << name << "'." << endl;
                        }
 
+                       // FIXME UNICODE encoding of name and path may be
+                       // wrong (makeAbsPath expects utf8)
+                       if (fs::exists(makeAbsPath(name, path).toFilesystemEncoding()))
+                               fix_relative_filename(name);
+                       else
+                               cerr << "Warning: Could not find graphics file '"
+                                    << name << "'." << endl;
+
                        context.check_layout(os);
                        begin_inset(os, "Graphics ");
                        os << "\n\tfilename " << name << '\n';
@@ -1301,7 +1624,8 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        end_inset(os);
                }
 
-               else if (t.cs() == "footnote") {
+               else if (t.cs() == "footnote" ||
+                        (t.cs() == "thanks" && context.layout->intitle)) {
                        p.skip_spaces();
                        context.check_layout(os);
                        begin_inset(os, "Foot\n");
@@ -1322,9 +1646,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                else if (t.cs() == "ensuremath") {
                        p.skip_spaces();
                        context.check_layout(os);
-                       Context newcontext(false, context.textclass);
-                       newcontext.font = context.font;
-                       string s = parse_text(p, FLAG_ITEM, false, newcontext);
+                       string const s = p.verbatim_item();
                        if (s == "±" || s == "³" || s == "²" || s == "µ")
                                os << s;
                        else
@@ -1340,6 +1662,8 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                }
 
                else if (t.cs() == "makeindex" || t.cs() == "maketitle") {
+                       // FIXME: Somehow prevent title layouts if
+                       // "maketitle" was not found
                        p.skip_spaces();
                        skip_braces(p); // swallow this
                }
@@ -1383,48 +1707,48 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
 
                else if (t.cs() == "textrm")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\family",
-                                             context.font.family, "roman");
+                                             context, "\\family",
+                                             context.font.family, "roman");
 
                else if (t.cs() == "textsf")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\family",
-                                             context.font.family, "sans");
+                                             context, "\\family",
+                                             context.font.family, "sans");
 
                else if (t.cs() == "texttt")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\family",
-                                             context.font.family, "typewriter");
+                                             context, "\\family",
+                                             context.font.family, "typewriter");
 
                else if (t.cs() == "textmd")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\series",
-                                             context.font.series, "medium");
+                                             context, "\\series",
+                                             context.font.series, "medium");
 
                else if (t.cs() == "textbf")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\series",
-                                             context.font.series, "bold");
+                                             context, "\\series",
+                                             context.font.series, "bold");
 
                else if (t.cs() == "textup")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\shape",
-                                             context.font.shape, "up");
+                                             context, "\\shape",
+                                             context.font.shape, "up");
 
                else if (t.cs() == "textit")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\shape",
-                                             context.font.shape, "italic");
+                                             context, "\\shape",
+                                             context.font.shape, "italic");
 
                else if (t.cs() == "textsl")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\shape",
-                                             context.font.shape, "slanted");
+                                             context, "\\shape",
+                                             context.font.shape, "slanted");
 
                else if (t.cs() == "textsc")
                        parse_text_attributes(p, os, FLAG_ITEM, outer,
-                                             context, "\\shape",
-                                             context.font.shape, "smallcaps");
+                                             context, "\\shape",
+                                             context.font.shape, "smallcaps");
 
                else if (t.cs() == "textnormal" || t.cs() == "normalfont") {
                        context.check_layout(os);
@@ -1436,18 +1760,20 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        os << "\n\\shape " << context.font.shape << "\n";
                        if (t.cs() == "textnormal") {
                                parse_text_snippet(p, os, FLAG_ITEM, outer, context);
+                               output_font_change(os, context.font, oldFont);
                                context.font = oldFont;
-                               os << "\n\\shape " << oldFont.shape << "\n";
-                               os << "\n\\series " << oldFont.series << "\n";
-                               os << "\n\\family " << oldFont.family << "\n";
                        } else
                                eat_whitespace(p, os, context, false);
                }
 
                else if (t.cs() == "underbar") {
+                       // Do NOT handle \underline.
+                       // \underbar cuts through y, g, q, p etc.,
+                       // \underline does not.
                        context.check_layout(os);
                        os << "\n\\bar under\n";
                        parse_text_snippet(p, os, FLAG_ITEM, outer, context);
+                       context.check_layout(os);
                        os << "\n\\bar default\n";
                }
 
@@ -1455,15 +1781,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        context.check_layout(os);
                        os << "\n\\" << t.cs() << " on\n";
                        parse_text_snippet(p, os, FLAG_ITEM, outer, context);
+                       context.check_layout(os);
                        os << "\n\\" << t.cs() << " default\n";
                }
 
                else if (use_natbib &&
-                        is_known(t.cs(), known_natbib_commands) &&
-                        ((t.cs() != "citefullauthor" &&
+                        is_known(t.cs(), known_natbib_commands) &&
+                        ((t.cs() != "citefullauthor" &&
                           t.cs() != "citeyear" &&
                           t.cs() != "citeyearpar") ||
-                         p.next_token().asInput() != "*")) {
+                         p.next_token().asInput() != "*")) {
                        context.check_layout(os);
                        // tex                       lyx
                        // \citet[before][after]{a}  \citet[after][before]{a}
@@ -1484,8 +1811,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        // text after the citation
                        string after;
 
-                       boost::tie(before, after) =
-                               getCiteArguments(p, os, context, true);
+                       boost::tie(before, after) = getCiteArguments(p, true);
                        if (command == "\\cite") {
                                // \cite without optional argument means
                                // \citet, \cite with at least one optional
@@ -1510,16 +1836,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                }
 
                else if (use_jurabib &&
-                        is_known(t.cs(), known_jurabib_commands)) {
+                        is_known(t.cs(), known_jurabib_commands)) {
                        context.check_layout(os);
                        string const command = '\\' + t.cs();
                        char argumentOrder = '\0';
                        vector<string> const & options = used_packages["jurabib"];
                        if (std::find(options.begin(), options.end(),
-                                     "natbiborder") != options.end())
+                                     "natbiborder") != options.end())
                                argumentOrder = 'n';
                        else if (std::find(options.begin(), options.end(),
-                                          "jurabiborder") != options.end())
+                                          "jurabiborder") != options.end())
                                argumentOrder = 'j';
 
                        // text before the citation
@@ -1528,17 +1854,16 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        string after;
 
                        boost::tie(before, after) =
-                               getCiteArguments(p, os, context,
-                                                argumentOrder != 'j');
+                               getCiteArguments(p, argumentOrder != 'j');
                        string const citation = p.verbatim_item();
                        if (!before.empty() && argumentOrder == '\0') {
                                cerr << "Warning: Assuming argument order "
-                                    << "of jurabib version 0.6 for\n'"
+                                       "of jurabib version 0.6 for\n'"
                                     << command << before << after << '{'
                                     << citation << "}'.\n"
-                                    << "Add 'jurabiborder' to the jurabib "
-                                    << "package options if you used an\n"
-                                    << "earlier jurabib version." << endl;
+                                       "Add 'jurabiborder' to the jurabib "
+                                       "package options if you used an\n"
+                                       "earlier jurabib version." << endl;
                        }
                        begin_inset(os, "LatexCommand ");
                        os << command << after << before
@@ -1574,85 +1899,90 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        skip_braces(p);
                }
 
-               else if (is_known(t.cs(), known_sizes)) {
+               else if (is_known(t.cs(), known_sizes) &&
+                        context.new_layout_allowed) {
                        char const * const * where = is_known(t.cs(), known_sizes);
                        context.check_layout(os);
+                       Font const oldFont = context.font;
                        context.font.size = known_coded_sizes[where - known_sizes];
-                       os << "\n\\size " << context.font.size << '\n';
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
 
-               else if (is_known(t.cs(), known_font_families)) {
+               else if (is_known(t.cs(), known_font_families) &&
+                        context.new_layout_allowed) {
                        char const * const * where =
                                is_known(t.cs(), known_font_families);
                        context.check_layout(os);
+                       Font const oldFont = context.font;
                        context.font.family =
                                known_coded_font_families[where - known_font_families];
-                       os << "\n\\family " << context.font.family << '\n';
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
 
-               else if (is_known(t.cs(), known_font_series)) {
+               else if (is_known(t.cs(), known_font_series) &&
+                        context.new_layout_allowed) {
                        char const * const * where =
                                is_known(t.cs(), known_font_series);
                        context.check_layout(os);
+                       Font const oldFont = context.font;
                        context.font.series =
                                known_coded_font_series[where - known_font_series];
-                       os << "\n\\series " << context.font.series << '\n';
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
 
-               else if (is_known(t.cs(), known_font_shapes)) {
+               else if (is_known(t.cs(), known_font_shapes) &&
+                        context.new_layout_allowed) {
                        char const * const * where =
                                is_known(t.cs(), known_font_shapes);
                        context.check_layout(os);
+                       Font const oldFont = context.font;
                        context.font.shape =
                                known_coded_font_shapes[where - known_font_shapes];
-                       os << "\n\\shape " << context.font.shape << '\n';
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
-               else if (is_known(t.cs(), known_old_font_families)) {
+               else if (is_known(t.cs(), known_old_font_families) &&
+                        context.new_layout_allowed) {
                        char const * const * where =
                                is_known(t.cs(), known_old_font_families);
                        context.check_layout(os);
-                       string oldsize = context.font.size;
+                       Font const oldFont = context.font;
                        context.font.init();
-                       context.font.size = oldsize;
+                       context.font.size = oldFont.size;
                        context.font.family =
                                known_coded_font_families[where - known_old_font_families];
-                       os << "\n\\family " << context.font.family << "\n"
-                          <<   "\\series " << context.font.series << "\n"
-                          <<   "\\shape "  << context.font.shape  << "\n";
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
 
-               else if (is_known(t.cs(), known_old_font_series)) {
+               else if (is_known(t.cs(), known_old_font_series) &&
+                        context.new_layout_allowed) {
                        char const * const * where =
                                is_known(t.cs(), known_old_font_series);
                        context.check_layout(os);
-                       string oldsize = context.font.size;
+                       Font const oldFont = context.font;
                        context.font.init();
-                       context.font.size = oldsize;
+                       context.font.size = oldFont.size;
                        context.font.series =
                                known_coded_font_series[where - known_old_font_series];
-                       os << "\n\\family " << context.font.family << "\n"
-                          <<   "\\series " << context.font.series << "\n"
-                          <<   "\\shape "  << context.font.shape  << "\n";
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
 
-               else if (is_known(t.cs(), known_old_font_shapes)) {
+               else if (is_known(t.cs(), known_old_font_shapes) &&
+                        context.new_layout_allowed) {
                        char const * const * where =
                                is_known(t.cs(), known_old_font_shapes);
                        context.check_layout(os);
-                       string oldsize = context.font.size;
+                       Font const oldFont = context.font;
                        context.font.init();
-                       context.font.size = oldsize;
+                       context.font.size = oldFont.size;
                        context.font.shape =
                                known_coded_font_shapes[where - known_old_font_shapes];
-                       os << "\n\\family " << context.font.family << "\n"
-                          <<   "\\series " << context.font.series << "\n"
-                          <<   "\\shape "  << context.font.shape  << "\n";
+                       output_font_change(os, oldFont, context.font);
                        eat_whitespace(p, os, context, false);
                }
 
@@ -1739,6 +2069,15 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        }
                }
 
+               else if (t.cs() == "verb") {
+                       context.check_layout(os);
+                       char const delimiter = p.next_token().character();
+                       string const arg = p.getArg(delimiter, delimiter);
+                       ostringstream oss;
+                       oss << "\\verb" << delimiter << arg << delimiter;
+                       handle_ert(os, oss.str(), context);
+               }
+
                else if (t.cs() == "\"") {
                        context.check_layout(os);
                        string const name = p.verbatim_item();
@@ -1757,22 +2096,34 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                else if (t.cs() == "=" && (flags & FLAG_TABBING))
                        handle_ert(os, t.asInput(), context);
 
-               else if (t.cs() == "H" || t.cs() == "c" || t.cs() == "^" || t.cs() == "'"
-                     || t.cs() == "~" || t.cs() == "." || t.cs() == "=") {
+               else if (t.cs() == "H" || t.cs() == "c" || t.cs() == "^"
+                        || t.cs() == "'" || t.cs() == "`"
+                        || t.cs() == "~" || t.cs() == "." || t.cs() == "=") {
                        // we need the trim as the LyX parser chokes on such spaces
+                       // The argument of InsetLatexAccent is parsed as a
+                       // subset of LaTeX, so don't parse anything here,
+                       // but use the raw argument.
+                       // Otherwise we would convert \~{\i} wrongly.
+                       // This will of course not translate \~{\ss} to \~{ß},
+                       // but that does at least compile and does only look
+                       // strange on screen.
                        context.check_layout(os);
-                       os << "\n\\i \\" << t.cs() << "{"
-                          << trim(parse_text(p, FLAG_ITEM, outer, context), " ") << "}\n";
+                       os << "\\i \\" << t.cs() << "{"
+                          << trim(p.verbatim_item(), " ")
+                          << "}\n";
                }
 
                else if (t.cs() == "ss") {
                        context.check_layout(os);
                        os << "ß";
+                       skip_braces(p); // eat {}
                }
 
-               else if (t.cs() == "i" || t.cs() == "j") {
+               else if (t.cs() == "i" || t.cs() == "j" || t.cs() == "l" ||
+                        t.cs() == "L") {
                        context.check_layout(os);
-                       os << "\\" << t.cs() << ' ';
+                       os << "\\i \\" << t.cs() << "{}\n";
+                       skip_braces(p); // eat {}
                }
 
                else if (t.cs() == "\\") {
@@ -1797,26 +2148,47 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                                name += p.get_token().asInput();
                        context.check_layout(os);
                        begin_inset(os, "Include ");
-                       string filename(p.getArg('{', '}'));
-                       string lyxname(lyx::support::ChangeExtension(filename, ".lyx"));
-                       if (tex2lyx(filename, lyxname)) {
-                               os << name << '{' << lyxname << "}\n";
+                       string filename(normalize_filename(p.getArg('{', '}')));
+                       string const path = getMasterFilePath();
+                       // We want to preserve relative / absolute filenames,
+                       // therefore path is only used for testing
+                       // FIXME UNICODE encoding of filename and path may be
+                       // wrong (makeAbsPath expects utf8)
+                       if (t.cs() == "include" &&
+                           !fs::exists(makeAbsPath(filename, path).toFilesystemEncoding())) {
+                               // The file extension is probably missing.
+                               // Now try to find it out.
+                               string const tex_name =
+                                       find_file(filename, path,
+                                                 known_tex_extensions);
+                               if (!tex_name.empty())
+                                       filename = tex_name;
+                       }
+                       // FIXME UNICODE encoding of filename and path may be
+                       // wrong (makeAbsPath expects utf8)
+                       if (fs::exists(makeAbsPath(filename, path).toFilesystemEncoding())) {
+                               string const abstexname =
+                                       makeAbsPath(filename, path).absFilename();
+                               string const abslyxname =
+                                       changeExtension(abstexname, ".lyx");
+                               fix_relative_filename(filename);
+                               string const lyxname =
+                                       changeExtension(filename, ".lyx");
+                               if (t.cs() != "verbatiminput" &&
+                                   tex2lyx(abstexname, FileName(abslyxname))) {
+                                       os << name << '{' << lyxname << "}\n";
+                               } else {
+                                       os << name << '{' << filename << "}\n";
+                               }
                        } else {
+                               cerr << "Warning: Could not find included file '"
+                                    << filename << "'." << endl;
                                os << name << '{' << filename << "}\n";
                        }
                        os << "preview false\n";
                        end_inset(os);
                }
 
-               else if (t.cs() == "fancyhead") {
-                       context.check_layout(os);
-                       ostringstream ss;
-                       ss << "\\fancyhead";
-                       ss << p.getOpt();
-                       ss << '{' << p.verbatim_item() << "}\n";
-                       handle_ert(os, ss.str(), context);
-               }
-
                else if (t.cs() == "bibliographystyle") {
                        // store new bibliographystyle
                        bibliographystyle = p.verbatim_item();
@@ -1841,7 +2213,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        parse_box(p, os, FLAG_ITEM, outer, context, true);
 
                else if (t.cs() == "smallskip" ||
-                        t.cs() == "medskip" ||
+                        t.cs() == "medskip" ||
                         t.cs() == "bigskip" ||
                         t.cs() == "vfill") {
                        context.check_layout(os);
@@ -1851,6 +2223,55 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
                        skip_braces(p);
                }
 
+               else if (is_known(t.cs(), known_spaces)) {
+                       char const * const * where = is_known(t.cs(), known_spaces);
+                       context.check_layout(os);
+                       begin_inset(os, "InsetSpace ");
+                       os << '\\' << known_coded_spaces[where - known_spaces]
+                          << '\n';
+                       // LaTeX swallows whitespace after all spaces except
+                       // "\\,". We have to do that here, too, because LyX
+                       // adds "{}" which would make the spaces significant.
+                       if (t.cs() !=  ",")
+                               eat_whitespace(p, os, context, false);
+                       // LyX adds "{}" after all spaces except "\\ " and
+                       // "\\,", so we have to remove "{}".
+                       // "\\,{}" is equivalent to "\\," in LaTeX, so we
+                       // remove the braces after "\\,", too.
+                       if (t.cs() != " ")
+                               skip_braces(p);
+               }
+
+               else if (t.cs() == "newpage" ||
+                        t.cs() == "clearpage" ||
+                        t.cs() == "cleardoublepage") {
+                       context.check_layout(os);
+                       // FIXME: what about \\pagebreak?
+                       os << "\n\\" << t.cs() << "\n";
+                       skip_braces(p); // eat {}
+               }
+
+               else if (t.cs() == "newcommand" ||
+                        t.cs() == "providecommand" ||
+                        t.cs() == "renewcommand") {
+                       // these could be handled by parse_command(), but
+                       // we need to call add_known_command() here.
+                       string name = t.asInput();
+                       if (p.next_token().asInput() == "*") {
+                               // Starred form. Eat '*'
+                               p.get_token();
+                               name += '*';
+                       }
+                       string const command = p.verbatim_item();
+                       string const opt1 = p.getOpt();
+                       string const opt2 = p.getFullOpt();
+                       add_known_command(command, opt1, !opt2.empty());
+                       string const ert = name + '{' + command + '}' +
+                                          opt1 + opt2 +
+                                          '{' + p.verbatim_item() + '}';
+                       handle_ert(os, ert, context);
+               }
+
                else if (t.cs() == "vspace") {
                        bool starred = false;
                        if (p.next_token().asInput() == "*") {
@@ -1962,5 +2383,7 @@ void parse_text(Parser & p, ostream & os, unsigned flags, bool outer,
        }
 }
 
-
 // }])
+
+
+} // namespace lyx