.

wcfs/client/wcfs.cpp

@@ -431,17 +431,17 @@ pair<Mapping, error> _FileH::mmap(int64_t blk_start, int64_t blk_len, VMA *vma)
     mmap->mem_stop  = mem_stop;
     mmap->vma       = vma;
 
-    for (auto _ : f._pinned) {  // XXX keep f._pinned ↑blk and use binary search?
+    for (auto _ : f._pinned) {  // TODO keep f._pinned ↑blk and use binary search
         int64_t    blk = _.first;
         zodb::Tid  rev = _.second;
         if (!(blk_start <= blk && blk < blk_stop))
-            continue;   // blk out of this mapping
+            continue;   // blk ∉ this mapping
         err = mmap->_remmapblk(blk, rev);
         if (err != nil)
             return make_pair(nil, E(err));
     }
 
-    f._mmaps.push_back(mmap);    // XXX keep f._mmaps ↑blk_start
+    f._mmaps.push_back(mmap);   // TODO keep f._mmaps ↑blk_start
 
     if (vma != NULL) {
         vma->mmap_overlay_server = mmap._ptr(); // XXX +giveref

wcfs/client/wcfs.h

@@ -207,8 +207,8 @@ private:
 typedef refptr<struct _FileH> FileH;
 struct _FileH : object {
     Conn        wconn;
-    zodb::Oid   foid;       // ZBigFile root object ID
-    size_t      blksize;    // block size of this file  XXX -> off_t ?
+    zodb::Oid   foid;       // ZBigFile root object ID (does not change after fileh open)
+    size_t      blksize;    // block size of this file (does not change after fileh open)
     os::File    _headf;     // file object of head/file
     off_t       _headfsize; // head/file size is known to be at least headfsize (size ↑=)