On Thu, Oct 17, 2019 at 3:08 AM Lars Ingebrigtsen <address@hidden
> Aren't these mostly doc string bugs, though? That is, the doc string
> says "t" instead of "non-nil"; it's a common bug.
Depends. When the function is a predicate, t/nil makes more sense (generally speaking, there are exceptions) that non-nil/nil.
In this and the other patches, I've opted for the fix (either doc fix, or code fix) that I thought it was more logical, or matched better the function's intent, or less prone to cause a bug.