X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2Fconverter.C;h=d192e26aca7cd9e06ee7be456422793e2e9dec0d;hb=98c966c64594611e469313314abd1e59524adb4a;hp=d9f0f72cf76732b9c9295912372f3a47d379278b;hpb=fb424bae1b693200ba4a506b2775929d25d77be3;p=lyx.git

diff --git a/src/converter.C b/src/converter.C
index d9f0f72cf7..d192e26aca 100644
--- a/src/converter.C
+++ b/src/converter.C
@@ -1,8 +1,8 @@
 /* This file is part of
- * ====================================================== 
- * 
+ * ======================================================
+ *
  *           LyX, The Document Processor
- *        
+ *
  *           Copyright 1995 Matthias Ettrich
  *           Copyright 1995-2001 The LyX Team.
  *
@@ -18,21 +18,22 @@
 
 #include "converter.h"
 #include "lyxrc.h"
-#include "support/syscall.h"
-#include "support/path.h"
-#include "support/filetools.h"
 #include "buffer.h"
 #include "bufferview_funcs.h"
 #include "LaTeX.h"
 #include "LyXView.h"
-#include "lyx_gui_misc.h"
 #include "lyx_cb.h" // ShowMessage()
-#include "support/lyxfunctional.h"
-#include "frontends/Alert.h"
 #include "gettext.h"
 #include "BufferView.h"
 #include "debug.h"
 
+#include "frontends/Alert.h"
+
+#include "support/filetools.h"
+#include "support/lyxfunctional.h"
+#include "support/path.h"
+#include "support/systemcall.h"
+
 using std::vector;
 using std::queue;
 using std::endl;
@@ -71,7 +72,7 @@ bool Format::isChildFormat() const
 {
 	if (name_.empty())
 		return false;
-        return isdigit(name_[name_.length() - 1]);
+	return isdigit(name_[name_.length() - 1]);
 }
 
 
@@ -115,10 +116,10 @@ void Formats::add(string const & name)
 }
 
 
-void Formats::add(string const & name, string const & extension, 
+void Formats::add(string const & name, string const & extension,
 		  string const & prettyname, string const & shortcut)
 {
-	FormatList::iterator it = 
+	FormatList::iterator it =
 		find_if(formatlist.begin(), formatlist.end(),
 			lyx::compare_memfun(&Format::name, name));
 	if (it == formatlist.end())
@@ -133,7 +134,7 @@ void Formats::add(string const & name, string const & extension,
 
 void Formats::erase(string const & name)
 {
-	FormatList::iterator it = 
+	FormatList::iterator it =
 		find_if(formatlist.begin(), formatlist.end(),
 			lyx::compare_memfun(&Format::name, name));
 	if (it != formatlist.end())
@@ -184,10 +185,10 @@ bool Formats::view(Buffer const * buffer, string const & filename,
 		if (paper_size == "letter")
 			paper_size = "us";
 		command += " " + paper_size;
-		if (buffer->params.orientation 
+		if (buffer->params.orientation
 		    == BufferParams::ORIENTATION_LANDSCAPE)
 			command += 'r';
-        }
+	}
 
 	command += " " + QuoteName(OnlyFilename((filename)));
 
@@ -195,8 +196,8 @@ bool Formats::view(Buffer const * buffer, string const & filename,
 	ShowMessage(buffer, _("Executing command:"), command);
 
 	Path p(OnlyPath(filename));
-	Systemcalls one;
-	int const res = one.startscript(Systemcalls::SystemDontWait, command);
+	Systemcall one;
+	int const res = one.startscript(Systemcall::DontWait, command);
 
 	if (res) {
 		Alert::alert(_("Cannot view file"),
@@ -324,7 +325,7 @@ void Converters::add(string const & from, string const & to,
 		converter.flags = flags;
 	}
 	converter.readFlags();
-	
+
 	if (converter.latex && (latex_command_.empty() || to == "dvi"))
 		latex_command_ = subst(command, token_from, "");
 	// If we have both latex & pdflatex, we set latex_command to latex.
@@ -354,10 +355,10 @@ void Converters::erase(string const & from, string const & to)
 // This method updates the pointers From and To in all the converters.
 // The code is not very efficient, but it doesn't matter as the number
 // of formats and converters is small.
-// Furthermore, this method is called only on startup, or after 
+// Furthermore, this method is called only on startup, or after
 // adding/deleting a format in FormPreferences (the latter calls can be
 // eliminated if the formats in the Formats class are stored using a map or
-// a list (instead of a vector), but this will cause other problems). 
+// a list (instead of a vector), but this will cause other problems).
 void Converters::update(Formats const & formats)
 {
 	ConverterList::iterator it = converterlist_.begin();
@@ -418,7 +419,7 @@ Converters::getReachableTo(string const & target, bool clear_visited)
 		if (i != s || target != "lyx") {
 			result.push_back(&formats.get(i));
 		}
-		
+
 		vector<int>::iterator it = vertices_[i].in_vertices.begin();
 		vector<int>::iterator end = vertices_[i].in_vertices.end();
 		for (; it != end; ++it) {
@@ -522,7 +523,7 @@ Converters::getPath(string const & from, string const & to)
 			found = true;
 			break;
 		}
-		
+
 		vector<int>::const_iterator beg =
 			vertices_[i].out_vertices.begin();
 		vector<int>::const_iterator cit = beg;
@@ -614,8 +615,8 @@ bool Converters::convert(Buffer const * buffer,
 		} else {
 			if (conv.need_aux && !run_latex
 			    && !latex_command_.empty()) {
-				lyxerr[Debug::FILES] 
-					<< "Running " << latex_command_ 
+				lyxerr[Debug::FILES]
+					<< "Running " << latex_command_
 					<< " to update aux file"<<  endl;
 				runLaTeX(buffer, latex_command_);
 			}
@@ -644,9 +645,9 @@ bool Converters::convert(Buffer const * buffer,
 			if (buffer)
 				ShowMessage(buffer, _("Executing command:"), command);
 
-			Systemcalls::Starttype type = (dummy)
-				? Systemcalls::SystemDontWait : Systemcalls::System;
-			Systemcalls one;
+			Systemcall::Starttype type = (dummy)
+				? Systemcall::DontWait : Systemcall::Wait;
+			Systemcall one;
 			int res;
 			if (conv.original_dir && buffer) {
 				Path p(buffer->filePath());
@@ -656,10 +657,11 @@ bool Converters::convert(Buffer const * buffer,
 
 			if (!conv.parselog.empty()) {
 				string const logfile =  infile2 + ".log";
-				string const command2 = conv.parselog +
+				string const script = LibScriptSearch(conv.parselog);
+				string const command2 = script +
 					" < " + QuoteName(infile2 + ".out") +
 					" > " + QuoteName(logfile);
-				one.startscript(Systemcalls::System, command2);
+				one.startscript(Systemcall::Wait, command2);
 				if (!scanLog(buffer, command, logfile))
 					return false;
 			}
@@ -698,11 +700,11 @@ bool Converters::convert(Buffer const * buffer,
 			}
 		}
 		return true;
-	} else 
+	} else
 		return move(outfile, to_file, conv.latex);
 }
 
-// If from = /path/file.ext and to = /path2/file2.ext2 then this method 
+// If from = /path/file.ext and to = /path2/file2.ext2 then this method
 // moves each /path/file*.ext file to /path2/file2*.ext2'
 bool Converters::move(string const & from, string const & to, bool copy)
 {
@@ -722,7 +724,7 @@ bool Converters::move(string const & from, string const & to, bool copy)
 			string from2 = path + *it;
 			string to2 = to_base + it->substr(base.length());
 			to2 = ChangeExtension(to2, to_extension);
-			lyxerr[Debug::FILES] << "moving " << from2 
+			lyxerr[Debug::FILES] << "moving " << from2
 					     << " to " << to2 << endl;
 			bool moved = (copy)
 				? lyx::copy(from2, to2)
@@ -782,11 +784,9 @@ bool Converters::scanLog(Buffer const * buffer, string const & command,
 		return false;
 
 	BufferView * bv = buffer->getUser();
-	bool need_redraw = false;
 	if (bv) {
 		bv->owner()->prohibitInput();
-		// Remove all error insets
-		need_redraw = bv->removeAutoInsets();
+		// all error insets should have been removed by now
 	}
 
 	LaTeX latex("", filename, "");
@@ -796,9 +796,6 @@ bool Converters::scanLog(Buffer const * buffer, string const & command,
 		if ((result & LaTeX::ERRORS)) {
 			// Insert all errors as errors boxes
 			bv->insertErrors(terr);
-			need_redraw = true;
-		}
-		if (need_redraw) {
 			bv->redraw();
 			bv->fitCursor();
 		}
@@ -838,34 +835,24 @@ bool Converters::runLaTeX(Buffer const * buffer, string const & command)
 		return false;
 
 	BufferView * bv = buffer->getUser();
-	string name = buffer->getLatexName();
-	bool need_redraw = false;
 
 	if (bv) {
 		bv->owner()->prohibitInput();
 		bv->owner()->message(_("Running LaTeX..."));
-		// Remove all error insets
-		need_redraw = bv->removeAutoInsets();
+		// all the autoinsets have already been removed
 	}
 
-
-	// do the LaTex run(s)
-	TeXErrors terr;
+	// do the LaTeX run(s)
+	string name = buffer->getLatexName();
 	LaTeX latex(command, name, buffer->filePath());
+	TeXErrors terr;
 	int result = latex.run(terr,
 			       bv ? bv->owner()->getLyXFunc() : 0);
-	
 
 	if (bv) {
 		if ((result & LaTeX::ERRORS)) {
 			// Insert all errors as errors boxes
 			bv->insertErrors(terr);
-			need_redraw = true;
-		}
-
-		// if we removed error insets before we ran LaTeX or if we inserted
-		// error insets after we ran LaTeX this must be run:
-		if (need_redraw) {
 			bv->redraw();
 			bv->fitCursor();
 		}
@@ -897,12 +884,12 @@ bool Converters::runLaTeX(Buffer const * buffer, string const & command)
 
 	if (bv)
 		bv->owner()->allowInput();
- 
-	int const ERROR_MASK = 
+
+	int const ERROR_MASK =
 			LaTeX::NO_LOGFILE |
 			LaTeX::ERRORS |
 			LaTeX::NO_OUTPUT;
-	
+
 	return (result & ERROR_MASK) == 0;
 
 }