Re: [PATCH] Produce a nicer output in case of sha1_object_info failures in ls-tree -l
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 19, 2009, 21:55 UTC
- Message-ID
- <7v4oxp89eb.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <20090319203002.GA31014@blimp.localdomain>
Alex Riesen <raa.lkml@gmail.com> writes:
Show 30 quoted lines
> Initialize the size with 0. The error message is already printed
> by sha1_object_info itself. Otherwise the uninitialized size is
> printed, which does not make any sense at all.
>
> Signed-off-by: Alex Riesen <raa.lkml@gmail.com>
> ---
>
> builtin-ls-tree.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/builtin-ls-tree.c b/builtin-ls-tree.c
> index fca4631..a8cdafb 100644
> --- a/builtin-ls-tree.c
> +++ b/builtin-ls-tree.c
> @@ -60,7 +60,6 @@ static int show_tree(const unsigned char *sha1, const char *base, int baselen,
> {
> int retval = 0;
> const char *type = blob_type;
> - unsigned long size;
>
> if (S_ISGITLINK(mode)) {
> /*
> @@ -91,6 +90,7 @@ static int show_tree(const unsigned char *sha1, const char *base, int baselen,
> if (!(ls_options & LS_NAME_ONLY)) {
> if (ls_options & LS_SHOW_SIZE) {
> if (!strcmp(type, blob_type)) {
> + unsigned long size = 0;
> sha1_object_info(sha1, &size);
> printf("%06o %s %s %7lu\t", mode, type,
> abbrev ? find_unique_abbrev(sha1, abbrev)Hmm, shouldn't you be checking the return value from sha1_object_info() and skipping the printf() altogether instead?