Code

diff --quiet: make it synonym to --exit-code >/dev/null
authorJunio C Hamano <gitster@pobox.com>
Tue, 2 Sep 2008 06:20:26 +0000 (23:20 -0700)
committerJunio C Hamano <gitster@pobox.com>
Sun, 7 Sep 2008 02:15:04 +0000 (19:15 -0700)
commitdf58a8274d6865020682a6739bc59b87a9761991
tree058e4fc78ab6c9f13a012f7ed8aabd3ada0ef097
parent9d865356abc08b253bc15b5fb50471d80c828be1
diff --quiet: make it synonym to --exit-code >/dev/null

The point of --quiet was to return the status as early as possible without
doing any extra processing.  Well behaved scripts, when they expect to run
many diff operations inside, are supposed to run "update-index --refresh"
upfront; we do not want them to pay the price of iterating over the index
and comparing the contents to fix the stat dirtiness, and we avoided most
of the processing in diffcore_std() when --quiet is in effect.

But scripts that adhere to the good practice won't have to pay any more
price than the necessary lstat(2) that will report stat cleanliness, as
long as only -q is given without any fancier diff options.

More importantly, users who do ask for "--quiet -M --filter=D" (in order
to notice only the deletion, not paths that disappeared only because they
have been renamed away) deserve to get the result they asked for, even it
means they have to pay the extra price; the alternative is to get a cheap
early return that gives a result they did not ask for, which is much
worse.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
diff.c