X-Git-Url: https://git.lyx.org/gitweb/?a=blobdiff_plain;f=src%2FDepTable.C;h=59ddbd77bfeeacc6f4ffdcc5b37edb9357d7b3d2;hb=34d28f9daae536db06d6cf1a9dcf62dc0f5aa432;hp=0bc13ef436c0ef739596e4488b13f2006acbd0a9;hpb=83acbbd5237373926c629855379e1df9a04209b2;p=lyx.git diff --git a/src/DepTable.C b/src/DepTable.C index 0bc13ef436..59ddbd77bf 100644 --- a/src/DepTable.C +++ b/src/DepTable.C @@ -7,6 +7,7 @@ * * This file is Copyright 1996-2001 * Lars Gullik Bjønnes + * Ben Stanley * * ====================================================== */ @@ -18,57 +19,107 @@ #endif #include "DepTable.h" +#include "debug.h" + #include "support/lyxlib.h" #include "support/filetools.h" +#include "support/lstrings.h" + +#include +#include +#include + #include using std::make_pair; using std::ofstream; using std::ifstream; +using std::flush; using std::endl; -void DepTable::insert(string const & fi, - bool upd, - unsigned long one, - unsigned long two) + +inline +bool DepTable::dep_info::changed() const +{ + return crc_prev != crc_cur && crc_cur != 0; +} + + +void DepTable::insert(string const & fi, bool upd) { // not quite sure if this is the correct place for MakeAbsPath - string f = MakeAbsPath(fi); + string const f = MakeAbsPath(fi); if (deplist.find(f) == deplist.end()) { + dep_info di; + di.crc_prev = 0; if (upd) { - one = two; - two = lyx::sum(f); + lyxerr[Debug::DEPEND] << " CRC..." << flush; + di.crc_cur = lyx::sum(f); + lyxerr[Debug::DEPEND] << "done." << endl; + struct stat f_info; + stat(fi.c_str(), &f_info); + di.mtime_cur = f_info.st_mtime; + } else { + di.crc_cur = 0; + di.mtime_cur = 0; } - deplist[f] = make_pair(one, two); + deplist[f] = di; + } else { + lyxerr[Debug::DEPEND] << " Already in DepTable" << endl; } } void DepTable::update() { - for (DepList::iterator itr = deplist.begin(); - itr != deplist.end(); - ++itr) { - unsigned long const one = (*itr).second.second; - unsigned long const two = lyx::sum((*itr).first); - (*itr).second = make_pair(one, two); + lyxerr[Debug::DEPEND] << "Updating DepTable..." << endl; + time_t start_time = time(0); + + DepList::iterator itr = deplist.begin(); + while (itr != deplist.end()) { + dep_info &di = itr->second; + + struct stat f_info; + if (stat(itr->first.c_str(), &f_info) == 0) { + if (di.mtime_cur == f_info.st_mtime) { + 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 = lyx::sum(itr->first); + 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." << endl; + DepList::iterator doomed = itr++; + deplist.erase(doomed); + continue; + } + if (lyxerr.debugging(Debug::DEPEND)) { - lyxerr << "Update dep: " << (*itr).first << " " - << one << " " << two; - if (one != two) + if (di.changed()) lyxerr << " +"; lyxerr << endl; } + ++itr; } + time_t time_sec = time(0) - start_time; + lyxerr[Debug::DEPEND] << "Finished updating DepTable (" + << time_sec << " sec)." << endl; } bool DepTable::sumchange() const { - for (DepList::const_iterator cit = deplist.begin(); - cit != deplist.end(); - ++cit) { - if ((*cit).second.first != (*cit).second.second) return true; + DepList::const_iterator cit = deplist.begin(); + DepList::const_iterator end = deplist.end(); + for (; cit != end; ++cit) { + if (cit->second.changed()) return true; } return false; } @@ -77,11 +128,10 @@ bool DepTable::sumchange() const bool DepTable::haschanged(string const & f) const { // not quite sure if this is the correct place for MakeAbsPath - string fil = MakeAbsPath(f); + string const fil = MakeAbsPath(f); DepList::const_iterator cit = deplist.find(fil); if (cit != deplist.end()) { - if ((*cit).second.first != (*cit).second.second - && (*cit).second.second != 0) + if (cit->second.changed()) return true; } return false; @@ -90,11 +140,11 @@ bool DepTable::haschanged(string const & f) const bool DepTable::extchanged(string const & ext) const { - for (DepList::const_iterator cit = deplist.begin(); - cit != deplist.end(); - ++cit) { - if (suffixIs((*cit).first, ext)) { - if ((*cit).second.first != (*cit).second.second) + DepList::const_iterator cit = deplist.begin(); + DepList::const_iterator end = deplist.end(); + for (; cit != end; ++cit) { + if (suffixIs(cit->first, ext)) { + if (cit->second.changed()) return true; } } @@ -102,41 +152,81 @@ bool DepTable::extchanged(string const & ext) const } -bool DepTable::exist(string const & fil) const +bool DepTable::ext_exist(const string& ext ) const { - DepList::const_iterator cit = deplist.find(fil); - if (cit != deplist.end()) return true; + DepList::const_iterator cit = deplist.begin(); + DepList::const_iterator end = deplist.end(); + for (; cit != end; ++cit ) { + if (suffixIs(cit->first, ext)) { + return true; + } + } return false; } +bool DepTable::exist(string const & fil) const +{ + return deplist.find(fil) != deplist.end(); +} + + void DepTable::remove_files_with_extension(string const & suf) { - DepList tmp; - // we want const_iterator (Lgb) - for (DepList::iterator cit = deplist.begin(); - cit != deplist.end(); ++cit) { - if (!suffixIs((*cit).first, suf)) - tmp[(*cit).first] = (*cit).second; + DepList::iterator cit = deplist.begin(); + DepList::iterator end = deplist.end(); + while (cit != end) { + if (suffixIs(cit->first, suf)) { + // Can't erase the current iterator, but we + // can increment and then erase. + // Deplist is a map so only the erased + // iterator is invalidated. + DepList::iterator doomed = cit++; + deplist.erase(doomed); + continue; + } + ++cit; + } +} + + +void DepTable::remove_file(string const & filename) +{ + DepList::iterator cit = deplist.begin(); + DepList::iterator end = deplist.end(); + while (cit != end) { + if (OnlyFilename(cit->first) == filename) { + // Can't erase the current iterator, but we + // can increment and then erase. + // deplist is a map so only the erased + // iterator is invalidated. + DepList::iterator doomed = cit++; + deplist.erase(doomed); + continue; + } + ++cit; } - deplist.swap(tmp); } void DepTable::write(string const & f) const { ofstream ofs(f.c_str()); - for (DepList::const_iterator cit = deplist.begin(); - cit != deplist.end(); ++cit) { + DepList::const_iterator cit = deplist.begin(); + DepList::const_iterator end = deplist.end(); + for (; cit != end; ++cit) { if (lyxerr.debugging(Debug::DEPEND)) { + // Store the second (most recently calculated) + // CRC value. + // The older one is effectively set to 0 upon re-load. lyxerr << "Write dep: " - << (*cit).first << " " - << (*cit).second.first << " " - << (*cit).second.second << endl; + << cit->first << " " + << cit->second.crc_cur << " " + << cit->second.mtime_cur << endl; } - ofs << (*cit).first << " " - << (*cit).second.first << " " - << (*cit).second.second << endl; + ofs << cit->first << " " + << cit->second.crc_cur << " " + << cit->second.mtime_cur << endl; } } @@ -145,15 +235,18 @@ void DepTable::read(string const & f) { ifstream ifs(f.c_str()); string nome; - unsigned long one = 0; - unsigned long two = 0; - while(ifs >> nome >> one >> two) { + dep_info di; + // This doesn't change through the loop. + di.crc_prev = 0; + + while (ifs >> nome >> di.crc_cur >> di.mtime_cur) { if (lyxerr.debugging(Debug::DEPEND)) { lyxerr << "Read dep: " << nome << " " - << one << " " - << two << endl; + << di.crc_cur << " " + << di.mtime_cur << endl; } - deplist[nome] = make_pair(one, two); + deplist[nome] = di; } } +