Dump hacky overoptimisation in addir -- it doesn't actually get the case

[mmh] / ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 6afe9ad..5f02b17 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2008-08-04  Eric Gillespie  <epg@pretzelnet.org>
+
+       * uip/folder.c: Simplify dodir/addir/addfold.  Dump hacky
+       over-optimization in addir that tried to avoid readdir after all
+       child directories had been read; this was also trying to support
+       symlinks to directories, but would have been failing (because
+       nlink may have gone to 0 with symlinks to directories remaining)
+       had the lstat usage been correct (lstat doesn't fail for normal
+       directories; should have used S_ISLNK).
+
 2008-08-03  Peter Maydell  <pmaydell@chiark.greenend.org.uk>
 
        * test/setup-test: use 'set -e' so we stop on compile failure.