Re: [PATCH] update-index/diff-index: use core.preloadindex to improve performance
- From
karsten.blees@dcon.de <karsten.blees@dcon.de>
- Date
- Nov 13, 2012, 15:36 UTC
- Message-ID
- <OF27D0F811.18A373FB-ONC1257AB5.0055809D-C1257AB5.0055B47D@dcon.de>
- In-Reply-To
- <20121102153800.GE11170@sigill.intra.peff.net>
Jeff King <peff@peff.net> wrote on 02.11.2012 16:38:00:
Show 11 quoted lines
> On Fri, Nov 02, 2012 at 11:26:16AM -0400, Jeff King wrote: > > > Still, I don't think we need to worry about performance regressions, > > because people who don't have a setup suitable for it will not turn on > > core.preloadindex in the first place. And if they have it on, the more > > places we use it, probably the better. > > BTW, your patch was badly damaged in transit (wrapped, and tabs > converted to spaces). I was able to fix it up, but please check your > mailer's settings. >
Yes, I feared as much, that's why I included the pull URL (the company MTA only accepts outbound mail from Notes clients, sorry).
Is there a policy for people with broken mailers (send patch as attachment, add pull URL more prominently, don't include plaintext patch at all...)?