David Fetter <da...@fetter.org> writes:
> Please find attached a run of a tool that looks for duplicated tokens.
> I've removed some things that seem like false positives, basically all
> from the stemmer part of the source, but there's still a lot.
I thought you were talking about problems like "that that" typos,
but on looking at the file, what this is actually complaining
about is any duplicated code segments anywhere. I do not find
this helpful. Refactoring to the point that dozen-line code
stanzas never appear more than once would be incredibly invasive,
likely very bad for performance, and I don't think it'd improve
readability either.
> - Would it make sense to make some kind of git commit trigger that at
> least warns when a new one has been introduced?
Commit triggers are NOT the place for heuristics about code quality,
even if they're well-considered heuristics.
regards, tom lane
