Yeah, maybe it's a remnant from that?  We definitely don't cache unwatched directories now though -- getContents() checks isWatched() before storing anything.

I guess there's a potential edge case due to the 'relaxed' flag -- if you read a directory while watchers are starting up and then watching fails _without_ watchers going fully offline, it looks like we might not clear the dir's cache.  But otoh, we also won't get a chance notification for that dir, so it still seems unlikely this code would help.

Anyway, I guess I'll push up the change to add a `true` arg there, and just take it on faith that it will work as well in the other two cases... since I can't seem to come up with any cases to exercise that code :-)