git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 3/4] git-gui: fix exception when trying to stage with empty file list

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jan 9, 2018, 14:33 UTC
Message-ID
<c227a5f8354c736abca92e9b040f7e6f569af119.1515508360.git.johannes.schindelin@gmx.de>
In-Reply-To
<cover.1515508360.git.johannes.schindelin@gmx.de>

If there is nothing to stage, there is nothing to stage. Let's not try to, even if the file list contains nothing at all.

This fixes https://github.com/git-for-windows/git/issues/1075
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
 git-gui/git-gui.sh | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh
index 3c085cddc61..ca2cdebdc4f 100755
--- a/git-gui/git-gui.sh
+++ b/git-gui/git-gui.sh
@@ -2504,7 +2504,9 @@ proc toggle_or_diff {mode w args} {
 		if {$last_clicked ne {}} {
 			set lno [lindex $last_clicked 1]
 		} else {
-			if {[llength $file_lists($w)] == 0} {
+			if {![info exists file_lists]
+				|| ![info exists file_lists($w)]
+				|| [llength $file_lists($w)] == 0} {
 				set last_clicked {}
 				return
 			}
@@ -2518,7 +2520,13 @@ proc toggle_or_diff {mode w args} {
 		}
 	}
 
-	set path [lindex $file_lists($w) [expr {$lno - 1}]]
+	if {![info exists file_lists]
+		|| ![info exists file_lists($w)]
+		|| [llength $file_lists($w)] < $lno - 1} {
+		set path {}
+	} else {
+		set path [lindex $file_lists($w) [expr {$lno - 1}]]
+	}
 	if {$path eq {}} {
 		set last_clicked {}
 		return
-- 
2.15.1.windows.2.395.g5bb0817ee52
Previous: Johannes SchindelinNext: Johannes Schindelin
Message 4 of 8 in “"Fast-"track Git GUI changes”
  1. 0/4 "Fast-"track Git GUI changesJohannes Schindelin, Jan 9, 2018
  2. 1/4 git gui: fix staging a second line to a 1-line fileJohannes Schindelin, Jan 9, 2018
  3. 2/4 git-gui: avoid exception upon Ctrl+T in an empty listJohannes Schindelin, Jan 9, 2018
  4. 3/4 git-gui: fix exception when trying to stage with empty file listJohannes Schindelin, Jan 9, 2018
  5. 4/4 git-gui: allow Ctrl+T to toggle multiple pathsJohannes Schindelin, Jan 9, 2018
  6. Junio C HamanoJan 9, 2018
  7. Junio C HamanoJan 9, 2018
  8. Johannes SchindelinJan 10, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.