Re: [PATCH v3 0/4] remove extern from function declarations
- From
Jacob Keller <jacob.keller@gmail.com>
- Date
- May 5, 2019, 18:09 UTC
- Message-ID
- <CA+P7+xonqw76E4FU3NX7gGafWy3YAcyeQgvYsPjgBvUh_KD_=A@mail.gmail.com>
- In-Reply-To
- <xmqqo94hsc63.fsf@gitster-ct.c.googlers.com>
On Sat, May 4, 2019 at 10:28 PM Junio C Hamano <gitster@pobox.com> wrote:
Show 16 quoted lines
> > Jeff King <peff@peff.net> writes: > > > I think spatch is smart enough not to hit the same header multiple > > times. But the problem is that we invoke it once per file, so it > > actually processes cache.h many times. That's slow, but also produces > > bogus patches. > > Yes, I've seen this and was a bit irritated myself, but not enough > to do something about it myself, yet. > > > > Jacob Keller's patches to collapse this to a single invocation do fix it > > (and I've used them selectively in the past rather than cleaning up the > > resulting patch manually ;) ). > > Ah, that is nice to know. Do we want to resurrect it?
I remember deciding that the memory cost trade off was too high back then.
Thanks, Jake