]> git.lyx.org Git - lyx.git/blobdiff - src/DepTable.C
The speed patch: redraw only rows that have changed
[lyx.git] / src / DepTable.C
index e04b2cde5dc86c26d95f791976930934e446fdf8..2718bcacf934fb4d4565f341fa376f4bcb95ebe4 100644 (file)
-/* This file is part of
- * ====================================================== 
- * 
- *           LyX, The Document Processor
- *          Copyright 1995 Matthias Ettrich
- *           Copyright 1995-2001 The LyX Team.
+/**
+ * \file DepTable.C
+ * This file is part of LyX, the document processor.
+ * Licence details can be found in the file COPYING.
  *
- *           This file is Copyright 1996-2001
- *           Lars Gullik Bjønnes
+ * \author Lars Gullik Bjønnes
+ * \author Jean-Marc Lasgouttes
+ * \author Ben Stanley
  *
- * ====================================================== 
+ * Full author contact details are available in file CREDITS.
  */
 
 #include <config.h>
 
-#ifdef __GNUG__
-#pragma implementation
-#endif
-
 #include "DepTable.h"
+
 #include "debug.h"
 
 #include "support/lyxlib.h"
 #include "support/filetools.h"
 #include "support/lstrings.h"
+#include "support/lyxtime.h"
 
-#include <sys/types.h>
 #include <sys/stat.h>
-#include <unistd.h>
 
 #include <fstream>
 
-using std::make_pair;
+#ifndef CXX_GLOBAL_CSTD
+using std::time;
+#endif
+
+using lyx::support::ltrim;
+using lyx::support::MakeAbsPath;
+using lyx::support::OnlyFilename;
+using lyx::support::suffixIs;
+using lyx::support::sum;
+
+using std::endl;
+using std::flush;
+using std::getline;
+using std::string;
 using std::ofstream;
 using std::ifstream;
-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()) {
-               long mtime = 0;
+               dep_info di;
+               di.crc_prev = 0;
                if (upd) {
-                       one = two;
-                       two = lyx::sum(f);
+                       lyxerr[Debug::DEPEND] << " CRC..." << flush;
+                       di.crc_cur = sum(f);
+                       lyxerr[Debug::DEPEND] << "done." << endl;
                        struct stat f_info;
                        stat(fi.c_str(), &f_info);
-                       mtime = f_info.st_mtime;
+                       di.mtime_cur = f_info.st_mtime;
+               } else {
+                       di.crc_cur = 0;
+                       di.mtime_cur = 0;
                }
-               dep_info di;
-               di.first = one;
-               di.second = two;
-               di.mtime = mtime;
-#if 0          
-               deplist[f] = make_pair(one, two);
-#else
                deplist[f] = di;
-#endif
+       } 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 two = one;
-               long mtime = itr->second.mtime;
-               struct stat f_info;
-               stat(itr->first.c_str(), &f_info);
+       lyxerr[Debug::DEPEND] << "Updating DepTable..." << endl;
+       lyx::time_type const start_time = lyx::current_time();
 
-               if (mtime != f_info.st_mtime) {
-                       two = lyx::sum(itr->first);
-                       mtime = f_info.st_mtime;
+       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" << endl;
+                       } else {
+                               di.crc_prev = di.crc_cur;
+                               lyxerr[Debug::DEPEND] << itr->first << " CRC... " << flush;
+                               di.crc_cur = sum(itr->first);
+                               lyxerr[Debug::DEPEND] << "done" << endl;
+                       }
+               } 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 0
-               itr->second = make_pair(one, two);
-#else
-               dep_info di;
-               di.first = one;
-               di.second = two;
-               di.mtime = mtime;
-               
-               itr->second = di;
-#endif
+
                if (lyxerr.debugging(Debug::DEPEND)) {
-                       lyxerr << "Update dep: " << itr->first << " "
-                              << one << " " << two;
-                       if (one != two)
+                       if (di.changed())
                                lyxerr << " +";
                        lyxerr << endl;
                }
+               ++itr;
        }
+       lyx::time_type const time_sec = lyx::current_time() - 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;
 }
@@ -115,11 +132,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;
@@ -128,11 +144,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) {
+       DepList::const_iterator cit = deplist.begin();
+       DepList::const_iterator end = deplist.end();
+       for (; cit != end; ++cit) {
                if (suffixIs(cit->first, ext)) {
-                       if (cit->second.first != cit->second.second)
+                       if (cit->second.changed())
                                return true;
                }
        }
@@ -140,71 +156,102 @@ bool DepTable::extchanged(string const & ext) const
 }
 
 
-bool DepTable::exist(string const & fil) const
+bool DepTable::ext_exist(string const & 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 << " "
-                              << cit->second.mtime << endl;
+                              << cit->second.crc_cur << ' '
+                              << cit->second.mtime_cur << ' '
+                              << cit->first << endl;
                }
-               ofs << cit->first << " "
-                   << cit->second.first << " "
-                   << cit->second.second << " "
-                   << cit->second.mtime << endl;
+               ofs << cit->second.crc_cur << ' '
+                   << cit->second.mtime_cur << ' '
+                   << cit->first << endl;
        }
 }
 
 
-void DepTable::read(string const & f)
+bool DepTable::read(string const & f)
 {
        ifstream ifs(f.c_str());
        string nome;
-       unsigned long one = 0;
-       unsigned long two = 0;
-       unsigned long mtime = 0;
-       
-       while(ifs >> nome >> one >> two >> mtime) {
+       dep_info di;
+       // This doesn't change through the loop.
+       di.crc_prev = 0;
+
+       while (ifs >> di.crc_cur >> di.mtime_cur && getline(ifs, nome)) {
+               nome = ltrim(nome);
                if (lyxerr.debugging(Debug::DEPEND)) {
                        lyxerr << "Read dep: "
-                              << nome << " "
-                              << one << " "
-                              << two << " "
-                              << mtime << endl;
+                              << di.crc_cur << ' '
+                              << di.mtime_cur << ' '
+                              << nome << endl;
                }
-               dep_info di;
-               di.first = one;
-               di.second = two;
-               di.mtime = mtime;
-#if 0          
-               deplist[nome] = make_pair(one, two);
-#else
                deplist[nome] = di;
-#endif
        }
+       return deplist.size();
 }