summaryrefslogtreecommitdiff
path: root/fetch-negotiator.c
diff options
context:
space:
mode:
authorLibravatar David Aguilar <davvid@gmail.com>2021-10-12 19:45:39 -0700
committerLibravatar Junio C Hamano <gitster@pobox.com>2021-10-13 11:04:04 -0700
commit571f4348dd845fd4eccc6b19d93a7b422ed1a466 (patch)
tree38f440eb32d19ff4e9f80f83d29a7aee96d8ebcc /fetch-negotiator.c
parentGit 2.33.1 (diff)
downloadtgif-571f4348dd845fd4eccc6b19d93a7b422ed1a466.tar.xz
mergetools/xxdiff: prevent segfaults from stopping difftool
Users often use "git difftool HEAD^" to review their work, and have "mergetool.prompt" set to false so that difftool does not prompt them before diffing each file. This is very convenient because users can see all their diffs by reviewing the xxdiff windows one at a time. A problem occurs when xxdiff encounters some binary files. It can segfault and return exit code 128, which is special-cased by git-difftool-helper as being an extraordinary situation that aborts the process. Suppress the exit code from xxdiff in its diff_cmd() implementation when we see exit code 128 so that the GIT_EXTERNAL_DIFF loop continues on uninterrupted to the next file rather than aborting when it encounters the first binary file. Signed-off-by: David Aguilar <davvid@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'fetch-negotiator.c')
0 files changed, 0 insertions, 0 deletions