Use only ctime to compare newness (instead of both ctime and mtime)
authorStelian Pop <stelian@popies.net>
Thu, 10 Jun 2010 12:17:35 +0000 (12:17 +0000)
committerStelian Pop <stelian@popies.net>
Thu, 10 Jun 2010 12:17:35 +0000 (12:17 +0000)
CHANGES
dump/traverse.c

diff --git a/CHANGES b/CHANGES
index 9caf7d8e49c847f90b152d14c99cdabd9f2c6d01..bcbb3b5ca0fb0254cef87a6ff51ae3f900170a88 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,4 +1,4 @@
-$Id: CHANGES,v 1.312 2010/06/10 12:01:30 stelian Exp $
+$Id: CHANGES,v 1.313 2010/06/10 12:17:35 stelian Exp $
 
 Changes between versions 0.4b42 and 0.4b43 (released ?????????????)
 ===================================================================
@@ -45,7 +45,14 @@ Changes between versions 0.4b42 and 0.4b43 (released ?????????????)
 
 10.    Improve level 1 dump speed by rearanging the "mapdirs" code.
        Thanks to Andreas Kies <andikies@t-online.de> for finding the
-       bottleneck and sending a patch.
+       bottleneck and sending a patch (Sourceforge bug #2998119).
+
+11.    Use only ctime in order to compare newness instead of both
+       ctime and mtime. This should both speed up dumping time and
+       correct some side-effect bugs (like dumping files with an mtime
+       in the future in every incremental backup). Thanks to
+       Kieran Clancy <codebeard@users.sourceforge.net> for reporting the
+       bug (Sourceforge bug #2999207).
 
 Changes between versions 0.4b41 and 0.4b42 (released June 18, 2009)
 ===================================================================
index 0a394aea1c304c3ab4ee66125639fb87ee2341b2..6ed1d287f3d572cbb7ec7f08cc4538ba9f36bc9e 100644 (file)
@@ -37,7 +37,7 @@
 
 #ifndef lint
 static const char rcsid[] =
-       "$Id: traverse.c,v 1.70 2010/06/10 12:01:30 stelian Exp $";
+       "$Id: traverse.c,v 1.71 2010/06/10 12:17:35 stelian Exp $";
 #endif /* not lint */
 
 #include <config.h>
@@ -240,7 +240,7 @@ blockest(struct dinode const *dp)
 #define MSINCE(dp, t) \
        ((dp)->di_mtime >= (t))
 #define        CHANGEDSINCE(dp, t) \
-       (CSINCE(dp, t) || MSINCE(dp, t))
+       CSINCE(dp, t)
 
 /* The NODUMP_FLAG macro tests if a file has the nodump flag. */
 #ifdef UF_NODUMP