Clearer checking for return from route_g_init

src/common/slurm_route.c

@@ -348,7 +348,7 @@ extern int route_g_split_hostlist(hostlist_t hl,
 	char *buf;
 
 	nnodes = nnodex = 0;
-	if (route_g_init(NULL) < 0)
+	if (route_g_init(NULL) != SLURM_SUCCESS)
 		return SLURM_ERROR;
 
 	if (debug_flags & DEBUG_FLAG_ROUTE) {
@@ -387,7 +387,7 @@ extern int route_g_split_hostlist(hostlist_t hl,
  */
 extern int route_g_reconfigure(void)
 {
-	if (route_g_init(NULL) < 0)
+	if (route_g_init(NULL) != SLURM_SUCCESS)
 		return SLURM_ERROR;
 	debug_flags = slurm_get_debug_flags();
 	tree_width = slurm_get_tree_width();
@@ -407,7 +407,7 @@ extern int route_g_reconfigure(void)
  */
 extern slurm_addr_t* route_g_next_collector ( bool *is_collector )
 {
-	if (route_g_init(NULL) < 0)
+	if (route_g_init(NULL) != SLURM_SUCCESS)
 		return NULL;
 
 	return (*(ops.next_collector))(is_collector);
@@ -423,7 +423,7 @@ extern slurm_addr_t* route_g_next_collector ( bool *is_collector )
  */
 extern slurm_addr_t* route_g_next_collector_backup ( void )
 {
-	if (route_g_init(NULL) < 0)
+	if (route_g_init(NULL) != SLURM_SUCCESS)
 		return NULL;
 
 	return (*(ops.next_collector_backup))();