Merge pull request #717 from open-telemetry/main

merged from upstream

Author: malkia

api/include/opentelemetry/baggage/baggage.h

@@ -121,7 +121,7 @@ class Baggage
     }
 
     nostd::shared_ptr<Baggage> baggage(new Baggage(cnt));
-    bool kv_valid;
+    bool kv_valid{false};
     nostd::string_view key, value;
 
     while (kv_str_tokenizer.next(kv_valid, key, value) && baggage->kv_properties_->Size() < cnt)

api/include/opentelemetry/trace/propagation/http_trace_context.h

@@ -80,7 +80,7 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
 
   static TraceFlags TraceFlagsFromHex(nostd::string_view trace_flags)
   {
-    uint8_t flags;
+    uint8_t flags{0};
     detail::HexToBinary(trace_flags, &flags, sizeof(flags));
     return TraceFlags(flags);
   }
@@ -140,7 +140,7 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
     }
 
     // hex is valid, convert it to binary
-    uint8_t version_binary;
+    uint8_t version_binary{kInvalidVersion};
     detail::HexToBinary(version_hex, &version_binary, sizeof(version_binary));
     if (version_binary == kInvalidVersion)
     {

api/include/opentelemetry/trace/propagation/jaeger.h

@@ -112,7 +112,7 @@ class JaegerPropagator : public context::propagation::TextMapPropagator
       return SpanContext::GetInvalid();
     }
 
-    uint8_t flags;
+    uint8_t flags{0};
     if (!detail::HexToBinary(flags_hex, &flags, sizeof(flags)))
     {
       return SpanContext::GetInvalid();

api/include/opentelemetry/trace/trace_state.h

@@ -64,7 +64,7 @@ class TraceState
     }
 
     nostd::shared_ptr<TraceState> ts(new TraceState(cnt));
-    bool kv_valid;
+    bool kv_valid{false};
     nostd::string_view key, value;
     while (kv_str_tokenizer.next(kv_valid, key, value) && ts->kv_properties_->Size() < cnt)
     {

sdk/include/opentelemetry/sdk/metrics/view/view_registry.h

@@ -68,7 +68,7 @@ class ViewRegistry
 
       if (aggregation_type == AggregationType::kDefault)
       {
-        bool is_monotonic;
+        bool is_monotonic{false};
         aggregation_type = DefaultAggregation::GetDefaultAggregationType(
             instrument_selector->GetInstrumentType(), is_monotonic);
       }

sdk/test/configuration/yaml_distribution_test.cc

@@ -67,7 +67,7 @@ file_format: "1.0-distribution"
   auto *distribution = config->distribution.get();
   ASSERT_NE(distribution, nullptr);
   ASSERT_EQ(distribution->entries.size(), 2);
-  opentelemetry::sdk::configuration::DocumentNode *node;
+  opentelemetry::sdk::configuration::DocumentNode *node{nullptr};
   std::unique_ptr<opentelemetry::sdk::configuration::DocumentNode> property;
   std::string name;