From 2ccdfb1c78e1695c4df714887b73ab417ca60f22 Mon Sep 17 00:00:00 2001 From: Pratyush Yadav Date: Mon, 26 Aug 2019 04:23:13 +0530 Subject: [PATCH 1/1] git-gui: return early when patch fails to apply In the procedure apply_or_revert_range_or_line, if the patch does not apply successfully, a dialog is shown, but execution proceeds after that. Instead, return early on error so the parts that come after this don't work on top of an error state. Signed-off-by: Pratyush Yadav --- lib/diff.tcl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/diff.tcl b/lib/diff.tcl index ffca7880e1..0659029632 100644 --- a/lib/diff.tcl +++ b/lib/diff.tcl @@ -848,6 +848,8 @@ proc apply_or_revert_range_or_line {x y revert} { puts -nonewline $p $wholepatch close $p} err]} { error_popup "$failed_msg\n\n$err" + unlock_index + return } unlock_index -- 2.47.1