On Jan 13, 2008 3:17 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> "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 used a pseudo unlimited number just to make code shorter.
After considering it again, i find that the code is still brief by using
a real unlimited number. So i'll correct it.
> 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.
>
Reasonable
--
Ping Yin
-
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
