Jeff King <peff@xxxxxxxx> writes: > On Mon, Aug 02, 2021 at 10:45:25AM +0200, Ævar Arnfjörð Bjarmason wrote: > >> I would like to see us have a setting to turn these on by default, but >> think it would be better to make that a diff.* config setting to put >> into ~/.gitconfig, i.e. we'd extend git itself to know about a list of >> extensions for the given userdiff drivers, and use them when rendering >> diffs. > > A long time ago we discussed doing this. The relevant thread is: > > https://lore.kernel.org/git/20111216110000.GA15676@xxxxxxxxxxxxxxxxxxxxx/ > > which references a few others: > > https://lore.kernel.org/git/4E569F10.8060808@xxxxxxxxxxx/ > > https://lore.kernel.org/git/4E6E928A.6080003@xxxxxxxxxxxxxx/ > ... Thanks for pointers. One good suggestion given there was to use diff=c and diff=perl in our own .gitattributes to use the patterns ourselves, which we seem to have been doing just fine ;-) As long as the default built-in ones are (1) at least 90% of the time improvement over, or at least is not broken compared to, the unconfigured case, and (2) at the lowest priority that users can easily countermand for the rest 10% cases I do not think it is too bad to resurrect the old patches from these threads. Thanks.
