Re: [PATCH] Add --show-size to git log to print message size
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jul 16, 2007, 22:37 UTC
- Message-ID
- <7vveck2cvs.fsf@assigned-by-dhcp.cox.net>
- In-Reply-To
- <e5bfff550707161102n3b6da174i7bbff64e1739a1b2@mail.gmail.com>
"Marco Costalba" <mcostalba@gmail.com> writes:
Show 21 quoted lines
> On 7/16/07, Marco Costalba <mcostalba@gmail.com> wrote: >> On 7/16/07, Junio C Hamano <gitster@pobox.com> wrote: >> > >> > I'd say making --show-message-size option incompatible with diff output >> > would be good enough futureproofing for now. >> > >> Oooops, I didn't see your post. >> >> I agree 100%, please tell me if doc it's clear enough or it would be >> better to clarify that "message log" it means only message and no diff >> content. >> > > Sorry to bother you again, but my English is very bad and I would like > to be clear. > > When i say no diff content I mean that git log --log-size -p it's a > perfect valid command but --log-size will make git print the size of > _only_ the log part, it means from the line after "log size xxx\n" > until the end of log message that can be '\0' (if no diff) or the > beginning of diff part.
What I originally meant with my comment was to _error_ out if the user says "git log --log-size -p" (or "git show --log-size" without "-s").
But I guess it is probably fine, as long as it is crystal clear to the users that we do not do size for non log part. I still have this nagging feeling that this is an ugly workaround for Qt library's programming interface, though...