There are currently 9 hazards reported relating to TokenStream::Position. These are false positives since AutoKeepAtoms is in effect when TokenStream in use, so the atoms contained in Position are not going to be collected. To get rid of these hazards, I've made Position take a dummy AutoKeepAtoms reference in its constructor to ensure that one of these is present, and told the static analysis to regard it as rooted. I've also marked Position and AutoKeepAtoms as stack-only classes for good measure. I'm hoping you agree this is a sensible thing to do :-)

Comment on attachment 742397 [details] [diff] [review] Proposed changes Review of attachment 742397 [details] [diff] [review]: ----------------------------------------------------------------- I like it. Requiring the AutoKeep reference makes it way harder to use this incorrectly in the future.