X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FDepTable.cpp;h=8b15ec745dedf64aaada71fef1a8a5183a2ea6c5;hb=e87ace901a4e52ab8cb7461d3c4910e04320d652;hp=f93752d95039f693ef9cca1e6215c214d8ec40d9;hpb=f3a442440810056c9da6a280b2c985a750e66278;p=lyx.git

diff --git a/src/DepTable.cpp b/src/DepTable.cpp
index f93752d950..8b15ec745d 100644
--- a/src/DepTable.cpp
+++ b/src/DepTable.cpp
@@ -3,7 +3,7 @@
  * This file is part of LyX, the document processor.
  * Licence details can be found in the file COPYING.
  *
- * \author Lars Gullik Bjønnes
+ * \author Lars Gullik Bjønnes
  * \author Jean-Marc Lasgouttes
  * \author Ben Stanley
  *
@@ -15,30 +15,17 @@
 #include "DepTable.h"
 
 #include "support/debug.h"
-#include "support/lyxlib.h"
-#include "support/filetools.h"
+#include "support/FileName.h"
 #include "support/lstrings.h"
 #include "support/lyxtime.h"
 
-#include <sys/stat.h>
-
 #include <fstream>
 
-using std::endl;
-using std::flush;
-using std::getline;
-using std::string;
-using std::ofstream;
-using std::ifstream;
-
+using namespace std;
+using namespace lyx::support;
 
 namespace lyx {
 
-using support::FileName;
-using support::ltrim;
-using support::onlyFilename;
-using support::suffixIs;
-using support::sum;
 
 inline
 bool DepTable::dep_info::changed() const
@@ -52,13 +39,11 @@ void DepTable::insert(FileName const & f, bool upd)
 	if (deplist.find(f) == deplist.end()) {
 		dep_info di;
 		di.crc_prev = 0;
-		if (upd) {
+		if (upd && f.exists()) {
 			LYXERR(Debug::DEPEND, " CRC...");
-			di.crc_cur = sum(f);
+			di.crc_cur = f.checksum();
 			LYXERR(Debug::DEPEND, "done.");
-			struct stat f_info;
-			stat(f.toFilesystemEncoding().c_str(), &f_info);
-			di.mtime_cur = long(f_info.st_mtime);
+			di.mtime_cur = f.lastModified();
 		} else {
 			di.crc_cur = 0;
 			di.mtime_cur = 0;
@@ -77,25 +62,24 @@ void DepTable::update()
 
 	DepList::iterator itr = deplist.begin();
 	while (itr != deplist.end()) {
+		FileName const & fn = itr->first;
 		dep_info &di = itr->second;
 
-		struct stat f_info;
-		if (stat(itr->first.toFilesystemEncoding().c_str(), &f_info) == 0) {
-			if (di.mtime_cur == f_info.st_mtime) {
+		if (fn.exists()) {
+			if (di.mtime_cur == fn.lastModified()) {
 				di.crc_prev = di.crc_cur;
 				LYXERR(Debug::DEPEND, itr->first << " same mtime");
 			} else {
 				di.crc_prev = di.crc_cur;
 				LYXERR(Debug::DEPEND, itr->first << " CRC... ");
-				di.crc_cur = sum(itr->first);
+				di.crc_cur = fn.checksum();
 				LYXERR(Debug::DEPEND, "done");
 			}
 		} else {
 			// file doesn't exist
 			// remove stale files - if it's re-created, it
 			// will be re-inserted by deplog.
-			LYXERR(Debug::DEPEND, itr->first
-				<< " doesn't exist. removing from DepTable.");
+			LYXERR(Debug::DEPEND, fn << " doesn't exist. removing from DepTable.");
 			DepList::iterator doomed = itr++;
 			deplist.erase(doomed);
 			continue;
@@ -119,7 +103,8 @@ bool DepTable::sumchange() const
 	DepList::const_iterator cit = deplist.begin();
 	DepList::const_iterator end = deplist.end();
 	for (; cit != end; ++cit) {
-		if (cit->second.changed()) return true;
+		if (cit->second.changed())
+			return true;
 	}
 	return false;
 }
@@ -141,7 +126,7 @@ bool DepTable::extchanged(string const & ext) const
 	DepList::const_iterator cit = deplist.begin();
 	DepList::const_iterator end = deplist.end();
 	for (; cit != end; ++cit) {
-		if (suffixIs(cit->first.absFilename(), ext)) {
+		if (suffixIs(cit->first.absFileName(), ext)) {
 			if (cit->second.changed())
 				return true;
 		}
@@ -155,9 +140,8 @@ bool DepTable::ext_exist(string const & ext) const
 	DepList::const_iterator cit = deplist.begin();
 	DepList::const_iterator end = deplist.end();
 	for (; cit != end; ++cit) {
-		if (suffixIs(cit->first.absFilename(), ext)) {
+		if (suffixIs(cit->first.absFileName(), ext))
 			return true;
-		}
 	}
 	return false;
 }
@@ -174,7 +158,7 @@ void DepTable::remove_files_with_extension(string const & suf)
 	DepList::iterator cit = deplist.begin();
 	DepList::iterator end = deplist.end();
 	while (cit != end) {
-		if (suffixIs(cit->first.absFilename(), suf)) {
+		if (suffixIs(cit->first.absFileName(), suf)) {
 			// Can't erase the current iterator, but we
 			// can increment and then erase.
 			// Deplist is a map so only the erased
@@ -244,7 +228,7 @@ bool DepTable::read(FileName const & f)
 
 		deplist[FileName(nome)] = di;
 	}
-	return deplist.size();
+	return !deplist.empty();
 }