Fix cmake clang warnings

psx95 · psx95 · commit adc5fff66bc3 · 2024-10-31T02:15:35.000Z
diff --git a/api/test/trace/propagation/detail/string_test.cc b/api/test/trace/propagation/detail/string_test.cc
@@ -9,5 +9,5 @@
 
 TEST(StringTest, SplitString)
 {
-  //TODO(psx95): Add tests
+  // TODO(psx95): Add tests
 }
diff --git a/ext/test/w3c_tracecontext_test/main.cc b/ext/test/w3c_tracecontext_test/main.cc
@@ -71,7 +71,7 @@ class TextMapCarrierTest : public context::propagation::TextMapCarrier
     {
       return false;
     }
-    for (int i = 0; i < str1.length(); i++)
+    for (size_t i = 0; i < str1.length(); i++)
     {
       if (tolower(str1[i]) != tolower(str2[i]))
       {

Original file line number	Diff line number	Diff line change
`@@ -9,5 +9,5 @@`
`9`	`9`
`10`	`10`	`TEST(StringTest, SplitString)`
`11`	`11`	`{`
`12`		`- //TODO(psx95): Add tests`
	`12`	`+ // TODO(psx95): Add tests`
`13`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -71,7 +71,7 @@ class TextMapCarrierTest : public context::propagation::TextMapCarrier`
`71`	`71`	`{`
`72`	`72`	`return false;`
`73`	`73`	`}`
`74`		`- for (int i = 0; i < str1.length(); i++)`
	`74`	`+ for (size_t i = 0; i < str1.length(); i++)`
`75`	`75`	`{`
`76`	`76`	`if (tolower(str1[i]) != tolower(str2[i]))`
`77`	`77`	`{`