Re: [PATCH] attr: map builtin userdiff drivers to well-known extensions

On Fri, Dec 16, 2011 at 07:26:00PM -0000, Philip Oakley wrote:

quoted

+ "*.m diff=objc",

There is a conflict here with the Matlab community which also uses
"*.m" files for its scripts and code.
They fit the "It's not that hard to do, but it's an extra step that
the user might not even know is an option." rationale.

If the objc.m is used as a default it must be overidable easily, and
listed in the appropriate documentation to mitigate the "might not
even know" risk.

It is easily overridable; just put your own "*.m" (or anything that
matches your files) entry into your gitattributes file. I'm more
concerned that people will start getting worse results than with the
default, and not know how to fix it.

If you have some Matlab files, would you mind doing diffs with the
default driver and with the objc driver, and comment on how good or bad
the results are?

-Peff