Re: [PATCH MISC 1/1] Make gcc warning about empty if body go away.
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Nov 8, 2007, 02:01 UTC
- Message-ID
- <7vode57awg.fsf@gitster.siamese.dyndns.org>
- In-Reply-To
- <1194430832-6224-2-git-send-email-madcoder@debian.org>
Pierre Habouzit <madcoder@debian.org> writes:
Show 13 quoted lines
> diff --git a/builtin-diff.c b/builtin-diff.c > index f77352b..80392a8 100644 > --- a/builtin-diff.c > +++ b/builtin-diff.c > @@ -204,7 +204,7 @@ static void refresh_index_quietly(void) > if (write_cache(fd, active_cache, active_nr) || > close(fd) || > commit_locked_index(lock_file)) > - ; /* > + (void)0; /* > * silently ignore it -- we haven't mucked > * with the real index. > */
Wouldn't this be much easier to read, by the way?
The point is that if we touched the active_cache, we try to write it out and make it the index file for later users to use by calling "commit", but we do not really care about the failure from this sequence because it is done purely as an optimization.
The original code called three functions primarily for their side effects, which is admittedly a bad style.
builtin-diff.c | 12 +++--------- 1 files changed, 3 insertions(+), 9 deletions(-)
diff --git a/builtin-diff.c b/builtin-diff.c index f77352b..906c924 100644 --- a/builtin-diff.c +++ b/builtin-diff.c @@ -200,15 +200,9 @@ static void refresh_index_quietly(void) discard_cache(); read_cache(); refresh_cache(REFRESH_QUIET|REFRESH_UNMERGED); - if (active_cache_changed) { - if (write_cache(fd, active_cache, active_nr) || - close(fd) || - commit_locked_index(lock_file)) - ; /* - * silently ignore it -- we haven't mucked - * with the real index. - */ - } + if (active_cache_changed && + !write_cache(fd, active_cache, active_nr) && !close(fd)) + commit_locked_index(lock_file); rollback_lock_file(lock_file); }