Просмотр исходного кода

Increase code coverage from 19.4% to 100% by adding defines
to remove debug print code in production builds. Also removed
function by ifdef that is never used in the project.

(Logical change 1.71)


git-svn-id: http://svn.fedorahosted.org/svn/corosync/trunk@254 fd59a12c-fef9-0310-b244-a6a79926bd2f

Steven Dake 21 лет назад
Родитель
Сommit
bda91bf27f
1 измененных файлов с 19 добавлено и 13 удалено
  1. 19 13
      exec/print.c

+ 19 - 13
exec/print.c

@@ -77,6 +77,21 @@ internal_log_printf_checkdebug (int level, char *string, ...)
 	va_end(ap);
 }
 
+#ifdef DEBUG
+char *getSaNameT (SaNameT *name)
+{
+	static char ret_name[300];
+
+	memset (ret_name, 0, sizeof (ret_name));
+	if (name->length > 299) {
+		memcpy (ret_name, name->value, 299);
+	} else {
+
+		memcpy (ret_name, name->value, name->length);
+	}
+	return (ret_name);
+}
+
 extern char *getSaClmNodeAddressT (SaClmNodeAddressT *nodeAddress) {
 	int i;
 	static char node_address[300];
@@ -103,21 +118,10 @@ void printSaClmClusterNodeT (char *description, SaClmClusterNodeT *clusterNode)
 
 	log_printf (LOG_LEVEL_NOTICE, "\tTimestamp is %llx nanoseconds\n", clusterNode->bootTimestamp);
 }
+#endif /* DEBUG */
 
-char *getSaNameT (SaNameT *name)
-{
-	static char ret_name[300];
-
-	memset (ret_name, 0, sizeof (ret_name));
-	if (name->length > 299) {
-		memcpy (ret_name, name->value, 299);
-	} else {
-
-		memcpy (ret_name, name->value, name->length);
-	}
-	return (ret_name);
-}
 
+#ifdef CODE_COVERAGE_COMPILE_OUT
 void saAmfPrintGroups (void)
 {
 	struct list_head *AmfGroupList;
@@ -190,3 +194,5 @@ void saAmfPrintGroups (void)
 		log_printf (LOG_LEVEL_DEBUG, "}\n");
 	}
 }
+#endif /* CODE_COVERAGE_COMPILE_OUT */
+