From 907f80b4a48169748ebd28b05b08a712b91627e2 Mon Sep 17 00:00:00 2001
From: Moe Jette <jette1@llnl.gov>
Date: Mon, 3 Jun 2002 23:35:49 +0000
Subject: [PATCH] Renamed parse_node_name to parse_node_names. Changed
 xmalloc/xfree and error to malloc/free and printf.

---
 src/common/nodelist.c   | 31 +++++++++++++++++--------------
 src/common/nodelist.h   |  4 ++--
 src/scontrol/scontrol.c |  2 +-
 3 files changed, 20 insertions(+), 17 deletions(-)

diff --git a/src/common/nodelist.c b/src/common/nodelist.c
index 94037ff165e..1f833276d49 100644
--- a/src/common/nodelist.c
+++ b/src/common/nodelist.c
@@ -20,9 +20,6 @@
 #include <string.h>
 #include <syslog.h>
 
-#include "xmalloc.h"
-#include "log.h"
-
 /*
  * bitfmt2int - convert a string describing bitmap (e.g. "0-30,45,50-60") 
  *	into an array of integer (start/end) pairs terminated by -1
@@ -73,7 +70,7 @@ bitfmt2int (char *bit_str_ptr)
 }
 
 /* 
- * parse_node_name - parse the node name for regular expressions and return a 
+ * parse_node_names - parse the node name for regular expressions and return a 
  *                   sprintf format generate multiple node names as needed.
  * input: node_name - node name to parse
  * output: format - sprintf format for generating names
@@ -85,14 +82,14 @@ bitfmt2int (char *bit_str_ptr)
  *       is no longer needed
  */
 int 
-parse_node_name (char *node_name, char **format, int *start_inx, int *end_inx,
+parse_node_names (char *node_name, char **format, int *start_inx, int *end_inx,
 		 int *count_inx) 
 {
 	int base, format_pos, precision, i;
 	char type[1];
 
 	i = strlen (node_name);
-	format[0] = (char *) xmalloc (i + 1);
+	format[0] = (char *) malloc (i + 1);
 
 	*start_inx = 0;
 	*end_inx = 0;
@@ -113,9 +110,11 @@ parse_node_name (char *node_name, char **format, int *start_inx, int *end_inx,
 			if (node_name[++i] == (char) NULL)
 				break;
 			if (base != 0) {
-				error ("parse_node_name: invalid '[' in node name %s\n",
-					node_name);
-				xfree (format[0]);
+#if DEBUG_SYSTEM > 1
+				printf ("parse_node_name: invalid '[' in node name %s\n", 
+					node_name); 
+#endif
+				free (format[0]);
 				return EINVAL;
 			}
 			if (node_name[i] == 'o') {
@@ -140,9 +139,11 @@ parse_node_name (char *node_name, char **format, int *start_inx, int *end_inx,
 					i++;
 					break;
 				}
-				error ("parse_node_name: invalid '%c' in node name %s\n",
-					 node_name[i], node_name);
-				xfree (format[0]);
+#if DEBUG_SYSTEM > 1
+				printf ("parse_node_name: invalid '%c' in node name %s\n", 
+					 node_name[i], node_name); 
+#endif
+				free (format[0]);
 				return EINVAL;
 			}
 			while (1) {
@@ -157,9 +158,11 @@ parse_node_name (char *node_name, char **format, int *start_inx, int *end_inx,
 					i++;
 					break;
 				}
-				error ("parse_node_name: invalid '%c' in node name %s\n",
+#if DEBUG_SYSTEM > 1
+				printf ("parse_node_name: invalid '%c' in node name %s\n",
 					 node_name[i], node_name);
-				xfree (format[0]);
+#endif
+				free (format[0]);
 				return EINVAL;
 			}
 			*count_inx = (*end_inx - *start_inx) + 1;
diff --git a/src/common/nodelist.h b/src/common/nodelist.h
index 2996046e934..e40db10bfa7 100644
--- a/src/common/nodelist.h
+++ b/src/common/nodelist.h
@@ -8,7 +8,7 @@
 #ifndef _NODELIST_H
 #define _NODELIST_H
 
-int * bitfmt2int (char *bit_str_ptr);
-int parse_node_name(char *name, char **fmt, int *start, int *end, int *count); 
+extern int * bitfmt2int (char *bit_str_ptr);
+extern int parse_node_names(char *name, char **fmt, int *start, int *end, int *count); 
 
 #endif /* !_NODELIST_H */
diff --git a/src/scontrol/scontrol.c b/src/scontrol/scontrol.c
index 99596074b22..137dddeb230 100644
--- a/src/scontrol/scontrol.c
+++ b/src/scontrol/scontrol.c
@@ -451,7 +451,7 @@ print_node_list (char *node_list)
 		str_ptr2 = (char *) strtok_r (my_node_list, ",", &str_ptr1);
 		while (str_ptr2) {	/* break apart by comma separators */
 			error_code =
-				parse_node_name (str_ptr2, &format,
+				parse_node_names (str_ptr2, &format,
 						 &start_inx, &end_inx,
 						 &count_inx);
 			if (error_code) {
-- 
GitLab