add quieter versions of parse_{tree,commit}
authorJeff King <peff@peff.net>
Mon, 1 Jun 2015 09:56:26 +0000 (05:56 -0400)
committerJunio C Hamano <gitster@pobox.com>
Mon, 1 Jun 2015 16:29:42 +0000 (09:29 -0700)
commit9cc2b07a7c95fad0bb5e3a7a8db29bebdb90d92b
tree36523a5d16a71ba803ce5576b207d2db1542b30e
parentfdf96a20acf96a6ac538df8113b2aafd6ed71d50
add quieter versions of parse_{tree,commit}

When we call parse_commit, it will complain to stderr if the
object does not exist or cannot be read. This means that we
may produce useless error messages if this situation is
expected (e.g., because the object is marked UNINTERESTING,
or because revs->ignore_missing_links is set).

We can fix this by adding a new "parse_X_gently" form that
takes a flag to suppress the messages. The existing
"parse_X" form is already gentle in the sense that it
returns an error rather than dying, and we could in theory
just add a "quiet" flag to it (with existing callers passing
"0"). But doing it this way means we do not have to disturb
existing callers.

Note also that the new flag is "quiet_on_missing", and not
just "quiet". We could add a flag to suppress _all_ errors,
but besides being a more invasive change (we would have to
pass the flag down to sub-functions, too), there is a good
reason not to: we would never want to use it. Missing a
linked object is expected in some circumstances, but it is
never expected to have a malformed commit, or to get a tree
when we wanted a commit.  We should always complain about
these corruptions.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
commit.c
commit.h
tree.c
tree.h