svn commit: samba r24519 - in branches: SAMBA_3_2/source/param SAMBA_3_2_0/source/param

obnox at samba.org obnox at samba.org
Fri Aug 17 13:07:53 GMT 2007


Author: obnox
Date: 2007-08-17 13:07:52 +0000 (Fri, 17 Aug 2007)
New Revision: 24519

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=24519

Log:
Refactor printing of one parameter into a helper function of its own.


Modified:
   branches/SAMBA_3_2/source/param/loadparm.c
   branches/SAMBA_3_2_0/source/param/loadparm.c


Changeset:
Modified: branches/SAMBA_3_2/source/param/loadparm.c
===================================================================
--- branches/SAMBA_3_2/source/param/loadparm.c	2007-08-17 13:01:51 UTC (rev 24518)
+++ branches/SAMBA_3_2/source/param/loadparm.c	2007-08-17 13:07:52 UTC (rev 24519)
@@ -2202,6 +2202,7 @@
 static BOOL hash_a_service(const char *name, int number);
 static void free_service_byindex(int iService);
 static char * canonicalize_servicename(const char *name);
+static void show_parameter(int parmIndex);
 
 /* This is a helper function for parametrical options support. */
 /* It returns a pointer to parametrical option value if it exists or NULL otherwise */
@@ -2883,14 +2884,14 @@
 }
 
 /***************************************************************************
- Show all parameter's name, type, [values,] and flags.
+ Show one parameter's name, type, [values,] and flags.
+ (helper functions for show_parameter_list)
 ***************************************************************************/
 
-void show_parameter_list(void)
+static void show_parameter(int parmIndex)
 {
-	int classIndex, parmIndex, enumIndex, flagIndex;
+	int enumIndex, flagIndex;
 	BOOL hadFlag;
-	const char *section_names[] = { "local", "global", NULL};
 	const char *type[] = { "P_BOOL", "P_BOOLREV", "P_CHAR", "P_INTEGER",
 		"P_OCTAL", "P_LIST", "P_STRING", "P_USTRING", "P_GSTRING",
 		"P_UGSTRING", "P_ENUM", "P_SEP"};
@@ -2901,35 +2902,46 @@
 		"FLAG_GLOBAL", "FLAG_WIZARD", "FLAG_ADVANCED", "FLAG_DEVELOPER",
 		"FLAG_DEPRECATED", "FLAG_HIDE", "FLAG_DOS_STRING", NULL};
 
-	for ( classIndex=0; section_names[classIndex]; classIndex++) {
+	printf("%s=%s", parm_table[parmIndex].label,
+	       type[parm_table[parmIndex].type]);
+	if (parm_table[parmIndex].type == P_ENUM) {
+		printf(",");
+		for (enumIndex=0;
+		     parm_table[parmIndex].enum_list[enumIndex].name;
+		     enumIndex++)
+		{
+			printf("%s%s",
+			       enumIndex ? "|" : "",
+			       parm_table[parmIndex].enum_list[enumIndex].name);
+		}
+	}
+	printf(",");
+	hadFlag = False;
+	for (flagIndex=0; flag_names[flagIndex]; flagIndex++) {
+		if (parm_table[parmIndex].flags & flags[flagIndex]) {
+			printf("%s%s",
+				hadFlag ? "|" : "",
+				flag_names[flagIndex]);
+			hadFlag = True;
+		}
+	}
+	printf("\n");
+}
+
+/***************************************************************************
+ Show all parameter's name, type, [values,] and flags.
+***************************************************************************/
+
+void show_parameter_list(void)
+{
+	int classIndex, parmIndex;
+	const char *section_names[] = { "local", "global", NULL};
+
+	for (classIndex=0; section_names[classIndex]; classIndex++) {
 		printf("[%s]\n", section_names[classIndex]);
 		for (parmIndex = 0; parm_table[parmIndex].label; parmIndex++) {
 			if (parm_table[parmIndex].p_class == classIndex) {
-				printf("%s=%s", 
-					parm_table[parmIndex].label,
-					type[parm_table[parmIndex].type]);
-				switch (parm_table[parmIndex].type) {
-				case P_ENUM:
-					printf(",");
-					for (enumIndex=0; parm_table[parmIndex].enum_list[enumIndex].name; enumIndex++)
-						printf("%s%s",
-							enumIndex ? "|" : "",
-							parm_table[parmIndex].enum_list[enumIndex].name);
-					break;
-				default:
-					break;
-				}
-				printf(",");
-				hadFlag = False;
-				for ( flagIndex=0; flag_names[flagIndex]; flagIndex++ ) {
-					if (parm_table[parmIndex].flags & flags[flagIndex]) {
-						printf("%s%s",
-							hadFlag ? "|" : "",
-							flag_names[flagIndex]);
-						hadFlag = True;
-					}
-				}
-				printf("\n");
+				show_parameter(parmIndex);
 			}
 		}
 	}

Modified: branches/SAMBA_3_2_0/source/param/loadparm.c
===================================================================
--- branches/SAMBA_3_2_0/source/param/loadparm.c	2007-08-17 13:01:51 UTC (rev 24518)
+++ branches/SAMBA_3_2_0/source/param/loadparm.c	2007-08-17 13:07:52 UTC (rev 24519)
@@ -2207,6 +2207,7 @@
 static BOOL hash_a_service(const char *name, int number);
 static void free_service_byindex(int iService);
 static char * canonicalize_servicename(const char *name);
+static void show_parameter(int parmIndex);
 
 /* This is a helper function for parametrical options support. */
 /* It returns a pointer to parametrical option value if it exists or NULL otherwise */
@@ -2888,14 +2889,14 @@
 }
 
 /***************************************************************************
- Show all parameter's name, type, [values,] and flags.
+ Show one parameter's name, type, [values,] and flags.
+ (helper functions for show_parameter_list)
 ***************************************************************************/
 
-void show_parameter_list(void)
+static void show_parameter(int parmIndex)
 {
-	int classIndex, parmIndex, enumIndex, flagIndex;
+	int enumIndex, flagIndex;
 	BOOL hadFlag;
-	const char *section_names[] = { "local", "global", NULL};
 	const char *type[] = { "P_BOOL", "P_BOOLREV", "P_CHAR", "P_INTEGER",
 		"P_OCTAL", "P_LIST", "P_STRING", "P_USTRING", "P_GSTRING",
 		"P_UGSTRING", "P_ENUM", "P_SEP"};
@@ -2906,35 +2907,46 @@
 		"FLAG_GLOBAL", "FLAG_WIZARD", "FLAG_ADVANCED", "FLAG_DEVELOPER",
 		"FLAG_DEPRECATED", "FLAG_HIDE", "FLAG_DOS_STRING", NULL};
 
-	for ( classIndex=0; section_names[classIndex]; classIndex++) {
+	printf("%s=%s", parm_table[parmIndex].label,
+	       type[parm_table[parmIndex].type]);
+	if (parm_table[parmIndex].type == P_ENUM) {
+		printf(",");
+		for (enumIndex=0;
+		     parm_table[parmIndex].enum_list[enumIndex].name;
+		     enumIndex++)
+		{
+			printf("%s%s",
+			       enumIndex ? "|" : "",
+			       parm_table[parmIndex].enum_list[enumIndex].name);
+		}
+	}
+	printf(",");
+	hadFlag = False;
+	for (flagIndex=0; flag_names[flagIndex]; flagIndex++) {
+		if (parm_table[parmIndex].flags & flags[flagIndex]) {
+			printf("%s%s",
+				hadFlag ? "|" : "",
+				flag_names[flagIndex]);
+			hadFlag = True;
+		}
+	}
+	printf("\n");
+}
+
+/***************************************************************************
+ Show all parameter's name, type, [values,] and flags.
+***************************************************************************/
+
+void show_parameter_list(void)
+{
+	int classIndex, parmIndex;
+	const char *section_names[] = { "local", "global", NULL};
+
+	for (classIndex=0; section_names[classIndex]; classIndex++) {
 		printf("[%s]\n", section_names[classIndex]);
 		for (parmIndex = 0; parm_table[parmIndex].label; parmIndex++) {
 			if (parm_table[parmIndex].p_class == classIndex) {
-				printf("%s=%s", 
-					parm_table[parmIndex].label,
-					type[parm_table[parmIndex].type]);
-				switch (parm_table[parmIndex].type) {
-				case P_ENUM:
-					printf(",");
-					for (enumIndex=0; parm_table[parmIndex].enum_list[enumIndex].name; enumIndex++)
-						printf("%s%s",
-							enumIndex ? "|" : "",
-							parm_table[parmIndex].enum_list[enumIndex].name);
-					break;
-				default:
-					break;
-				}
-				printf(",");
-				hadFlag = False;
-				for ( flagIndex=0; flag_names[flagIndex]; flagIndex++ ) {
-					if (parm_table[parmIndex].flags & flags[flagIndex]) {
-						printf("%s%s",
-							hadFlag ? "|" : "",
-							flag_names[flagIndex]);
-						hadFlag = True;
-					}
-				}
-				printf("\n");
+				show_parameter(parmIndex);
 			}
 		}
 	}



More information about the samba-cvs mailing list