Options: remove option "text-editor-ask"

This option seems useless to me.
MusicPlayerDaemon · Sep 6, 2024 · 0c7c32b · 0c7c32b
1 parent ad9dced
commit 0c7c32b
Show file tree

Hide file tree

Showing 6 changed files with 2 additions and 18 deletions.
diff --git a/NEWS b/NEWS
@@ -3,6 +3,7 @@ ncmpc 0.50 - not yet released
 * require libfmt 9
 * draw progress bar as Unicode double line
 * add color style "text" for input text controls
+* remove option "text-editor-ask"
 * lyrics/musixmatch: add new lyrics extension
 * lyrics/google: fix partial loading of lyrics
 

diff --git a/doc/config.sample b/doc/config.sample
@@ -119,9 +119,6 @@
 ## The text editor used for editing lyrics.
 #text-editor = emacs
 
-## Ask before starting an editor.
-#text-editor-ask = yes
-
 ## The prefix of chat messages
 #chat-prefix = "<user> "
 

diff --git a/doc/index.rst b/doc/index.rst
@@ -168,8 +168,6 @@ used to receive lyrics on the lyrics screen.
 :command:`text-editor = EDITOR` - The text editor used for editing
 lyrics.
 
-:command:`text-editor-ask = yes|no` - Ask before starting an editor.
-
 :command:`chat-prefix = PREFIX` - Prefix messages send with the chat
 screen with PREFIX.  By default they are prefixed with the current
 user name enclosed by ``<`` and ``>`` and a space (i.e. "<name> ").

diff --git a/src/ConfigParser.cxx b/src/ConfigParser.cxx
@@ -734,11 +734,8 @@ parse_line(char *line)
 		{}
 #endif
 	else if (StringIsEqualIgnoreCase(name, CONF_TEXT_EDITOR_ASK))
-#ifdef ENABLE_LYRICS_SCREEN
-		options.text_editor_ask = str2bool(value);
-#else
+		/* obsolete, ignore */
 		{}
-#endif
 	else if (StringIsEqualIgnoreCase(name, CONF_CHAT_PREFIX))
 #ifdef ENABLE_CHAT_SCREEN
 		options.chat_prefix = GetStringValue(value);

diff --git a/src/LyricsPage.cxx b/src/LyricsPage.cxx
@@ -15,7 +15,6 @@
 #include "screen.hxx"
 #include "plugin.hxx"
 #include "TextPage.hxx"
-#include "screen_utils.hxx"
 #include "ncu.hxx"
 #include "lib/fmt/ToSpan.hxx"
 #include "util/StringAPI.hxx"
@@ -349,13 +348,6 @@ LyricsPage::Edit()
 	}
 
 	const char *editor = options.text_editor.c_str();
-	if (options.text_editor_ask) {
-		const char *prompt =
-			_("Do you really want to start an editor and edit these lyrics?");
-		bool really = screen_get_yesno(screen, prompt, false);
-		if (!really)
-			return;
-	}
 
 	if (!Save())
 		return;

diff --git a/src/Options.hxx b/src/Options.hxx
@@ -55,7 +55,6 @@ struct Options {
 	std::chrono::steady_clock::duration lyrics_timeout = std::chrono::minutes(1);
 	bool lyrics_autosave = false;
 	bool lyrics_show_plugin = false;
-	bool text_editor_ask = false;
 #endif
 
 	bool auto_center;