Merge pull request #37 from open-telemetry/main

malkia · web-flow · commit b595d65de026 · 2024-05-23T15:04:06.000-07:00
[BUILD] Remove the hard-coded separator in tracestate (open-telemetry#2672)
diff --git a/api/include/opentelemetry/trace/trace_state.h b/api/include/opentelemetry/trace/trace_state.h
@@ -95,7 +95,7 @@ class TraceState
         [&header_s, &first](nostd::string_view key, nostd::string_view value) noexcept {
           if (!first)
           {
-            header_s.append(",");
+            header_s.append(1, kMembersSeparator);
           }
           else
           {

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ class TraceState`
`95`	`95`	`[&header_s, &first](nostd::string_view key, nostd::string_view value) noexcept {`
`96`	`96`	`if (!first)`
`97`	`97`	`{`
`98`		`- header_s.append(",");`
	`98`	`+ header_s.append(1, kMembersSeparator);`
`99`	`99`	`}`
`100`	`100`	`else`
`101`	`101`	`{`