[CodeHealth] Fix clang-tidy warnings part 2 (#3496)

dbarker · web-flow · commit f74d413c63b0 · 2025-06-26T10:22:39.000+02:00
diff --git a/.github/workflows/clang-tidy.yaml b/.github/workflows/clang-tidy.yaml
@@ -17,9 +17,9 @@ jobs:
       matrix:
         include:
           - cmake_options: all-options-abiv1-preview
-            warning_limit: 172
+            warning_limit: 109
           - cmake_options: all-options-abiv2-preview
-            warning_limit: 173
+            warning_limit: 109
     steps:
       - name: Harden the runner (Audit all outbound calls)
         uses: step-security/harden-runner@002fdce3c6a235733a90a27c80493a3241e56863 # v2.12.1
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -33,6 +33,9 @@ Increment the:
 * [CodeHealth] Fix clang-tidy warnings part 1
   [#3493](https://github.com/open-telemetry/opentelemetry-cpp/pull/3493)
 
+* [CodeHealth] Fix clang-tidy warnings part 2
+  [#3496](https://github.com/open-telemetry/opentelemetry-cpp/pull/3496)
+
 Important changes:
 
 * [REMOVAL] Removed deprecated semantic convention header files
diff --git a/api/include/opentelemetry/plugin/hook.h b/api/include/opentelemetry/plugin/hook.h
@@ -4,6 +4,7 @@
 #pragma once
 
 #include "opentelemetry/nostd/unique_ptr.h"
+#include "opentelemetry/plugin/factory.h"
 #include "opentelemetry/version.h"
 
 #ifdef _WIN32
@@ -42,7 +43,6 @@ namespace plugin
 {
 
 struct LoaderInfo;
-class FactoryImpl;
 
 using OpenTelemetryHook =
     nostd::unique_ptr<Factory::FactoryImpl> (*)(const LoaderInfo &loader_info,
diff --git a/api/test/logs/logger_test.cc b/api/test/logs/logger_test.cc
@@ -206,7 +206,11 @@ class TestLogger : public Logger
 
   using Logger::EmitLogRecord;
 
-  void EmitLogRecord(nostd::unique_ptr<opentelemetry::logs::LogRecord> &&) noexcept override {}
+  void EmitLogRecord(
+      nostd::unique_ptr<opentelemetry::logs::LogRecord> &&log_record) noexcept override
+  {
+    auto log_record_ptr = std::move(log_record);
+  }
 };
 
 // Define a basic LoggerProvider class that returns an instance of the logger class defined above
diff --git a/examples/multi_processor/main.cc b/examples/multi_processor/main.cc
@@ -54,7 +54,7 @@ std::shared_ptr<InMemorySpanData> InitTracer()
       trace_sdk::TracerProviderFactory::Create(std::move(processors));
 
   // Set the global trace provider
-  trace_sdk::Provider::SetTracerProvider(std::move(provider));
+  trace_sdk::Provider::SetTracerProvider(provider);
 
   return data;
 }
diff --git a/examples/otlp/http_instrumented_main.cc b/examples/otlp/http_instrumented_main.cc
@@ -103,46 +103,45 @@ class MyThreadInstrumentation : public opentelemetry::sdk::common::ThreadInstrum
     {
       std::cout << ", priority " << priority_;
     }
-    std::cout << std::endl << std::flush;
+    std::cout << "\n" << std::flush;
   }
 
   void OnEnd() override
   {
     std::lock_guard<std::mutex> lock_guard(serialize);
-    std::cout << "OnEnd() thread " << thread_name_ << ", id " << std::this_thread::get_id()
-              << std::endl
+    std::cout << "OnEnd() thread " << thread_name_ << ", id " << std::this_thread::get_id() << "\n"
               << std::flush;
   }
 
   void BeforeWait() override
   {
     std::lock_guard<std::mutex> lock_guard(serialize);
     std::cout << "BeforeWait() thread " << thread_name_ << ", id " << std::this_thread::get_id()
-              << ", waiting" << std::endl
+              << ", waiting\n"
               << std::flush;
   }
 
   void AfterWait() override
   {
     std::lock_guard<std::mutex> lock_guard(serialize);
     std::cout << "AfterWait() thread " << thread_name_ << ", id " << std::this_thread::get_id()
-              << ", done waiting" << std::endl
+              << ", done waiting\n"
               << std::flush;
   }
 
   void BeforeLoad() override
   {
     std::lock_guard<std::mutex> lock_guard(serialize);
     std::cout << "BeforeLoad() thread " << thread_name_ << ", id " << std::this_thread::get_id()
-              << ", about to work" << std::endl
+              << ", about to work\n"
               << std::flush;
   }
 
   void AfterLoad() override
   {
     std::lock_guard<std::mutex> lock_guard(serialize);
     std::cout << "AfterLoad() thread " << thread_name_ << ", id " << std::this_thread::get_id()
-              << ", done working" << std::endl
+              << ", done working\n"
               << std::flush;
   }
 
@@ -341,25 +340,25 @@ int main(int argc, char *argv[])
     logger_opts.url = "http://localhost:4318/v1/logs";
   }
 
-  std::cout << "Initializing opentelemetry-cpp" << std::endl << std::flush;
+  std::cout << "Initializing opentelemetry-cpp\n" << std::flush;
 
   InitTracer();
   InitMetrics();
   InitLogger();
 
-  std::cout << "Application payload" << std::endl << std::flush;
+  std::cout << "Application payload\n" << std::flush;
 
   foo_library();
 
   std::string name{"otlp_http_metric_example"};
   foo_library::observable_counter_example(name);
 
-  std::cout << "Shutting down opentelemetry-cpp" << std::endl << std::flush;
+  std::cout << "Shutting down opentelemetry-cpp\n" << std::flush;
 
   CleanupLogger();
   CleanupMetrics();
   CleanupTracer();
 
-  std::cout << "Done" << std::endl << std::flush;
+  std::cout << "Done\n" << std::flush;
   return 0;
 }
diff --git a/exporters/ostream/test/ostream_log_test.cc b/exporters/ostream/test/ostream_log_test.cc
@@ -125,7 +125,7 @@ TEST(OstreamLogExporter, DefaultLogRecordToCout)
       "  severity_text      : INVALID\n",
       "  body               : \n",
       "  resource           : \n",
-      "    telemetry.sdk.version: " OPENTELEMETRY_VERSION "\n",
+      std::string{"    telemetry.sdk.version: "} + OPENTELEMETRY_VERSION + "\n",
       "    telemetry.sdk.name: opentelemetry\n",
       "    telemetry.sdk.language: cpp\n",
       "  attributes         : \n",
@@ -136,7 +136,7 @@ TEST(OstreamLogExporter, DefaultLogRecordToCout)
       "  trace_flags        : 00\n",
       "  scope              : \n",
       "    name             : otel-cpp\n",
-      "    version          : " OPENTELEMETRY_SDK_VERSION "\n",
+      std::string{"    version          : "} + OPENTELEMETRY_SDK_VERSION + "\n",
       "    schema_url       : https://opentelemetry.io/schemas/1.15.0\n",
       "    attributes       : \n",
       "      scope.attr.key: scope.attr.value\n",
@@ -199,7 +199,7 @@ TEST(OStreamLogRecordExporter, SimpleLogToCout)
           "  severity_text      : TRACE\n"
           "  body               : Message\n",
       "  resource           : \n",
-      "    telemetry.sdk.version: " OPENTELEMETRY_VERSION "\n",
+      std::string{"    telemetry.sdk.version: "} + OPENTELEMETRY_VERSION + "\n",
       "    telemetry.sdk.name: opentelemetry\n",
       "    telemetry.sdk.language: cpp\n",
       "  attributes         : \n",
@@ -210,7 +210,7 @@ TEST(OStreamLogRecordExporter, SimpleLogToCout)
       "  trace_flags        : 00\n",
       "  scope              : \n",
       "    name             : otel-cpp\n",
-      "    version          : " OPENTELEMETRY_SDK_VERSION "\n",
+      std::string{"    version          : "} + OPENTELEMETRY_SDK_VERSION + "\n",
       "    schema_url       : https://opentelemetry.io/schemas/1.15.0\n",
       "    attributes       : \n",
       "      scope.attr.key: scope.attr.value\n",
@@ -270,7 +270,7 @@ TEST(OStreamLogRecordExporter, LogWithStringAttributesToCerr)
       "  severity_text      : INVALID\n",
       "  body               : \n",
       "  resource           : \n",
-      "    telemetry.sdk.version: " OPENTELEMETRY_VERSION "\n",
+      std::string{"    telemetry.sdk.version: "} + OPENTELEMETRY_VERSION + "\n",
       "    telemetry.sdk.name: opentelemetry\n",
       "    telemetry.sdk.language: cpp\n",
       "    service.name: unknown_service\n",
@@ -284,7 +284,7 @@ TEST(OStreamLogRecordExporter, LogWithStringAttributesToCerr)
       "  trace_flags        : 00\n",
       "  scope              : \n",
       "    name             : otel-cpp\n",
-      "    version          : " OPENTELEMETRY_SDK_VERSION "\n",
+      std::string{"    version          : "} + OPENTELEMETRY_SDK_VERSION + "\n",
       "    schema_url       : https://opentelemetry.io/schemas/1.15.0\n",
       "    attributes       : \n",
       "      scope.attr.key: scope.attr.value\n",
@@ -352,7 +352,7 @@ TEST(OStreamLogRecordExporter, LogWithVariantTypesToClog)
       "  body               : \n",
       "  resource           : \n",
       "    service.name: unknown_service\n",
-      "    telemetry.sdk.version: " OPENTELEMETRY_VERSION "\n",
+      std::string{"    telemetry.sdk.version: "} + OPENTELEMETRY_VERSION + "\n",
       "    telemetry.sdk.name: opentelemetry\n",
       "    telemetry.sdk.language: cpp\n",
       "    res1: [1,2,3]\n",
@@ -365,7 +365,7 @@ TEST(OStreamLogRecordExporter, LogWithVariantTypesToClog)
       "  trace_flags        : 00\n",
       "  scope              : \n",
       "    name             : otel-cpp\n",
-      "    version          : " OPENTELEMETRY_SDK_VERSION "\n",
+      std::string{"    version          : "} + OPENTELEMETRY_SDK_VERSION + "\n",
       "    schema_url       : https://opentelemetry.io/schemas/1.15.0\n",
       "    attributes       : \n",
       "      scope.attr.key: scope.attr.value\n",
@@ -425,7 +425,7 @@ TEST(OStreamLogRecordExporter, IntegrationTest)
       "  severity_text      : DEBUG\n",
       "  body               : Hello\n",
       "  resource           : \n",
-      "    telemetry.sdk.version: " OPENTELEMETRY_VERSION "\n",
+      std::string{"    telemetry.sdk.version: "} + OPENTELEMETRY_VERSION + "\n",
       "    service.name: unknown_service\n",
       "    telemetry.sdk.name: opentelemetry\n",
       "    telemetry.sdk.language: cpp\n",
@@ -437,7 +437,7 @@ TEST(OStreamLogRecordExporter, IntegrationTest)
       "  trace_flags        : 00\n",
       "  scope              : \n",
       "    name             : opentelelemtry_library\n",
-      "    version          : " OPENTELEMETRY_SDK_VERSION "\n",
+      std::string{"    version          : "} + OPENTELEMETRY_SDK_VERSION + "\n",
       "    schema_url       : https://opentelemetry.io/schemas/1.11.0\n",
       "    attributes       : \n",
       "      scope.attr.key: 123\n",
@@ -496,7 +496,7 @@ TEST(OStreamLogRecordExporter, IntegrationTestWithEventId)
       "  severity_text      : DEBUG\n",
       "  body               : Hello {key1} {key2}\n",
       "  resource           : \n",
-      "    telemetry.sdk.version: " OPENTELEMETRY_VERSION "\n",
+      std::string{"    telemetry.sdk.version: "} + OPENTELEMETRY_VERSION + "\n",
       "    service.name: unknown_service\n",
       "    telemetry.sdk.name: opentelemetry\n",
       "    telemetry.sdk.language: cpp\n",
@@ -508,7 +508,7 @@ TEST(OStreamLogRecordExporter, IntegrationTestWithEventId)
       "  trace_flags        : 00\n",
       "  scope              : \n",
       "    name             : opentelelemtry_library\n",
-      "    version          : " OPENTELEMETRY_SDK_VERSION "\n",
+      std::string{"    version          : "} + OPENTELEMETRY_SDK_VERSION + "\n",
       "    schema_url       : https://opentelemetry.io/schemas/1.11.0\n",
       "    attributes       : \n",
       "      scope.attr.key: 123\n",
diff --git a/exporters/otlp/src/otlp_file_client.cc b/exporters/otlp/src/otlp_file_client.cc
@@ -1200,7 +1200,7 @@ class OPENTELEMETRY_LOCAL_SYMBOL OtlpFileSystemBackend : public OtlpFileAppender
       {
         if (file_pattern[i] == '%')
         {
-          int checked = static_cast<int>(file_pattern[i + 1]);
+          int checked = static_cast<unsigned char>(file_pattern[i + 1]);
           if (checked > 0 && checked < 128 && check_interval[checked] > 0)
           {
             if (0 == check_file_path_interval_ ||
diff --git a/exporters/otlp/src/otlp_grpc_client.cc b/exporters/otlp/src/otlp_grpc_client.cc
@@ -91,7 +91,7 @@ class OPENTELEMETRY_LOCAL_SYMBOL OtlpGrpcAsyncCallData : public OtlpGrpcAsyncCal
       result_callback;
 
   OtlpGrpcAsyncCallData() {}
-  virtual ~OtlpGrpcAsyncCallData() {}
+  ~OtlpGrpcAsyncCallData() override {}
 };
 }  // namespace
 #endif
diff --git a/exporters/otlp/src/otlp_http_client.cc b/exporters/otlp/src/otlp_http_client.cc
@@ -665,7 +665,7 @@ void ConvertListFieldToJson(nlohmann::json &value,
 OtlpHttpClient::OtlpHttpClient(OtlpHttpClientOptions &&options)
     : is_shutdown_(false),
       options_(std::move(options)),
-      http_client_(http_client::HttpClientFactory::Create(options.thread_instrumentation)),
+      http_client_(http_client::HttpClientFactory::Create(options_.thread_instrumentation)),
       start_session_counter_(0),
       finished_session_counter_(0)
 {
diff --git a/exporters/otlp/test/otlp_grpc_log_record_exporter_test.cc b/exporters/otlp/test/otlp_grpc_log_record_exporter_test.cc
@@ -125,7 +125,7 @@ class OtlpMockTraceServiceStub : public proto::collector::trace::v1::MockTraceSe
   public:
     async_interface(OtlpMockTraceServiceStub *owner) : stub_(owner) {}
 
-    virtual ~async_interface() override = default;
+    ~async_interface() override = default;
 
     void Export(
         ::grpc::ClientContext *context,
@@ -188,7 +188,7 @@ class OtlpMockLogsServiceStub : public proto::collector::logs::v1::MockLogsServi
   public:
     async_interface(OtlpMockLogsServiceStub *owner) : stub_(owner) {}
 
-    virtual ~async_interface() override = default;
+    ~async_interface() override = default;
 
     void Export(
         ::grpc::ClientContext *context,
@@ -255,15 +255,15 @@ class OtlpGrpcLogRecordExporterTestPeer : public ::testing::Test
       const std::shared_ptr<OtlpGrpcClient> &client)
   {
     return std::unique_ptr<sdk::logs::LogRecordExporter>(
-        new OtlpGrpcLogRecordExporter(std::move(stub_interface), std::move(client)));
+        new OtlpGrpcLogRecordExporter(std::move(stub_interface), client));
   }
 
   std::unique_ptr<sdk::trace::SpanExporter> GetExporter(
       std::unique_ptr<proto::collector::trace::v1::TraceService::StubInterface> &stub_interface,
       const std::shared_ptr<OtlpGrpcClient> &client)
   {
     return std::unique_ptr<sdk::trace::SpanExporter>(
-        new OtlpGrpcExporter(std::move(stub_interface), std::move(client)));
+        new OtlpGrpcExporter(std::move(stub_interface), client));
   }
 
   // Get the options associated with the given exporter.
diff --git a/sdk/src/common/global_log_handler.cc b/sdk/src/common/global_log_handler.cc
@@ -18,11 +18,9 @@ namespace
 struct GlobalLogHandlerData
 {
   nostd::shared_ptr<LogHandler> handler;
-  LogLevel log_level;
+  LogLevel log_level{LogLevel::Warning};
 
-  GlobalLogHandlerData()
-      : handler(nostd::shared_ptr<LogHandler>(new DefaultLogHandler)), log_level(LogLevel::Warning)
-  {}
+  GlobalLogHandlerData() : handler(nostd::shared_ptr<LogHandler>(new DefaultLogHandler)) {}
   ~GlobalLogHandlerData() { is_singleton_destroyed = true; }
 
   GlobalLogHandlerData(const GlobalLogHandlerData &) = delete;
diff --git a/sdk/src/trace/span.cc b/sdk/src/trace/span.cc
@@ -172,7 +172,7 @@ void Span::AddLinks(const opentelemetry::trace::SpanContextKeyValueIterable &lin
     return;
   }
 
-  links.ForEachKeyValue([&](opentelemetry::trace::SpanContext span_context,
+  links.ForEachKeyValue([&](const opentelemetry::trace::SpanContext &span_context,
                             const common::KeyValueIterable &attributes) {
     recordable_->AddLink(span_context, attributes);
     return true;
diff --git a/sdk/test/common/circular_buffer_benchmark.cc b/sdk/test/common/circular_buffer_benchmark.cc
@@ -10,6 +10,7 @@
 #include <memory>
 #include <random>
 #include <thread>
+#include <utility>
 #include <vector>
 #include "benchmark/benchmark.h"
 
@@ -29,8 +30,10 @@ static uint64_t ConsumeBufferNumbers(BaselineCircularBuffer<uint64_t> &buffer) n
 {
   uint64_t result = 0;
   buffer.Consume([&](std::unique_ptr<uint64_t> &&x) {
-    result += *x;
-    x.reset();
+    auto val = std::move(x);
+
+    result += *val;
+    val.reset();
   });
   return result;
 }
diff --git a/sdk/test/logs/logger_provider_sdk_test.cc b/sdk/test/logs/logger_provider_sdk_test.cc
@@ -224,7 +224,10 @@ class DummyProcessor : public LogRecordProcessor
     return std::unique_ptr<Recordable>(new DummyLogRecordable());
   }
 
-  void OnEmit(std::unique_ptr<Recordable> && /* record */) noexcept override {}
+  void OnEmit(std::unique_ptr<Recordable> &&record) noexcept override
+  {
+    auto record_ptr = std::move(record);
+  }
   bool ForceFlush(std::chrono::microseconds /* timeout */) noexcept override { return true; }
   bool Shutdown(std::chrono::microseconds /* timeout */) noexcept override { return true; }
 };
diff --git a/sdk/test/metrics/metric_collector_test.cc b/sdk/test/metrics/metric_collector_test.cc

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ std::shared_ptr<InMemorySpanData> InitTracer()`
`54`	`54`	`trace_sdk::TracerProviderFactory::Create(std::move(processors));`
`55`	`55`
`56`	`56`	`// Set the global trace provider`
`57`		`- trace_sdk::Provider::SetTracerProvider(std::move(provider));`
	`57`	`+ trace_sdk::Provider::SetTracerProvider(provider);`
`58`	`58`
`59`	`59`	`return data;`
`60`	`60`	`}`
Original file line number	Diff line number	Diff line change
`@@ -103,46 +103,45 @@ class MyThreadInstrumentation : public opentelemetry::sdk::common::ThreadInstrum`
`103`	`103`	`{`
`104`	`104`	`std::cout << ", priority " << priority_;`
`105`	`105`	`}`
`106`		`- std::cout << std::endl << std::flush;`
	`106`	`+ std::cout << "\n" << std::flush;`
`107`	`107`	`}`
`108`	`108`
`109`	`109`	`void OnEnd() override`
`110`	`110`	`{`
`111`	`111`	`std::lock_guard<std::mutex> lock_guard(serialize);`
`112`		`- std::cout << "OnEnd() thread " << thread_name_ << ", id " << std::this_thread::get_id()`
`113`		`- << std::endl`
	`112`	`+ std::cout << "OnEnd() thread " << thread_name_ << ", id " << std::this_thread::get_id() << "\n"`
`114`	`113`	`<< std::flush;`
`115`	`114`	`}`
`116`	`115`
`117`	`116`	`void BeforeWait() override`
`118`	`117`	`{`
`119`	`118`	`std::lock_guard<std::mutex> lock_guard(serialize);`
`120`	`119`	`std::cout << "BeforeWait() thread " << thread_name_ << ", id " << std::this_thread::get_id()`
`121`		`- << ", waiting" << std::endl`
	`120`	`+ << ", waiting\n"`
`122`	`121`	`<< std::flush;`
`123`	`122`	`}`
`124`	`123`
`125`	`124`	`void AfterWait() override`
`126`	`125`	`{`
`127`	`126`	`std::lock_guard<std::mutex> lock_guard(serialize);`
`128`	`127`	`std::cout << "AfterWait() thread " << thread_name_ << ", id " << std::this_thread::get_id()`
`129`		`- << ", done waiting" << std::endl`
	`128`	`+ << ", done waiting\n"`
`130`	`129`	`<< std::flush;`
`131`	`130`	`}`
`132`	`131`
`133`	`132`	`void BeforeLoad() override`
`134`	`133`	`{`
`135`	`134`	`std::lock_guard<std::mutex> lock_guard(serialize);`
`136`	`135`	`std::cout << "BeforeLoad() thread " << thread_name_ << ", id " << std::this_thread::get_id()`
`137`		`- << ", about to work" << std::endl`
	`136`	`+ << ", about to work\n"`
`138`	`137`	`<< std::flush;`
`139`	`138`	`}`
`140`	`139`
`141`	`140`	`void AfterLoad() override`
`142`	`141`	`{`
`143`	`142`	`std::lock_guard<std::mutex> lock_guard(serialize);`
`144`	`143`	`std::cout << "AfterLoad() thread " << thread_name_ << ", id " << std::this_thread::get_id()`
`145`		`- << ", done working" << std::endl`
	`144`	`+ << ", done working\n"`
`146`	`145`	`<< std::flush;`
`147`	`146`	`}`
`148`	`147`
`@@ -341,25 +340,25 @@ int main(int argc, char *argv[])`
`341`	`340`	`logger_opts.url = "http://localhost:4318/v1/logs";`
`342`	`341`	`}`
`343`	`342`
`344`		`- std::cout << "Initializing opentelemetry-cpp" << std::endl << std::flush;`
	`343`	`+ std::cout << "Initializing opentelemetry-cpp\n" << std::flush;`
`345`	`344`
`346`	`345`	`InitTracer();`
`347`	`346`	`InitMetrics();`
`348`	`347`	`InitLogger();`
`349`	`348`
`350`		`- std::cout << "Application payload" << std::endl << std::flush;`
	`349`	`+ std::cout << "Application payload\n" << std::flush;`
`351`	`350`
`352`	`351`	`foo_library();`
`353`	`352`
`354`	`353`	`std::string name{"otlp_http_metric_example"};`
`355`	`354`	`foo_library::observable_counter_example(name);`
`356`	`355`
`357`		`- std::cout << "Shutting down opentelemetry-cpp" << std::endl << std::flush;`
	`356`	`+ std::cout << "Shutting down opentelemetry-cpp\n" << std::flush;`
`358`	`357`
`359`	`358`	`CleanupLogger();`
`360`	`359`	`CleanupMetrics();`
`361`	`360`	`CleanupTracer();`
`362`	`361`
`363`		`- std::cout << "Done" << std::endl << std::flush;`
	`362`	`+ std::cout << "Done\n" << std::flush;`
`364`	`363`	`return 0;`
`365`	`364`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1200,7 +1200,7 @@ class OPENTELEMETRY_LOCAL_SYMBOL OtlpFileSystemBackend : public OtlpFileAppender`
`1200`	`1200`	`{`
`1201`	`1201`	`if (file_pattern[i] == '%')`
`1202`	`1202`	`{`
`1203`		`- int checked = static_cast<int>(file_pattern[i + 1]);`
	`1203`	`+ int checked = static_cast<unsigned char>(file_pattern[i + 1]);`
`1204`	`1204`	`if (checked > 0 && checked < 128 && check_interval[checked] > 0)`
`1205`	`1205`	`{`
`1206`	`1206`	`if (0 == check_file_path_interval_ \|\|`
Original file line number	Diff line number	Diff line change
`@@ -665,7 +665,7 @@ void ConvertListFieldToJson(nlohmann::json &value,`
`665`	`665`	`OtlpHttpClient::OtlpHttpClient(OtlpHttpClientOptions &&options)`
`666`	`666`	`: is_shutdown_(false),`
`667`	`667`	`options_(std::move(options)),`
`668`		`- http_client_(http_client::HttpClientFactory::Create(options.thread_instrumentation)),`
	`668`	`+ http_client_(http_client::HttpClientFactory::Create(options_.thread_instrumentation)),`
`669`	`669`	`start_session_counter_(0),`
`670`	`670`	`finished_session_counter_(0)`
`671`	`671`	`{`