On Thu, Mar 16, 2017 at 7:30 AM, Kyle Huey <me@xxxxxxxxxxxx> wrote:
> On Thu, Mar 16, 2017 at 4:09 AM, Michael Ellerman <mpe@xxxxxxxxxxxxxx> wrote:
>>
>> Presumably he's done:
>>
>> $ git config diff.context 8
>
> Indeed. In my case it dates back to my days hacking on Firefox, which
> wants 8 lines of context for patches. I'll remove it.
You can make those kinds of options per-repository rather than
user-global, so you can keep the 8-line thing for firefox without
having it for the kernel.
Not that it *hurts* for the kernel, but it just looked odd to me
because the patch looked so much bigger. More context lines can
obviously help with ambiguous cases, but we very seldom have that kind
of ambiguity.
I'm assuming firefox has files that have lots and lots of repeated
lines (or things like autogenerated content with a lot of common
boilerplate or whatever), which is why they like lots of context?
Linus
