]> git.lyx.org Git - lyx.git/blobdiff - src/output_latex.cpp
Reintroduce the code related to InsetEnvSeparator
[lyx.git] / src / output_latex.cpp
index 2112b80e7065b202fb1baf3c50d801e32737a903..735b898eb9871d0017202a8a7e24f2637d070061 100644 (file)
@@ -33,6 +33,7 @@
 #include "support/convert.h"
 #include "support/debug.h"
 #include "support/lstrings.h"
+#include "support/textutils.h"
 
 #include <algorithm>
 #include <boost/next_prior.hpp>
@@ -52,6 +53,10 @@ enum OpenEncoding {
        CJK
 };
 
+// FIXME THREAD
+// There could easily be a conflict here, with the export process
+// setting this one way, and a View>Source process (possbily for
+// another Buffer) resetting it.
 static int open_encoding_ = none;
 static int cjk_inherited_ = 0;
 Language const * prev_env_language_ = 0;
@@ -162,8 +167,11 @@ static TeXEnvironmentData prepareEnvironment(Buffer const & buf,
 
        if (style.isEnvironment()) {
                os << "\\begin{" << from_ascii(style.latexname()) << '}';
-               if (style.latexargs().size() != 0)
-                   latexArgInsets(*pit, os, runparams, style.latexargs());
+               if (!style.latexargs().empty()) {
+                       OutputParams rp = runparams;
+                       rp.local_font = &pit->getFirstFontSettings(bparams);
+                       latexArgInsets(paragraphs, pit, os, rp, style.latexargs());
+               }
                if (style.latextype == LATEX_LIST_ENVIRONMENT) {
                        os << '{'
                           << pit->params().labelWidthString()
@@ -195,7 +203,7 @@ static TeXEnvironmentData prepareEnvironment(Buffer const & buf,
 
 
 static void finishEnvironment(otexstream & os, OutputParams const & runparams,
-                              TeXEnvironmentData const & data)
+                             TeXEnvironmentData const & data)
 {
        if (open_encoding_ == CJK && data.cjk_nested) {
                // We need to close the encoding even if it does not change
@@ -205,7 +213,8 @@ static void finishEnvironment(otexstream & os, OutputParams const & runparams,
        }
 
        if (data.style->isEnvironment()) {
-               os << "\\end{" << from_ascii(data.style->latexname()) << "}\n";
+               os << breakln
+                  << "\\end{" << from_ascii(data.style->latexname()) << "}\n";
                prev_env_language_ = data.par_language;
                if (runparams.encoding != data.prev_encoding) {
                        runparams.encoding = data.prev_encoding;
@@ -215,7 +224,7 @@ static void finishEnvironment(otexstream & os, OutputParams const & runparams,
        }
 
        if (data.leftindent_open) {
-               os << "\\end{LyXParagraphLeftIndent}\n";
+               os << breakln << "\\end{LyXParagraphLeftIndent}\n";
                prev_env_language_ = data.par_language;
                if (runparams.encoding != data.prev_encoding) {
                        runparams.encoding = data.prev_encoding;
@@ -223,6 +232,10 @@ static void finishEnvironment(otexstream & os, OutputParams const & runparams,
                                os << setEncoding(data.prev_encoding->iconvName());
                }
        }
+
+       // Check whether we should output a blank line after the environment
+       if (!data.style->nextnoindent)
+               os << '\n';
 }
 
 
@@ -269,21 +282,6 @@ void TeXEnvironment(Buffer const & buf, Text const & text,
                // Or     par->params().depth() > current_depth
                // Or     par->params().leftIndent() != current_left_indent)
 
-               if (par->layout().isParagraph()) {
-                       // FIXME (Lgb): How to handle this?
-                       //&& !suffixIs(os, "\n\n")
-
-                       // (ARRae) There should be at least one '\n' already but we need there to
-                       // be two for Standard paragraphs that are depth-increment'ed to be
-                       // output correctly. However, tables can also be paragraphs so
-                       // don't adjust them.
-
-                       // FIXME (Lgb): Will it ever harm to have one '\n' too
-                       // many? i.e. that we sometimes will have
-                       // three in a row.
-                       os << '\n';
-               }
-
                // FIXME This test should not be necessary.
                // We should perhaps issue an error if it is.
                bool const force_plain_layout = text.inset().forcePlainLayout();
@@ -309,43 +307,26 @@ void TeXEnvironment(Buffer const & buf, Text const & text,
                LYXERR(Debug::LATEX, "TeXEnvironment for paragraph " << par_begin << " done.");
 }
 
-} // namespace anon
-
 
-void latexArgInsets(Paragraph const & par, otexstream & os,
-       OutputParams const & runparams, Layout::LaTeXArgMap latexargs)
+void getArgInsets(otexstream & os, OutputParams const & runparams, Layout::LaTeXArgMap const & latexargs,
+                 map<int, lyx::InsetArgument const *> ilist, vector<string> required, string const & prefix)
 {
-       map<int, InsetArgument const *> ilist;
-       vector<string> required;
-
-       InsetList::const_iterator it = par.insetList().begin();
-       InsetList::const_iterator end = par.insetList().end();
-       for (; it != end; ++it) {
-               if (it->inset->lyxCode() == ARG_CODE) {
-                       InsetArgument const * ins =
-                               static_cast<InsetArgument const *>(it->inset);
-                       if (ins->name().empty())
-                               LYXERR0("Error: Unnamed argument inset!");
-                       else {
-                               unsigned int const nr = convert<unsigned int>(ins->name());
-                               ilist[nr] = ins;
-                               Layout::LaTeXArgMap::const_iterator const lit =
-                                               latexargs.find(nr);
-                               if (lit != latexargs.end()) {
-                                       Layout::latexarg arg = (*lit).second;
-                                       if (!arg.requires.empty()) {
-                                               vector<string> req = getVectorFromString(arg.requires);
-                                               required.insert(required.end(), req.begin(), req.end());
-                                       }
-                               }
-                       }
-               }
-       }
-
        unsigned int const argnr = latexargs.size();
        if (argnr == 0)
                return;
 
+       // Default and preset args are always output, so if they require
+       // other arguments, consider this.
+       Layout::LaTeXArgMap::const_iterator lit = latexargs.begin();
+       Layout::LaTeXArgMap::const_iterator const lend = latexargs.end();
+       for (; lit != lend; ++lit) {
+               Layout::latexarg arg = (*lit).second;
+               if ((!arg.presetarg.empty() || !arg.defaultarg.empty()) && !arg.requires.empty()) {
+                               vector<string> req = getVectorFromString(arg.requires);
+                               required.insert(required.end(), req.begin(), req.end());
+                       }
+       }
+
        for (unsigned int i = 1; i <= argnr; ++i) {
                map<int, InsetArgument const *>::const_iterator lit = ilist.find(i);
                bool inserted = false;
@@ -353,7 +334,7 @@ void latexArgInsets(Paragraph const & par, otexstream & os,
                        InsetArgument const * ins = (*lit).second;
                        if (ins) {
                                Layout::LaTeXArgMap::const_iterator const lait =
-                                               latexargs.find(convert<unsigned int>(ins->name()));
+                                               latexargs.find(ins->name());
                                if (lait != latexargs.end()) {
                                        Layout::latexarg arg = (*lait).second;
                                        docstring ldelim = arg.mandatory ?
@@ -364,7 +345,7 @@ void latexArgInsets(Paragraph const & par, otexstream & os,
                                                ldelim = arg.ldelim;
                                        if (!arg.rdelim.empty())
                                                rdelim = arg.rdelim;
-                                       ins->latexArgument(os, runparams, ldelim, rdelim);
+                                       ins->latexArgument(os, runparams, ldelim, rdelim, arg.presetarg);
                                        inserted = true;
                                }
                        }
@@ -373,16 +354,29 @@ void latexArgInsets(Paragraph const & par, otexstream & os,
                        Layout::LaTeXArgMap::const_iterator lait = latexargs.begin();
                        Layout::LaTeXArgMap::const_iterator const laend = latexargs.end();
                        for (; lait != laend; ++lait) {
-                               if ((*lait).first == i) {
+                               string const name = prefix + convert<string>(i);
+                               if ((*lait).first == name) {
                                        Layout::latexarg arg = (*lait).second;
+                                       docstring preset = arg.presetarg;
+                                       if (!arg.defaultarg.empty()) {
+                                               if (!preset.empty())
+                                                       preset += ",";
+                                               preset += arg.defaultarg;
+                                       }
                                        if (arg.mandatory) {
                                                docstring ldelim = arg.ldelim.empty() ?
                                                                from_ascii("{") : arg.ldelim;
                                                docstring rdelim = arg.rdelim.empty() ?
                                                                from_ascii("}") : arg.rdelim;
-                                               os << ldelim << rdelim;
+                                               os << ldelim << preset << rdelim;
+                                       } else if (!preset.empty()) {
+                                               docstring ldelim = arg.ldelim.empty() ?
+                                                               from_ascii("[") : arg.ldelim;
+                                               docstring rdelim = arg.rdelim.empty() ?
+                                                               from_ascii("]") : arg.rdelim;
+                                               os << ldelim << preset << rdelim;
                                        } else if (find(required.begin(), required.end(),
-                                                  convert<string>((*lait).first)) != required.end()) {
+                                                  (*lait).first) != required.end()) {
                                                docstring ldelim = arg.ldelim.empty() ?
                                                                from_ascii("[") : arg.ldelim;
                                                docstring rdelim = arg.rdelim.empty() ?
@@ -396,6 +390,103 @@ void latexArgInsets(Paragraph const & par, otexstream & os,
        }
 }
 
+
+} // namespace anon
+
+
+void latexArgInsets(Paragraph const & par, otexstream & os,
+       OutputParams const & runparams, Layout::LaTeXArgMap const & latexargs, string const & prefix)
+{
+       map<int, InsetArgument const *> ilist;
+       vector<string> required;
+
+       InsetList::const_iterator it = par.insetList().begin();
+       InsetList::const_iterator end = par.insetList().end();
+       for (; it != end; ++it) {
+               if (it->inset->lyxCode() == ARG_CODE) {
+                       InsetArgument const * ins =
+                               static_cast<InsetArgument const *>(it->inset);
+                       if (ins->name().empty())
+                               LYXERR0("Error: Unnamed argument inset!");
+                       else {
+                               string const name = prefix.empty() ? ins->name() : split(ins->name(), ':');
+                               unsigned int const nr = convert<unsigned int>(name);
+                               ilist[nr] = ins;
+                               Layout::LaTeXArgMap::const_iterator const lit =
+                                               latexargs.find(ins->name());
+                               if (lit != latexargs.end()) {
+                                       Layout::latexarg const & arg = (*lit).second;
+                                       if (!arg.requires.empty()) {
+                                               vector<string> req = getVectorFromString(arg.requires);
+                                               required.insert(required.end(), req.begin(), req.end());
+                                       }
+                               }
+                       }
+               }
+       }
+       getArgInsets(os, runparams, latexargs, ilist, required, prefix);
+}
+
+
+void latexArgInsets(ParagraphList const & pars, ParagraphList::const_iterator pit,
+       otexstream & os, OutputParams const & runparams, Layout::LaTeXArgMap const & latexargs,
+       string const & prefix)
+{
+       map<int, InsetArgument const *> ilist;
+       vector<string> required;
+
+       depth_type const current_depth = pit->params().depth();
+       Layout const current_layout = pit->layout();
+
+       // get the first paragraph in sequence with this layout and depth
+       pit_type offset = 0;
+       while (true) {
+               if (boost::prior(pit, offset) == pars.begin())
+                       break;
+               ParagraphList::const_iterator priorpit = boost::prior(pit, offset + 1);
+               if (priorpit->layout() == current_layout
+                   && priorpit->params().depth() == current_depth)
+                       ++offset;
+               else
+                       break;
+       }
+
+       ParagraphList::const_iterator spit = boost::prior(pit, offset);
+
+       for (; spit != pars.end(); ++spit) {
+               if (spit->layout() != current_layout || spit->params().depth() < current_depth)
+                       break;
+               if (spit->params().depth() > current_depth)
+                       continue;
+               InsetList::const_iterator it = spit->insetList().begin();
+               InsetList::const_iterator end = spit->insetList().end();
+               for (; it != end; ++it) {
+                       if (it->inset->lyxCode() == ARG_CODE) {
+                               InsetArgument const * ins =
+                                       static_cast<InsetArgument const *>(it->inset);
+                               if (ins->name().empty())
+                                       LYXERR0("Error: Unnamed argument inset!");
+                               else {
+                                       string const name = prefix.empty() ? ins->name() : split(ins->name(), ':');
+                                       unsigned int const nr = convert<unsigned int>(name);
+                                       if (ilist.find(nr) == ilist.end())
+                                               ilist[nr] = ins;
+                                       Layout::LaTeXArgMap::const_iterator const lit =
+                                                       latexargs.find(ins->name());
+                                       if (lit != latexargs.end()) {
+                                               Layout::latexarg const & arg = (*lit).second;
+                                               if (!arg.requires.empty()) {
+                                                       vector<string> req = getVectorFromString(arg.requires);
+                                                       required.insert(required.end(), req.begin(), req.end());
+                                               }
+                                       }
+                               }
+                       }
+               }
+       }
+       getArgInsets(os, runparams, latexargs, ilist, required, prefix);
+}
+
 namespace {
 
 // output the proper paragraph start according to latextype.
@@ -406,15 +497,18 @@ void parStartCommand(Paragraph const & par, otexstream & os,
        case LATEX_COMMAND:
                os << '\\' << from_ascii(style.latexname());
 
-               // Separate handling of optional argument inset.
-               if (style.latexargs().size() != 0)
+               // Command arguments
+               if (!style.latexargs().empty())
                        latexArgInsets(par, os, runparams, style.latexargs());
-               else
-                       os << from_ascii(style.latexparam());
+               os << from_ascii(style.latexparam());
                break;
        case LATEX_ITEM_ENVIRONMENT:
        case LATEX_LIST_ENVIRONMENT:
-               os << "\\item ";
+               os << "\\" + style.itemcommand();
+               // Item arguments
+               if (!style.itemargs().empty())
+                       latexArgInsets(par, os, runparams, style.itemargs(), "item:");
+               os << " ";
                break;
        case LATEX_BIB_ENVIRONMENT:
                // ignore this, the inset will write itself
@@ -435,7 +529,8 @@ void TeXOnePar(Buffer const & buf,
               string const & everypar,
               int start_pos, int end_pos)
 {
-       BufferParams const & bparams = buf.params();
+       BufferParams const & bparams = runparams_in.is_child
+               ? buf.masterParams() : buf.params();
        ParagraphList const & paragraphs = text.paragraphs();
        Paragraph const & par = paragraphs.at(pit);
        // FIXME This check should not really be needed.
@@ -466,7 +561,7 @@ void TeXOnePar(Buffer const & buf,
                open_encoding_ = none;
        }
 
-       if (text.inset().getLayout().isPassThru()) {
+       if (text.inset().isPassThru()) {
                Font const outerfont = text.outerFont(pit);
 
                // No newline before first paragraph in this lyxtext
@@ -498,7 +593,7 @@ void TeXOnePar(Buffer const & buf,
                        os << '\n';
                if (!style.parbreak_is_newline) {
                        os << '\n';
-               } else if (nextpar) {
+               } else if (nextpar && !style.isEnvironment()) {
                        Layout const nextstyle = text.inset().forcePlainLayout()
                                ? bparams.documentClass().plainLayout()
                                : nextpar->layout();
@@ -548,10 +643,21 @@ void TeXOnePar(Buffer const & buf,
                getPolyglossiaEnvName(doc_language) : doc_language->babel();
        string const outer_lang = use_polyglossia ?
                getPolyglossiaEnvName(outer_language) : outer_language->babel();
-       string const lang_begin_command = use_polyglossia ?
+       string lang_begin_command = use_polyglossia ?
                "\\begin{$$lang}" : lyxrc.language_command_begin;
-       string const lang_end_command = use_polyglossia ?
+       string lang_end_command = use_polyglossia ?
                "\\end{$$lang}" : lyxrc.language_command_end;
+       // the '%' is necessary to prevent unwanted whitespace
+       string lang_command_termination = "%\n";
+
+       // In some insets (such as Arguments), we cannot use \selectlanguage
+       bool const localswitch = !use_polyglossia
+               && text.inset().getLayout().forcelocalfontswitch();
+       if (localswitch) {
+               lang_begin_command = lyxrc.language_command_local;
+               lang_end_command = "}";
+               lang_command_termination.clear();
+       }
 
        if (par_lang != prev_lang
                // check if we already put language command in TeXEnvironment()
@@ -567,8 +673,7 @@ void TeXOnePar(Buffer const & buf,
                        os << from_ascii(subst(lang_end_command,
                                "$$lang",
                                prev_lang))
-                          // the '%' is necessary to prevent unwanted whitespace
-                          << "%\n";
+                          << lang_command_termination;
                }
 
                // We need to open a new language if we couldn't close the previous
@@ -626,8 +731,7 @@ void TeXOnePar(Buffer const & buf,
                                                os << "["
                                                  << from_ascii(par_language->polyglossiaOpts())
                                                  << "]";
-                                  // the '%' is necessary to prevent unwanted whitespace
-                               os << "%\n";
+                               os << lang_command_termination;
                        }
                }
        }
@@ -649,7 +753,7 @@ void TeXOnePar(Buffer const & buf,
                                par.getFontSettings(bparams, i).language()->encoding();
                        if (encoding->package() != Encoding::CJK
                                && runparams.encoding->package() == Encoding::inputenc
-                               && c < 0x80)
+                               && isASCII(c))
                                continue;
                        if (par.isInset(i))
                                break;
@@ -683,8 +787,7 @@ void TeXOnePar(Buffer const & buf,
                                                lang_begin_command,
                                                "$$lang",
                                                par_lang))
-                                       // the '%' is necessary to prevent unwanted whitespace
-                                       << "%\n";
+                                       << lang_command_termination;
                                }
                                runparams.encoding = encoding;
                        }
@@ -698,7 +801,7 @@ void TeXOnePar(Buffer const & buf,
        bool const useSetSpace = bparams.documentClass().provides("SetSpace");
        if (par.allowParagraphCustomization()) {
                if (par.params().startOfAppendix()) {
-                       os << "\\appendix\n";
+                       os << "\n\\appendix\n";
                }
 
                if (!par.params().spacing().isDefault()
@@ -714,7 +817,6 @@ void TeXOnePar(Buffer const & buf,
        }
 
        parStartCommand(par, os, runparams, style);
-
        Font const outerfont = text.outerFont(pit);
 
        // FIXME UNICODE
@@ -744,6 +846,8 @@ void TeXOnePar(Buffer const & buf,
                os << "\\" << from_ascii(font.latexSize()) << " \\par}";
        } else if (is_command) {
                os << '}';
+               if (!style.postcommandargs().empty())
+                       latexArgInsets(par, os, runparams, style.postcommandargs(), "post:");
                if (runparams.encoding != prev_encoding) {
                        runparams.encoding = prev_encoding;
                        if (!runparams.isFullUnicode())
@@ -756,7 +860,7 @@ void TeXOnePar(Buffer const & buf,
        switch (style.latextype) {
        case LATEX_ITEM_ENVIRONMENT:
        case LATEX_LIST_ENVIRONMENT:
-               if (nextpar && (par.params().depth() < nextpar->params().depth()))
+               if (nextpar && par.params().depth() < nextpar->params().depth())
                        pending_newline = true;
                break;
        case LATEX_ENVIRONMENT: {
@@ -780,7 +884,13 @@ void TeXOnePar(Buffer const & buf,
                        && (runparams.isLastPar || !nextpar->hasSameLayout(par))) {
                        if (pending_newline)
                                os << '\n';
-                       os << from_ascii(par.params().spacing().writeEnvirEnd(useSetSpace));
+
+                       string const endtag =
+                               par.params().spacing().writeEnvirEnd(useSetSpace);
+                       if (prefixIs(endtag, "\\end{"))
+                               os << breakln;
+
+                       os << from_ascii(endtag);
                        pending_newline = true;
                }
        }
@@ -792,7 +902,7 @@ void TeXOnePar(Buffer const & buf,
                // not for ArabTeX
                && (par_language->lang() != "arabic_arabtex"
                    && outer_language->lang() != "arabic_arabtex")
-                    // have we opened and \L or \R environment?
+               // have we opened an \L or \R environment?
                && runparams.local_font != 0
                && runparams.local_font->isRightToLeft() != par_language->rightToLeft()
                // are we about to close the language?
@@ -827,27 +937,31 @@ void TeXOnePar(Buffer const & buf,
                                                lang_begin_command,
                                                "$$lang",
                                                current_lang));
-                                       pending_newline = true;
-                                       unskip_newline = true;
+                                       pending_newline = !localswitch;
+                                       unskip_newline = !localswitch;
                                }
                        } else if (!par_lang.empty()) {
                                os << from_ascii(subst(
                                        lang_end_command,
                                        "$$lang",
                                        par_lang));
-                               pending_newline = true;
-                               unskip_newline = true;
+                               pending_newline = !localswitch;
+                               unskip_newline = !localswitch;
                        }
                }
        }
        if (closing_rtl_ltr_environment)
                os << "}";
 
+       bool const last_was_separator =
+               par.size() > 0 && par.isEnvSeparator(par.size() - 1);
+
        if (pending_newline) {
                if (unskip_newline)
                        // prevent unwanted whitespace
                        os << '%';
-               os << '\n';
+               if (!os.afterParbreak() && !last_was_separator)
+                       os << '\n';
        }
 
        // if this is a CJK-paragraph and the next isn't, close CJK
@@ -911,20 +1025,51 @@ void TeXOnePar(Buffer const & buf,
        // we don't need a newline for the last paragraph!!!
        // Note from JMarc: we will re-add a \n explicitly in
        // TeXEnvironment, because it is needed in this case
-       if (nextpar) {
+       if (nextpar && !os.afterParbreak() && !last_was_separator) {
+               // Make sure to start a new line
+               os << breakln;
                Layout const & next_layout = nextpar->layout();
-               if (style == next_layout
-                   // no blank lines before environments!
-                   || !next_layout.isEnvironment()
-                   // unless there's a depth change
-                   // FIXME What we really want to do here is put every \begin and \end
-                   // tag on a new line (which was not the case with nested environments).
-                   // But in the present state of play, we don't have access to the
-                   // information whether the current TeX row is empty or not.
-                   // For some ideas about how to fix this, see this thread:
-                   // http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg145787.html
-                   || nextpar->params().depth() != par.params().depth()) {
-                       os << '\n';
+               // A newline '\n' is always output before a command,
+               // so avoid doubling it.
+               if (!next_layout.isCommand()) {
+                       // Here we now try to avoid spurious empty lines by
+                       // outputting a paragraph break only if: (case 1) the
+                       // paragraph style allows parbreaks and no \begin, \end
+                       // or \item tags are going to follow (i.e., if the next
+                       // isn't the first or the current isn't the last
+                       // paragraph of an environment or itemize) and the
+                       // depth and alignment of the following paragraph is
+                       // unchanged, or (case 2) the following is a
+                       // non-environment paragraph whose depth is increased
+                       // but whose alignment is unchanged, or (case 3) the
+                       // paragraph is not an environment and the next one is a
+                       // non-itemize-like env at lower depth, or (case 4) the
+                       // paragraph is a command not followed by an environment
+                       // and the alignment of the current and next paragraph
+                       // is unchanged, or (case 5) the current alignment is
+                       // changed and a standard paragraph follows.
+                       DocumentClass const & tclass = bparams.documentClass();
+                       if ((style == next_layout
+                            && !style.parbreak_is_newline
+                            && style.latextype != LATEX_ITEM_ENVIRONMENT
+                            && style.latextype != LATEX_LIST_ENVIRONMENT
+                            && style.align == par.getAlign()
+                            && nextpar->getDepth() == par.getDepth()
+                            && nextpar->getAlign() == par.getAlign())
+                           || (!next_layout.isEnvironment()
+                               && nextpar->getDepth() > par.getDepth()
+                               && nextpar->getAlign() == par.getAlign())
+                           || (!style.isEnvironment()
+                               && next_layout.latextype == LATEX_ENVIRONMENT
+                               && nextpar->getDepth() < par.getDepth())
+                           || (style.isCommand()
+                               && !next_layout.isEnvironment()
+                               && style.align == par.getAlign()
+                               && next_layout.align == nextpar->getAlign())
+                           || (style.align != par.getAlign()
+                               && tclass.isDefaultLayout(next_layout))) {
+                               os << '\n';
+                       }
                }
        }
 
@@ -942,6 +1087,9 @@ void latexParagraphs(Buffer const & buf,
                     OutputParams const & runparams,
                     string const & everypar)
 {
+       LASSERT(runparams.par_begin <= runparams.par_end,
+               { os << "% LaTeX Output Error\n"; return; } );
+
        BufferParams const & bparams = buf.params();
 
        bool const maintext = text.isMainText();
@@ -979,7 +1127,6 @@ void latexParagraphs(Buffer const & buf,
        }
 
        ParagraphList const & paragraphs = text.paragraphs();
-       LASSERT(runparams.par_begin <= runparams.par_end, /**/);
 
        if (runparams.par_begin == runparams.par_end) {
                // The full doc will be exported but it is easier to just rely on
@@ -1097,8 +1244,12 @@ pair<bool, int> switchEncoding(odocstream & os, BufferParams const & bparams,
 {
        Encoding const & oldEnc = *runparams.encoding;
        bool moving_arg = runparams.moving_arg;
-       if (!force && ((bparams.inputenc != "auto" && bparams.inputenc != "default")
-               || moving_arg))
+       // If we switch from/to CJK, we need to switch anyway, despite custom inputenc
+       bool const from_to_cjk = 
+               (oldEnc.package() == Encoding::CJK && newEnc.package() != Encoding::CJK)
+               || (oldEnc.package() != Encoding::CJK && newEnc.package() == Encoding::CJK);
+       if (!force && !from_to_cjk
+           && ((bparams.inputenc != "auto" && bparams.inputenc != "default") || moving_arg))
                return make_pair(false, 0);
 
        // Do nothing if the encoding is unchanged.