cmdline: removed options --create-db and --no-create-db

Both options are deprecated, and should not be used anymore.  Many
users get confused by their presence.

3 files changed, 5 insertions, 27 deletions

diff --git a/src/cmdline.c b/src/cmdline.c
index 606f2667..d0e8e75d 100644
--- a/src/cmdline.c
+++ b/src/cmdline.c
@@ -82,17 +82,13 @@ void parse_cmdline(int argc, char **argv, struct options *options)
 	GOptionContext *context;
 	bool ret;
 	static gboolean option_version,
-		option_create_db, option_no_create_db, option_no_daemon,
+		option_no_daemon,
 		option_no_config;
 	const GOptionEntry entries[] = {
-		{ "create-db", 0, 0, G_OPTION_ARG_NONE, &option_create_db,
-		  "force (re)creation of database", NULL },
 		{ "kill", 0, 0, G_OPTION_ARG_NONE, &options->kill,
 		  "kill the currently running mpd session", NULL },
 		{ "no-config", 0, 0, G_OPTION_ARG_NONE, &option_no_config,
 		  "don't read from config", NULL },
-		{ "no-create-db", 0, 0, G_OPTION_ARG_NONE, &option_no_create_db,
-		  "don't create database, even if it doesn't exist", NULL },
 		{ "no-daemon", 0, 0, G_OPTION_ARG_NONE, &option_no_daemon,
 		  "don't detach from console", NULL },
 		{ "stdout", 0, 0, G_OPTION_ARG_NONE, &options->log_stderr,
@@ -110,7 +106,6 @@ void parse_cmdline(int argc, char **argv, struct options *options)
 	options->daemon = true;
 	options->log_stderr = false;
 	options->verbose = false;
-	options->create_db = 0;
 
 	context = g_option_context_new("[path/to/mpd.conf]");
 	g_option_context_add_main_entries(context, entries, NULL);
@@ -134,14 +129,6 @@ void parse_cmdline(int argc, char **argv, struct options *options)
 	   parser can use it already */
 	log_early_init(options->verbose);
 
-	if (option_create_db && option_no_create_db)
-		g_error("Cannot use both --create-db and --no-create-db\n");
-
-	if (option_no_create_db)
-		options->create_db = -1;
-	else if (option_create_db)
-		options->create_db = 1;
-
 	options->daemon = !option_no_daemon;
 
 	if (option_no_config) {
diff --git a/src/cmdline.h b/src/cmdline.h
index 05bf7379..eb626dbc 100644
--- a/src/cmdline.h
+++ b/src/cmdline.h
@@ -27,7 +27,6 @@ struct options {
 	gboolean daemon;
 	gboolean log_stderr;
 	gboolean verbose;
-	int create_db;
 };
 
 void parse_cmdline(int argc, char **argv, struct options *options);
diff --git a/src/main.c b/src/main.c
index f7b3d297..a0385733 100644
--- a/src/main.c
+++ b/src/main.c
@@ -123,7 +123,7 @@ glue_mapper_init(void)
  * process has been daemonized.
  */
 static bool
-glue_db_init_and_load(const struct options *options)
+glue_db_init_and_load(void)
 {
 	const char *path = config_get_path(CONF_DB_FILE);
 	bool ret;
@@ -142,19 +142,11 @@ glue_db_init_and_load(const struct options *options)
 
 	db_init(path);
 
-	if (options->create_db > 0)
-		/* don't attempt to load the old database */
-		return false;
-
 	ret = db_load(&error);
 	if (!ret) {
 		g_warning("Failed to load database: %s", error->message);
 		g_error_free(error);
 
-		if (options->create_db < 0)
-			g_error("can't open db file and using "
-				"\"--no-create-db\" command line option");
-
 		if (!db_check())
 			exit(EXIT_FAILURE);
 
@@ -331,7 +323,7 @@ int main(int argc, char *argv[])
 	decoder_plugin_init_all();
 	update_global_init();
 
-	create_db = !glue_db_init_and_load(&options);
+	create_db = !glue_db_init_and_load();
 
 	glue_sticker_init();
 
@@ -356,8 +348,8 @@ int main(int argc, char *argv[])
 	player_create();
 
 	if (create_db) {
-		/* the database failed to load, or MPD was started
-		   with --create-db: recreate a new database */
+		/* the database failed to load: recreate the
+		   database */
 		unsigned job = directory_update_init(NULL);
 		if (job == 0)
 			g_error("directory update failed");