diff options
| author |  Junio C Hamano <gitster@pobox.com> | 2014-12-12 14:31:39 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2014-12-12 14:31:39 -0800 |
| commit | 0ddedd4d6b9b3e8eb3557d8ed28e1a0b354a25f8 (patch) | |
| tree | bec8ec6f7d910bce51a681bb81a601709d95c54a /mergetools | |
| parent | Merge branch 'jk/colors-fix' (diff) | |
| parent | mergetools: stop setting $status in merge_cmd() (diff) | |
| download | tgif-0ddedd4d6b9b3e8eb3557d8ed28e1a0b354a25f8.tar.xz | |
Merge branch 'da/difftool-mergetool-simplify-reporting-status'
Code simplification.
* da/difftool-mergetool-simplify-reporting-status:
mergetools: stop setting $status in merge_cmd()
mergetool: simplify conditionals
difftool--helper: add explicit exit statement
mergetool--lib: remove use of $status global
mergetool--lib: remove no-op assignment to $status from setup_user_tool
Diffstat (limited to 'mergetools')
| -rw-r--r-- | mergetools/diffmerge | 1 | ||||
| -rw-r--r-- | mergetools/emerge | 1 | ||||
| -rw-r--r-- | mergetools/kdiff3 | 1 |
3 files changed, 0 insertions, 3 deletions
diff --git a/mergetools/diffmerge b/mergetools/diffmerge index 85ac720157..f138cb4e73 100644 --- a/mergetools/diffmerge +++ b/mergetools/diffmerge @@ -11,5 +11,4 @@ merge_cmd () { "$merge_tool_path" --merge \ --result="$MERGED" "$LOCAL" "$REMOTE" fi - status=$? } diff --git a/mergetools/emerge b/mergetools/emerge index f96d9e550a..7b895fdb1f 100644 --- a/mergetools/emerge +++ b/mergetools/emerge @@ -15,7 +15,6 @@ merge_cmd () { "$LOCAL" "$REMOTE" \ "$(basename "$MERGED")" fi - status=$? } translate_merge_tool_path() { diff --git a/mergetools/kdiff3 b/mergetools/kdiff3 index a30034f116..793d1293b1 100644 --- a/mergetools/kdiff3 +++ b/mergetools/kdiff3 @@ -20,5 +20,4 @@ merge_cmd () { -o "$MERGED" "$LOCAL" "$REMOTE" \ >/dev/null 2>&1 fi - status=$? } |