X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=play.c;h=5736a2ddbdfc04199f6d2bf6544f9ce2d35a56f8;hp=2cb0536dfc410eae63c0484ec9bd5535a50fe5ae;hb=7df87a748fae6a9f20d337d79ec83978856f0c48;hpb=e065f3eec9be99d82da855bf3da38d8e86752fef

diff --git a/play.c b/play.c
index 2cb0536d..5736a2dd 100644
--- a/play.c
+++ b/play.c
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2012-2013 Andre Noll <maan@systemlinux.org>
+ * Copyright (C) 2012-2014 Andre Noll <maan@systemlinux.org>
  *
  * Licensed under the GPL v2. For licencing details see COPYING.
  */
@@ -137,30 +137,14 @@ static void check_afh_receiver_or_die(void)
 	exit(EXIT_FAILURE);
 }
 
-/** Description to be included in the --detailed-help output. */
-#define PP_DESC \
-"para_play is a command line audio player.\n" \
-"\n" \
-"It operates either in command mode or in insert mode. In insert mode it\n" \
-"presents a prompt and allows to enter para_play commands like stop, play, pause\n" \
-"etc. In command mode, the current audio file is shown and the program reads\n" \
-"single key strokes from stdin. Keys may be mapped to para_play commands.\n" \
-"Whenever a mapped key is pressed, the associated command is executed.\n" \
-
 __noreturn static void print_help_and_die(void)
 {
-	int d = conf.detailed_help_given;
-	const char **p = d? play_args_info_detailed_help
-		: play_args_info_help;
-
-//	printf_or_die("%s\n\n", PLAY_CMDLINE_PARSER_PACKAGE "-"
-//		PLAY_CMDLINE_PARSER_VERSION);
-
-	printf_or_die("%s\n\n", play_args_info_usage);
-	if (d)
-		printf_or_die("%s\n", PP_DESC);
-	for (; *p; p++)
-		printf_or_die("%s\n", *p);
+	struct ggo_help help = DEFINE_GGO_HELP(play);
+	unsigned flags = conf.detailed_help_given?
+		GPH_STANDARD_FLAGS_DETAILED : GPH_STANDARD_FLAGS;
+
+	ggo_print_help(&help, flags);
+	printf("supported audio formats: %s\n", AUDIO_FORMAT_HANDLERS);
 	exit(0);
 }
 
@@ -176,12 +160,11 @@ static void parse_config_or_die(int argc, char *argv[])
 		.print_errors = 1
 	};
 
-	if (play_cmdline_parser_ext(argc, argv, &conf, &params))
-		exit(EXIT_FAILURE);
-	HANDLE_VERSION_FLAG("play", conf);
+	play_cmdline_parser_ext(argc, argv, &conf, &params);
+	loglevel = get_loglevel_by_name(conf.loglevel_arg);
+	version_handle_flag("play", conf.version_given);
 	if (conf.help_given || conf.detailed_help_given)
 		print_help_and_die();
-	loglevel = get_loglevel_by_name(conf.loglevel_arg);
 	if (conf.config_file_given)
 		config_file = para_strdup(conf.config_file_arg);
 	else {
@@ -198,6 +181,7 @@ static void parse_config_or_die(int argc, char *argv[])
 		params.initialize = 0;
 		params.check_required = 1;
 		play_cmdline_parser_config_file(config_file, &conf, &params);
+		loglevel = get_loglevel_by_name(conf.loglevel_arg);
 	}
 	for (i = 0; i < conf.key_map_given; i++) {
 		char *s = strchr(conf.key_map_arg[i] + 1, ':');
@@ -447,7 +431,7 @@ static int load_next_file(struct play_task *pt)
 	int ret;
 
 again:
-	if (pt->rq == CRT_NONE || pt->rq == CRT_FILE_CHANGE) {
+	if (pt->rq == CRT_NONE) {
 		pt->start_chunk = 0;
 		ret = next_valid_file(pt);
 		if (ret < 0)
@@ -990,15 +974,7 @@ out:
 
 static int play_i9e_line_handler(char *line)
 {
-	struct play_task *pt = &play_task;
-	int ret;
-
-	if (line == NULL || !*line)
-		return 0;
-	ret = run_command(line, pt);
-	if (ret < 0)
-		return ret;
-	return 0;
+	return run_command(line, &play_task);
 }
 
 static int play_i9e_key_handler(int key)
@@ -1044,7 +1020,7 @@ static void session_open(__a_unused struct play_task *pt)
 	char *history_file;
 	struct sigaction act;
 
-	PARA_NOTICE_LOG("\n%s\n", VERSION_TEXT("play"));
+	PARA_NOTICE_LOG("\n%s\n", version_text("play"));
 	if (conf.history_file_given)
 		history_file = para_strdup(conf.history_file_arg);
 	else {