From: Denis Vlasenko <vda.linux@googlemail.com>
Date: Wed, 15 Oct 2008 08:45:54 +0000 (-0000)
Subject: sysctl: remove trivial comments. no code changes
X-Git-Tag: 1_13_0~109
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=81944c9e92991eff0d1a9fc289b21cc745f43f9c;p=oweals%2Fbusybox.git

sysctl: remove trivial comments. no code changes
---

diff --git a/procps/sysctl.c b/procps/sysctl.c
index c095d16d0..90e47ea9a 100644
--- a/procps/sysctl.c
+++ b/procps/sysctl.c
@@ -70,41 +70,32 @@ int sysctl_main(int argc UNUSED_PARAM, char **argv)
 	return retval;
 } /* end sysctl_main() */
 
-/*
- * preload the sysctl's from a conf file
- * - we parse the file and then reform it (strip out whitespace)
+/* Set sysctl's from a conf file. Format example:
+ * # Controls IP packet forwarding
+ * net.ipv4.ip_forward = 0
  */
-
 static int sysctl_preload_file_and_exit(const char *filename)
 {
 	char *token[2];
 	parser_t *parser;
 
 	parser = config_open(filename);
-	while (config_read(parser, token, 2, 2, "# \t=", PARSE_NORMAL)) { // TODO: ';' is comment char too
-//		if (!token[1]) {
-//			bb_error_msg("warning: %s(%d): invalid syntax, continuing",
-//					filename, parser->lineno);
-//		} else {
-		{
+// TODO: ';' is comment char too
+	while (config_read(parser, token, 2, 2, "# \t=", PARSE_NORMAL)) {
 #if 0
-			char *s = xasprintf("%s=%s", token[0], token[1]);
-			sysctl_write_setting(s);
-			free(s);
-#else // PLAY_WITH_FIRE for -4 bytes?
-			sprintf(parser->line, "%s=%s", token[0], token[1]); // must have room by definition
-			sysctl_write_setting(parser->line);
+		char *s = xasprintf("%s=%s", token[0], token[1]);
+		sysctl_write_setting(s);
+		free(s);
+#else /* Save ~4 bytes by using parser internals */
+		sprintf(parser->line, "%s=%s", token[0], token[1]); // must have room by definition
+		sysctl_write_setting(parser->line);
 #endif
-		}
 	}
 	if (ENABLE_FEATURE_CLEAN_UP)
 		config_close(parser);
 	return 0;
 } /* end sysctl_preload_file_and_exit() */
 
-/*
- *     Write a single sysctl setting
- */
 static int sysctl_write_setting(const char *setting)
 {
 	int retval;
@@ -162,9 +153,6 @@ static int sysctl_write_setting(const char *setting)
 	return retval;
 } /* end sysctl_write_setting() */
 
-/*
- *     Read a sysctl setting
- */
 static int sysctl_read_setting(const char *name)
 {
 	int retval;
@@ -214,12 +202,9 @@ static int sysctl_read_setting(const char *name)
 	return retval;
 } /* end sysctl_read_setting() */
 
-/*
- *     Display all the sysctl settings
- */
 static int sysctl_display_all(const char *path)
 {
-	int retval = 0;
+	int retval = EXIT_SUCCESS;
 	DIR *dp;
 	struct dirent *de;
 	char *tmpdir;