X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FChktex.C;h=981999f303fb0c5b33275982cabc356d824e91ae;hb=e28331ed63062dea10d0a21b9ec12034b4b17b9a;hp=4f2fc6c6ae8bc36f3ed3526a4e5ddeacfaf398a2;hpb=44cd0fc9a1687cc63911c7f98d978594458e7813;p=lyx.git

diff --git a/src/Chktex.C b/src/Chktex.C
index 4f2fc6c6ae..981999f303 100644
--- a/src/Chktex.C
+++ b/src/Chktex.C
@@ -15,23 +15,26 @@
 
 #include "LaTeX.h" // TeXErrors
 
-#include "support/BoostFormat.h"
+#include "support/convert.h"
+#include "support/docstream.h"
 #include "support/filetools.h"
 #include "support/lstrings.h"
-#include "support/lyxlib.h"
 #include "support/systemcall.h"
 
-#include <fstream>
+#include <boost/format.hpp>
 
-using lyx::support::atoi;
-using lyx::support::ChangeExtension;
-using lyx::support::OnlyFilename;
-using lyx::support::split;
-using lyx::support::Systemcall;
+
+namespace lyx {
+
+using support::changeExtension;
+using support::FileName;
+using support::makeAbsPath;
+using support::onlyFilename;
+using support::split;
+using support::Systemcall;
 
 using std::getline;
 using std::string;
-using std::ifstream;
 
 
 Chktex::Chktex(string const & chktex, string const & f, string const & p)
@@ -43,7 +46,7 @@ Chktex::Chktex(string const & chktex, string const & f, string const & p)
 int Chktex::run(TeXErrors &terr)
 {
 	// run bibtex
-	string log = OnlyFilename(ChangeExtension(file, ".log"));
+	string log = onlyFilename(changeExtension(file, ".log"));
 	string tmp = cmd + " -q -v0 -b0 -x " + file + " -o " + log;
 	Systemcall one;
 	int result = one.startscript(Systemcall::Wait, tmp);
@@ -58,30 +61,34 @@ int Chktex::run(TeXErrors &terr)
 
 int Chktex::scanLogFile(TeXErrors & terr)
 {
-	string token;
 	int retval = 0;
 
-	string const tmp = OnlyFilename(ChangeExtension(file, ".log"));
+	// FIXME: Find out whether onlyFilename() is really needed,
+	// or whether makeAbsPath(onlyFilename()) is a noop here
+	FileName const tmp(makeAbsPath(onlyFilename(changeExtension(file, ".log"))));
 
 #if USE_BOOST_FORMAT
-	boost::format msg(_("ChkTeX warning id # %1$d"));
+	boost::basic_format<char_type> msg(_("ChkTeX warning id # %1$d"));
 #else
-	string const msg(_("ChkTeX warning id # "));
+	docstring const msg(_("ChkTeX warning id # "));
 #endif
-	ifstream ifs(tmp.c_str());
+	docstring token;
+	// FIXME UNICODE
+	// We have no idea what the encoding of the error file is
+	idocfstream ifs(tmp.toFilesystemEncoding().c_str());
 	while (getline(ifs, token)) {
-		string srcfile;
-		string line;
-		string pos;
-		string warno;
-		string warning;
+		docstring srcfile;
+		docstring line;
+		docstring pos;
+		docstring warno;
+		docstring warning;
 		token = split(token, srcfile, ':');
 		token = split(token, line, ':');
 		token = split(token, pos, ':');
 		token = split(token, warno, ':');
 		token = split(token, warning, ':');
 
-		int const lineno = atoi(line);
+		int const lineno = convert<int>(line);
 
 #if USE_BOOST_FORMAT
 		msg % warno;
@@ -95,3 +102,6 @@ int Chktex::scanLogFile(TeXErrors & terr)
 	}
 	return retval;
 }
+
+
+} // namespace lyx