fix issue when TERM is not set in environment

AdhocMan · AdhocMan · commit 950ecf85b852 · 2024-03-26T11:12:01.000+01:00
diff --git a/include/gtest_mpi/gtest_mpi_internal.hpp b/include/gtest_mpi/gtest_mpi_internal.hpp
@@ -112,7 +112,11 @@ void PrintFullTestCommentIfPresent(const ::testing::TestInfo &test_info) {
   }
 }
 
-bool CStringEquals(const char *l, const char *r) { return std::strcmp(l, r) == 0; }
+bool CStringEquals(const char *l, const char *r) {
+  if (l == nullptr) return r == nullptr;
+  if (r == nullptr) return false;
+  return std::strcmp(l, r) == 0; 
+}
 
 bool CaseInsensitiveCStringEquals(const char *l, const char *r) {
   bool equal = true;

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,11 @@ void PrintFullTestCommentIfPresent(const ::testing::TestInfo &test_info) {`
`112`	`112`	`}`
`113`	`113`	`}`
`114`	`114`
`115`		`-bool CStringEquals(const char l, const char r) { return std::strcmp(l, r) == 0; }`
	`115`	`+bool CStringEquals(const char l, const char r) {`
	`116`	`+ if (l == nullptr) return r == nullptr;`
	`117`	`+ if (r == nullptr) return false;`
	`118`	`+ return std::strcmp(l, r) == 0;`
	`119`	`+}`
`116`	`120`
`117`	`121`	`bool CaseInsensitiveCStringEquals(const char l, const char r) {`
`118`	`122`	`bool equal = true;`