Yikes! Replace a + with a *. No wonder the file was small enough
[sgt/agedu] / TODO
diff --git a/TODO b/TODO
index 6046cec..f8c723d 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,15 +1,6 @@
 TODO list for agedu
 ===================
 
- - stop trying to calculate an upper bound on the index file size.
-   Instead, just mmap it at initial size + delta, and periodically
-   re-mmap it during index building if it grows too big. If we run
-   out of address space, we'll hear about it eventually; and
-   computing upper bounds given the new optimised index tends to be
-   a factor of five out, which is bad because it'll lead to running
-   out of theoretical address space and erroneously reporting
-   failure long before we run out of it for real.
-
  - we could still be using more of the information coming from
    autoconf. Our config.h is defining a whole bunch of HAVE_FOOs for
    particular functions (e.g. HAVE_INET_NTOA, HAVE_MEMCHR,
@@ -63,6 +54,11 @@ TODO list for agedu
       Unclear what the right UI would be on Windows, too;
       command-line exactly as now might be considered just a
       _little_ unfriendly. Or perhaps not.
+       * Disk scan procedure: the FindFirstFile / FindNextFile
+        functions to scan a directory automatically return the file
+        times along with the filenames, so there's no need to stat
+        them later. Would want to fiddle the shape of the
+        abstraction layer to reflect this.
     + Alternatively, a much easier approach would be to write a
       Windows version of just the --scan-dump mode, which does a
       filesystem scan via the Windows API and generates a valid
@@ -81,5 +77,4 @@ TODO list for agedu
    how to make the indexers pluggable at both index-generation time
    and query time.
     * however, now we have the cut-down version of the continuous
-      index, it might be the case that the space gain is no longer
-      worthwhile.
+      index, the space saving is less compelling.