From dcb3fcb042612bfbb311a488379c65024bafd52b Mon Sep 17 00:00:00 2001
From: Denis Vlasenko <vda.linux@googlemail.com>
Date: Sat, 19 Jul 2008 22:57:00 +0000
Subject: [PATCH] libbb: config_read() update

---
 Config.in            |  4 ----
 include/libbb.h      | 14 +++++++++-----
 libbb/parse_config.c |  8 ++++----
 miscutils/crond.c    |  8 +++-----
 4 files changed, 16 insertions(+), 18 deletions(-)

diff --git a/Config.in b/Config.in
index 8de667804..8e55edb57 100644
--- a/Config.in
+++ b/Config.in
@@ -482,10 +482,6 @@ config INCLUDE_SUSv2
 config PARSE
 	bool "Uniform config file parser debugging applet: parse"
 
-config FEATURE_PARSE_COPY
-	bool "Keep a copy of current line"
-	depends on PARSE
-
 endmenu
 
 menu 'Installation Options'
diff --git a/include/libbb.h b/include/libbb.h
index af6c1385d..4e4e37911 100644
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -990,14 +990,18 @@ int bb_parse_mode(const char* s, mode_t* theMode) FAST_FUNC;
 /*
  * Config file parser
  */
-#define PARSE_DONT_REDUCE       0x00010000 // do not treat consecutive delimiters as one
-#define PARSE_DONT_TRIM         0x00020000 // do not trim line of leading and trailing delimiters
-#define PARSE_LAST_IS_GREEDY    0x00040000 // last token takes whole remainder of the line
-//#define PARSE_DONT_NULL         0x00080000 // do not set tokens[] to NULL
+enum {
+	PARSE_DONT_REDUCE    = 0x00010000, // do not treat consecutive delimiters as one
+	PARSE_DONT_TRIM      = 0x00020000, // do not trim line of leading and trailing delimiters
+	PARSE_LAST_IS_GREEDY = 0x00040000, // last token takes whole remainder of the line
+//	PARSE_DONT_NULL      = 0x00080000, // do not set tokens[] to NULL
+	// keep a copy of current line
+	PARSE_KEEP_COPY      = 0x00200000 * ENABLE_DEBUG_CROND_OPTION,
+};
 typedef struct parser_t {
 	FILE *fp;
 	char *line;
-	USE_FEATURE_PARSE_COPY(char *data;)
+	char *data;
 	int lineno;
 } parser_t;
 parser_t* config_open(const char *filename) FAST_FUNC;
diff --git a/libbb/parse_config.c b/libbb/parse_config.c
index 3945501ad..8d7c97eb1 100644
--- a/libbb/parse_config.c
+++ b/libbb/parse_config.c
@@ -75,10 +75,10 @@ static void config_free_data(parser_t *const parser)
 {
 	free(parser->line);
 	parser->line = NULL;
-	USE_FEATURE_PARSE_COPY(
+	if (PARSE_KEEP_COPY) { /* compile-time constant */
 		free(parser->data);
 		parser->data = NULL;
-	)
+	}
 }
 
 void FAST_FUNC config_close(parser_t *parser)
@@ -179,9 +179,9 @@ int FAST_FUNC config_read(parser_t *parser, char **tokens, unsigned flags, const
 
 	// store line
 	parser->line = line = xrealloc(line, ii + 1);
-	USE_FEATURE_PARSE_COPY(
+	if (flags & PARSE_KEEP_COPY) {
 		parser->data = xstrdup(line);
-	)
+	}
 
 	/* now split line to tokens */
 	ntokens--; // now it's max allowed token no
diff --git a/miscutils/crond.c b/miscutils/crond.c
index 154243c78..c7ee793a5 100644
--- a/miscutils/crond.c
+++ b/miscutils/crond.c
@@ -470,14 +470,12 @@ static void SynchronizeFile(const char *fileName)
 		pline = &file->cf_LineBase;
 
 		while (--maxLines
-		 && (n = config_read(parser, tokens, 6, 1, "# \t", PARSE_LAST_IS_GREEDY))
+		 && (n = config_read(parser, tokens, 6, 1, "# \t", PARSE_LAST_IS_GREEDY|PARSE_KEEP_COPY))
 		) {
 			CronLine *line;
 
-			USE_FEATURE_PARSE_COPY(
-				if (DebugOpt)
-					crondlog(LVL5 "user:%s entry:%s", fileName, parser->data);
-			)
+			if (DebugOpt)
+				crondlog(LVL5 "user:%s entry:%s", fileName, parser->data);
 
 			/* check if line is setting MAILTO= */
 			if (0 == strncmp(tokens[0], "MAILTO=", 7)) {