"Ping Yin" <pkufranky@xxxxxxxxx> writes:
> On Jan 12, 2008 4:36 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
>> Ping Yin <pkufranky@xxxxxxxxx> writes:
>>
>> > @@ -265,6 +267,10 @@ set_name_rev () {
>> > #
>> > modules_summary()
>> > {
>> > + summary_limit=${summary_limit:-1000000}
>>
>> Why a million?
> Because i think a million is big enough. I'd better define a constant
> for unlimited number.
I think that is a wrong approach to begin with. You are
assuming that you will always limit and by using improbably
large limit to pretend it is unlimited. Why not making the
summary list generator truely capable of produce an unlimited
list?
I also think using 100 or so as a sane default, allowing the
user to override to say "I do not want any limitation", is a
much better default.
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
