On Fri, Dec 19 09:35, Brandon Allbery wrote:
There's at least three variations of this module that have been requested; and, oddly, everyone proposing a variant thinks that it is intuitively obvious that it's the only sensible one and wonders why nobody else ever considered it or if there's some technical reason why it's not already "the" implementation.
Hm; I must not have expressed myself clearly. My intention in labelling the change as "personal preference", asking whether there were existing reasons for having it the current way, and putting it up for discussion, was precisely to indicate that I *don't* think this is the only sensible way, and that I suspect someone *has* considered different ways.
In other words, you should be providing an *alternative* instead of replacing the existing implementation, because your "obviously correct" is not obvious to everyone else.
Again, when I said "personal preference", I did not expect it to be taken as implying I believe it to be "obviously correct". I have attached a modified patch that adds this behaviour as a separate option that does not replace the existing behaviour. Cheers, -- Felix