Converter::Converter(string const & f, string const & t,
string const & c, string const & l)
: from_(f), to_(t), command_(c), flags_(l),
- From_(0), To_(0), latex_(false), xml_(false),
+ From_(nullptr), To_(nullptr), latex_(false), docbook_(false),
need_aux_(false), nice_(false), need_auth_(false)
{}
latex_flavor_ = flag_value.empty() ?
"latex" : flag_value;
} else if (flag_name == "xml")
- xml_ = true;
+ docbook_ = true;
else if (flag_name == "needaux") {
need_aux_ = true;
latex_flavor_ = flag_value.empty() ?
if (cit != converterlist_.end())
return &(*cit);
else
- return 0;
+ return nullptr;
}
// a list (instead of a vector), but this will cause other problems).
void Converters::update(Formats const & formats)
{
- ConverterList::iterator it = converterlist_.begin();
- ConverterList::iterator end = converterlist_.end();
- for (; it != end; ++it) {
- it->setFrom(formats.getFormat(it->from()));
- it->setTo(formats.getFormat(it->to()));
+ for (auto & cv : converterlist_) {
+ cv.setFrom(formats.getFormat(cv.from()));
+ cv.setTo(formats.getFormat(cv.to()));
}
}
OutputParams::FLAVOR Converters::getFlavor(Graph::EdgePath const & path,
Buffer const * buffer)
{
- for (Graph::EdgePath::const_iterator cit = path.begin();
- cit != path.end(); ++cit) {
- Converter const & conv = converterlist_[*cit];
+ for (auto const & edge : path) {
+ Converter const & conv = converterlist_[edge];
if (conv.latex() || conv.need_aux()) {
if (conv.latex_flavor() == "latex")
return OutputParams::LATEX;
if (conv.latex_flavor() == "pdflatex")
return OutputParams::PDFLATEX;
}
- if (conv.xml())
- return OutputParams::XML;
+ if (conv.docbook())
+ return OutputParams::DOCBOOK5;
}
return buffer ? buffer->params().getOutputFlavor()
: OutputParams::LATEX;
string Converters::getHyperrefDriver(Graph::EdgePath const & path)
{
- for (Graph::EdgePath::const_iterator cit = path.begin();
- cit != path.end(); ++cit) {
- Converter const & conv = converterlist_[*cit];
+ for (auto const & edge : path) {
+ Converter const & conv = converterlist_[edge];
if (!conv.hyperref_driver().empty())
return conv.hyperref_driver();
}
FileName const & from_file, FileName const & to_file,
FileName const & orig_from,
string const & from_format, string const & to_format,
- ErrorList & errorList, int conversionflags)
+ ErrorList & errorList, int conversionflags, bool includeall)
{
if (from_format == to_format)
return move(from_format, from_file, to_file, false) ?
// buffer is only invalid for importing, and then runparams is not
// used anyway.
- OutputParams runparams(buffer ? &buffer->params().encoding() : 0);
+ OutputParams runparams(buffer ? &buffer->params().encoding() : nullptr);
runparams.flavor = getFlavor(edgepath, buffer);
if (buffer) {
runparams.bibtex_command = bp.bibtexCommand();
runparams.index_command = (bp.index_command == "default") ?
string() : bp.index_command;
- runparams.document_language = bp.language->babel();
- runparams.xindy_language = bp.language->xindy();
+ runparams.document_language = bp.language->lang();
// Some macros rely on font encoding
runparams.main_fontenc = bp.main_font_encoding();
runparams.only_childbibs = !bp.useBiblatex()
&& !bp.useBibtopic()
&& bp.multibib == "child";
+ runparams.includeall = includeall;
}
// Some converters (e.g. lilypond) can only output files to the
string to_base = changeExtension(to_file.absFileName(), "");
FileName infile;
FileName outfile = from_file;
- for (Graph::EdgePath::const_iterator cit = edgepath.begin();
- cit != edgepath.end(); ++cit) {
- Converter const & conv = converterlist_[*cit];
+ for (auto const & edge : edgepath) {
+ Converter const & conv = converterlist_[edge];
bool dummy = conv.To()->dummy() && conv.to() != "program";
if (!dummy) {
LYXERR(Debug::FILES, "Converting from "
if (conv.latex()) {
// We are not importing, we have a buffer
- LATTEST(buffer);
+ LASSERT(buffer, return FAILURE);
run_latex = true;
string command = conv.command();
command = subst(command, token_from, "");
// FIXME KILLED
// Check changed return value here.
RetVal const retval = runLaTeX(*buffer, command, runparams, errorList);
- if (retval != SUCCESS)
- return retval;
+ if (retval != SUCCESS)
+ return retval;
} else {
if (conv.need_aux() && !run_latex) {
// We are not importing, we have a buffer
- LATTEST(buffer);
+ LASSERT(buffer, return FAILURE);
string command;
switch (runparams.flavor) {
case OutputParams::DVILUATEX:
" > " + quoteName(logfile);
res = one.startscript(starttype,
to_filesystem8bit(from_utf8(command2)),
- buffer->filePath(),
- buffer->layoutPos());
+ buffer ? buffer->filePath() : string(),
+ buffer ? buffer->layoutPos() : string());
if (res == Systemcall::KILLED) {
frontend::Alert::warning(
_("Converter killed"),
from_utf8(command)));
return KILLED;
}
- if (!scanLog(*buffer, command, makeAbsPath(logfile, path), errorList))
+ if (buffer && !scanLog(*buffer, command, makeAbsPath(logfile, path), errorList))
return FAILURE;
}
}
string const to_extension = getExtension(to.absFileName());
support::FileNameList const files = FileName(path).dirList(getExtension(from.absFileName()));
- for (support::FileNameList::const_iterator it = files.begin();
- it != files.end(); ++it) {
- string const from2 = it->absFileName();
+ for (auto const & f : files) {
+ string const from2 = f.absFileName();
string const file2 = onlyFileName(from2);
if (prefixIs(file2, base)) {
string const to2 = changeExtension(
Mover const & mover = getMover(fmt);
bool const moved = copy
- ? mover.copy(*it, FileName(to2))
- : mover.rename(*it, FileName(to2));
+ ? mover.copy(f, FileName(to2))
+ : mover.rename(f, FileName(to2));
if (!moved && no_errors) {
Alert::error(_("Cannot convert file"),
bformat(copy ?
bool Converters::formatIsUsed(string const & format)
{
- ConverterList::const_iterator cit = converterlist_.begin();
- ConverterList::const_iterator end = converterlist_.end();
- for (; cit != end; ++cit) {
- if (cit->from() == format || cit->to() == format)
+ for (auto const & cvt : converterlist_) {
+ if (cvt.from() == format || cvt.to() == format)
return true;
}
return false;
bool Converters::scanLog(Buffer const & buffer, string const & /*command*/,
FileName const & filename, ErrorList & errorList)
{
- OutputParams runparams(0);
+ OutputParams runparams(nullptr);
runparams.flavor = OutputParams::LATEX;
LaTeX latex("", runparams, filename);
TeXErrors terr;
string const name = buffer.latexName();
LaTeX latex(command, runparams, FileName(makeAbsPath(name)),
buffer.filePath(), buffer.layoutPos(),
- buffer.isClone(), buffer.lastPreviewError());
+ buffer.isClone(), buffer.freshStartRequired());
TeXErrors terr;
// The connection closes itself at the end of the scope when latex is
// destroyed. One cannot close (and destroy) buffer while the converter is
});
int const result = latex.run(terr);
- if (result == Systemcall::KILLED) {
+ if (result == Systemcall::KILLED || result == Systemcall::TIMEOUT) {
Alert::error(_("Export canceled"),
_("The export process was terminated by the user."));
return KILLED;
if (result & LaTeX::ERRORS)
buffer.bufferErrors(terr, errorList);
- if ((result & LaTeX::UNDEF_CIT) || (result & LaTeX::UNDEF_REF)) {
+ if ((result & LaTeX::UNDEF_CIT) || (result & LaTeX::UNDEF_UNKNOWN_REF)) {
buffer.bufferRefs(terr, errorList);
if (errorList.empty())
errorList.push_back(ErrorItem(_("Undefined reference"),
- _("Undefined reference or citation was found during the build, please check the Log."),
+ _("Undefined references or citations were found during the build.\n"
+ "Please check the warnings in the LaTeX log (Document > LaTeX Log)."),
&buffer));
}
LaTeX::NO_LOGFILE |
LaTeX::ERRORS |
LaTeX::UNDEF_CIT |
- LaTeX::UNDEF_REF |
+ LaTeX::UNDEF_UNKNOWN_REF |
LaTeX::NO_OUTPUT;
return (result & ERROR_MASK) == 0 ? SUCCESS : FAILURE;
// each of the converters knows how to convert one format to another
// so, for each of them, we create an arrow on the graph, going from
// the one to the other
- ConverterList::iterator it = converterlist_.begin();
- ConverterList::iterator const end = converterlist_.end();
- for (; it != end ; ++it) {
- int const from = theFormats().getNumber(it->from());
- int const to = theFormats().getNumber(it->to());
+ for (auto const & cvt : converterlist_) {
+ int const from = theFormats().getNumber(cvt.from());
+ int const to = theFormats().getNumber(cvt.to());
LASSERT(from >= 0, continue);
LASSERT(to >= 0, continue);
G_.addEdge(from, to);
{
set<int> excluded_numbers;
- set<string>::const_iterator sit = excludes.begin();
- set<string>::const_iterator const end = excludes.end();
- for (; sit != end; ++sit)
- excluded_numbers.insert(theFormats().getNumber(*sit));
+ for (auto const & ex : excludes)
+ excluded_numbers.insert(theFormats().getNumber(ex));
vector<int> const & reachables =
G_.getReachable(theFormats().getNumber(from),