On Thu, Sep 28, 2006 at 01:54:27PM -0700, Linus Torvalds wrote:
>
>
> On Tue, 26 Sep 2006, Junio C Hamano wrote:
> >
> > This adds two parameters to "diff --stat".
> >
> > . --stat-width=72 tells that the page should fit on 72-column output.
> >
> > . --stat-name-width=30 tells that the filename part is limited
> > to 30 columns.
>
> Thinking some more about this, I have to say, I do hate the syntax.
>
> It may be clear thanks to being verbose, but it's _hell_ to write.
>
> It has the same problem the "--stat-with-patch" argument had: sure, it
> worked, but it was really really inconvenient, and just doing a
> combination of "--stat -p" is much nicer.
>
> So how about just extending the existing "--stat" thing, and just making
> it do something like
>
> git diff --stat=72,30
>
> instead (perhaps along with a config option to set the defaults to
> something else if we want to).
>
> What do you think?
>...
What about staying compatible with diffstat?
IOW, change --stat-width=72 to -w72, or at least allow it alternatively?
> Linus
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
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