.

wcfs/wcfs_test.py

@@ -784,9 +784,6 @@ class tFile:
             # failed and shut down. But on test shutdown .fmmap is unmapped for
             # all opened tFiles, and so read will hit SIGSEGV. Prepare to catch
             # that SIGSEGV here.
-            #
-            # XXX after WatchLink is moved to pyx/nogil, do we still need to do
-            # here with nogil?
             have_read = chan(1)
             def _():
                 try:
@@ -823,7 +820,7 @@ class tFile:
         assert t.cached()[blk] > 0
 
         # verify full data of the block
-        # XXX assert individually for every block's page? (easier debugging?)
+        # TODO(?) assert individually for every block's page? (easier debugging?)
         assert blkview.tobytes() == dataok
 
         # we just accessed the block in full - it has to be in OS cache completely
@@ -885,27 +882,6 @@ class tWatchLink(wcfs.WatchLink):
             w.pinned = {}
         t._watching = {}
 
-# XXX just wrap req.at with tAt inplace
-"""
-    # recvReq is the same as WatchLink.recvReq but returns tSrvReq instead of PinReq.
-    def recvReq(t, ctx): # -> tSrvReq | None when EOF
-        req = super(tWatchLink, t).recvReq(ctx)
-        if req is not None:
-            assert req.__class__ is wcfs.PinReq
-            req.__class__ = tSrvReq
-        return req
-
-class tSrvReq(wcfs.PinReq):
-    # _parse is the same as PinReq._parse, but returns at wrapped with tAt.
-    # XXX -> just wrap `at`
-    def _parse(req): # -> (foid, blk, at|None)
-        foid, blk, at = super(tSrvReq, req)._parse()
-        if at is not None:
-            at = tAt(req.wlink.tdb, at)
-        return foid, blk, at
-"""
-
-
 
 # ---- infrastructure: watch setup/adjust ----