Re: [PATCH 10/15] for-each-ref: introduce format specifier %>(*) and %<(*)
- From
Duy Nguyen <pclouds@gmail.com>
- Date
- Jun 5, 2013, 10:15 UTC
- Message-ID
- <CACsJy8Bm5tmyUO_e2Uus3RuyOUphyrJOT955T646BdFT060UBA@mail.gmail.com>
- In-Reply-To
- <CALkWK0=9X6QhCOPfV6Gq31VVccyDoO3ciPxG+2kqpgL2foiQSw@mail.gmail.com>
On Wed, Jun 5, 2013 at 3:14 PM, Ramkumar Ramachandra <artagnon@gmail.com> wrote:
> Duy Nguyen wrote: >> I mentioned it before and I do it again. This is not optimal. > > Yeah, I'll attempt to fix this, but it's not urgent.
Agreed it's not urgent.
Show 6 quoted lines
>> But I guess it's ok in this >> shape unless you run this over hundreds of refs. > > Oh, you can run over a hundred refs just fine, for scripting purposes; > but why would you want to run over a hundred refs with a pretty that > includes %>(*) or %<(*)?
Good point. "git for-each-ref|wc -l" on my git repo says I have 673 refs. A naive use for-each-ref --pretty without any ref patterns could happen. But I guess people will quickly learn to limit the ref selection soon after doing that :-) -- Duy