]> git.lyx.org Git - features.git/commitdiff
First step toward fixing bug http://bugzilla.lyx.org/show_bug.cgi?id=5252
authorAbdelrazak Younes <younes@lyx.org>
Sat, 27 Sep 2008 09:15:55 +0000 (09:15 +0000)
committerAbdelrazak Younes <younes@lyx.org>
Sat, 27 Sep 2008 09:15:55 +0000 (09:15 +0000)
- Move lyx::sum() to FileName.cpp
- Get rid of the third slow implementation of sum() as istreambuf_iterator is supported since at least gcc 3.2.

git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@26588 a592a061-630c-0410-9148-cb99ea01b6c8

development/scons/scons_manifest.py
src/support/FileName.cpp
src/support/Makefile.am
src/support/lyxsum.cpp [deleted file]

index ed1088340cdbc8de8aea9cee05afb2c8190a3a09..8c92a83224ab73f9b20083c60e44d183a11a5973 100644 (file)
@@ -341,7 +341,6 @@ src_support_files = Split('''
     kill.cpp
     lassert.cpp
     lstrings.cpp
-    lyxsum.cpp
     lyxtime.cpp
     os.cpp
     qstring_helpers.cpp
index d87ab63f4bd36876fd3dcd18d8556593a82b0f37..43a6f1eda778fc4ef253280f16f31103742b9a3d 100644 (file)
@@ -16,6 +16,7 @@
 #include "support/convert.h"
 #include "support/debug.h"
 #include "support/filetools.h"
+#include "support/lassert.h"
 #include "support/lstrings.h"
 #include "support/qstring_helpers.h"
 #include "support/os.h"
 #include <QTemporaryFile>
 #include <QTime>
 
-#include "support/lassert.h"
+#include <boost/crc.hpp>
 #include <boost/scoped_array.hpp>
 
+#include <algorithm>
+#include <iterator>
+#include <fstream>
+#include <iomanip>
 #include <map>
 #include <sstream>
-#include <fstream>
-#include <algorithm>
 
 #ifdef HAVE_SYS_TYPES_H
 # include <sys/types.h>
@@ -75,11 +78,85 @@ extern "C" int mkstemp(char *);
 # endif
 #endif
 
+// Various implementations of sum(), depending on what methods
+// are available. Order is faster to slowest.
+#if defined(HAVE_MMAP) && defined(HAVE_MUNMAP)
+#define SUM_WITH_MMAP
+#ifdef HAVE_SYS_TYPES_H
+# include <sys/types.h>
+#endif
+#ifdef HAVE_SYS_STAT_H
+# include <sys/stat.h>
+#endif
+#include <fcntl.h>
+#ifdef HAVE_UNISTD_H
+# include <unistd.h>
+#endif
+#include <sys/mman.h>
+#endif // SUM_WITH_MMAP
+
 using namespace std;
 
+// OK, this is ugly, but it is the only workaround I found to compile
+// with gcc (any version) on a system which uses a non-GNU toolchain.
+// The problem is that gcc uses a weak symbol for a particular
+// instantiation and that the system linker usually does not
+// understand those weak symbols (seen on HP-UX, tru64, AIX and
+// others). Thus we force an explicit instanciation of this particular
+// template (JMarc)
+template struct boost::detail::crc_table_t<32, 0x04C11DB7, true>;
+
 namespace lyx {
 namespace support {
 
+static unsigned long sum(char const * file)
+{
+#ifdef SUM_WITH_MMAP
+       //LYXERR(Debug::FILES, "lyx::sum() using mmap (lightning fast)");
+
+       int fd = open(file, O_RDONLY);
+       if (!fd)
+               return 0;
+
+       struct stat info;
+       fstat(fd, &info);
+
+       void * mm = mmap(0, info.st_size, PROT_READ,
+                        MAP_PRIVATE, fd, 0);
+       // Some platforms have the wrong type for MAP_FAILED (compaq cxx).
+       if (mm == reinterpret_cast<void*>(MAP_FAILED)) {
+               close(fd);
+               return 0;
+       }
+
+       char * beg = static_cast<char*>(mm);
+       char * end = beg + info.st_size;
+
+       boost::crc_32_type crc;
+       crc.process_block(beg, end);
+       unsigned long result = crc.checksum();
+
+       munmap(mm, info.st_size);
+       close(fd);
+
+       return result;
+
+#else // no SUM_WITH_MMAP
+
+       //LYXERR(Debug::FILES, "lyx::sum() using istreambuf_iterator (fast)");
+       ifstream ifs(file, ios_base::in | ios_base::binary);
+       if (!ifs)
+               return 0;
+
+       istreambuf_iterator<char> beg(ifs);
+       istreambuf_iterator<char> end;
+       boost::crc_32_type crc;
+       crc = for_each(beg, end, crc);
+       return crc.checksum();
+
+#endif // SUM_WITH_MMAP
+}
+
 
 /////////////////////////////////////////////////////////////////////
 //
@@ -431,8 +508,6 @@ bool FileName::chdir() const
 }
 
 
-extern unsigned long sum(char const * file);
-
 unsigned long FileName::checksum() const
 {
        if (!exists()) {
index 1c1f8a038d2216d6395cb2e6fea9a16588fad053..1b5f7086e0b643314236f31d9e0381bc851a9fef 100644 (file)
@@ -72,7 +72,6 @@ liblyxsupport_la_SOURCES = \
        lyxlib.h \
        lyxtime.cpp \
        lyxtime.h \
-       lyxsum.cpp \
        Messages.cpp \
        Messages.h \
        os.cpp \
diff --git a/src/support/lyxsum.cpp b/src/support/lyxsum.cpp
deleted file mode 100644 (file)
index 0f15491..0000000
+++ /dev/null
@@ -1,133 +0,0 @@
-/**
- * \file lyxsum.cpp
- * This file is part of LyX, the document processor.
- * Licence details can be found in the file COPYING.
- *
- * \author Lars Gullik Bjønnes
- *
- * Full author contact details are available in file CREDITS.
- */
-
-#include <config.h>
-
-#include "support/debug.h"
-
-#include <boost/crc.hpp>
-
-#include <algorithm>
-#include <iomanip>
-
-using namespace std;
-
-// OK, this is ugly, but it is the only workaround I found to compile
-// with gcc (any version) on a system which uses a non-GNU toolchain.
-// The problem is that gcc uses a weak symbol for a particular
-// instantiation and that the system linker usually does not
-// understand those weak symbols (seen on HP-UX, tru64, AIX and
-// others). Thus we force an explicit instanciation of this particular
-// template (JMarc)
-template struct boost::detail::crc_table_t<32, 0x04C11DB7, true>;
-
-// Various implementations of lyx::sum(), depending on what methods
-// are available. Order is faster to slowest.
-#if defined(HAVE_MMAP) && defined(HAVE_MUNMAP)
-
-#ifdef HAVE_SYS_TYPES_H
-# include <sys/types.h>
-#endif
-#ifdef HAVE_SYS_STAT_H
-# include <sys/stat.h>
-#endif
-#include <fcntl.h>
-#ifdef HAVE_UNISTD_H
-# include <unistd.h>
-#endif
-
-#include <sys/mman.h>
-
-
-namespace lyx {
-namespace support {
-
-unsigned long sum(char const * file)
-{
-       //LYXERR(Debug::FILES, "lyx::sum() using mmap (lightning fast)");
-
-       int fd = open(file, O_RDONLY);
-       if (!fd)
-               return 0;
-
-       struct stat info;
-       fstat(fd, &info);
-
-       void * mm = mmap(0, info.st_size, PROT_READ,
-                        MAP_PRIVATE, fd, 0);
-       // Some platforms have the wrong type for MAP_FAILED (compaq cxx).
-       if (mm == reinterpret_cast<void*>(MAP_FAILED)) {
-               close(fd);
-               return 0;
-       }
-
-       char * beg = static_cast<char*>(mm);
-       char * end = beg + info.st_size;
-
-       boost::crc_32_type crc;
-       crc.process_block(beg, end);
-       unsigned long result = crc.checksum();
-
-       munmap(mm, info.st_size);
-       close(fd);
-
-       return result;
-}
-
-} // namespace support
-} // namespace lyx
-
-#else // No mmap
-
-#include <fstream>
-#include <iterator>
-
-namespace {
-
-template<typename InputIterator>
-inline
-unsigned long do_crc(InputIterator first, InputIterator last)
-{
-       boost::crc_32_type crc;
-       crc = for_each(first, last, crc);
-       return crc.checksum();
-}
-
-} // namespace anon
-
-
-namespace lyx {
-namespace support {
-
-
-unsigned long sum(char const * file)
-{
-       ifstream ifs(file, ios_base::in | ios_base::binary);
-       if (!ifs)
-               return 0;
-
-#if HAVE_DECL_ISTREAMBUF_ITERATOR
-       //LYXERR(Debug::FILES, "lyx::sum() using istreambuf_iterator (fast)");
-       istreambuf_iterator<char> beg(ifs);
-       istreambuf_iterator<char> end;
-#else
-       //LYXERR(Debug::FILES, "lyx::sum() using istream_iterator (slow as a snail)");
-       ifs.unsetf(ios::skipws);
-       istream_iterator<char> beg(ifs);
-       istream_iterator<char> end;
-#endif
-
-       return do_crc(beg,end);
-}
-
-} // namespace support
-} // namespace lyx
-
-#endif // mmap