db_completion: add missing mpdclient::FinishCommand() call

This consumes the status code. If we forget this, the connection will be closed by libmpdclient.
MusicPlayerDaemon · Sep 11, 2024 · 2625f14 · 2625f14
1 parent d0639fb
commit 2625f14
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/NEWS b/NEWS
@@ -11,6 +11,7 @@ ncmpc 0.50 - not yet released
 * support MPD 0.22 tag "label" (requires libmpdclient 2.17)
 * fix config file overriding MPD_HOST environment variable
 * click on progress bar seeks current song
+* fix tab completion bug
 
 ncmpc 0.49 - (2023-08-04)
 * fix UI freeze if lyrics plugin is stuck

diff --git a/src/db_completion.cxx b/src/db_completion.cxx
@@ -47,4 +47,6 @@ gcmp_list_from_path(struct mpdclient &c, const char *path,
 
 		completion.emplace(std::move(name));
 	}
+
+	c.FinishCommand();
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -47,4 +47,6 @@ gcmp_list_from_path(struct mpdclient &c, const char *path, @@
     		completion.emplace(std::move(name));
     	}
+    	c.FinishCommand();
     }