diff options
| author |  Jeff King <peff@peff.net> | 2017-09-05 08:14:43 -0400 |
|---|---|---|
| committer |  Junio C Hamano <gitster@pobox.com> | 2017-09-06 17:19:53 +0900 |
| commit | 9b028aa45a2016ae0dbdfeb85ad9d43f2017db0d (patch) | |
| tree | 5208c5c116ffeb67bec90c1d2e00d71631152739 /contrib | |
| parent | tempfile: handle NULL tempfile pointers gracefully (diff) | |
| download | tgif-9b028aa45a2016ae0dbdfeb85ad9d43f2017db0d.tar.xz | |
tempfile: replace die("BUG") with BUG()
Compared to die(), using BUG() triggers abort(). That may
give us an actual coredump, which should make it easier to
get a stack trace. And since the programming error for these
assertions is not in the functions themselves but in their
callers, such a stack trace is needed to actually find the
source of the bug.
In addition, abort() raises SIGABRT, which is more likely to
be caught by our test suite.
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions