[CODE HEALTH] Fix clang-tidy performance-unnecessary-value-param warnings (#3931)

marcalff · web-flow · commit 86e5c6dcf76b · 2026-03-14T16:41:31.000+01:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -54,6 +54,9 @@ Increment the:
 * [EXPORTER] Allow custom HttpClient in OTLP HTTP
   [#3930](https://github.com/open-telemetry/opentelemetry-cpp/pull/3930)
 
+* [CODE HEALTH] Fix clang-tidy unnecessary value param warnings
+  [#3931](https://github.com/open-telemetry/opentelemetry-cpp/pull/3931)
+
 Important changes:
 
 * [BUILD] Revisit EventLogger deprecation
diff --git a/examples/grpc/tracer_common.h b/examples/grpc/tracer_common.h
@@ -100,7 +100,8 @@ void CleanupTracer()
   opentelemetry::sdk::trace::Provider::SetTracerProvider(none);
 }
 
-opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(std::string tracer_name)
+opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(
+    const std::string &tracer_name)
 {
   auto provider = opentelemetry::trace::Provider::GetTracerProvider();
   return provider->GetTracer(tracer_name);
diff --git a/examples/http/tracer_common.h b/examples/http/tracer_common.h
@@ -89,7 +89,8 @@ void CleanupTracer()
   opentelemetry::sdk::trace::Provider::SetTracerProvider(none);
 }
 
-opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(std::string tracer_name)
+opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(
+    const std::string &tracer_name)
 {
   auto provider = opentelemetry::trace::Provider::GetTracerProvider();
   return provider->GetTracer(tracer_name);
diff --git a/exporters/otlp/test/otlp_file_metric_exporter_test.cc b/exporters/otlp/test/otlp_file_metric_exporter_test.cc
@@ -56,7 +56,7 @@ class ProtobufGlobalSymbolGuard
 };
 
 template <class IntegerType>
-static IntegerType JsonToInteger(nlohmann::json value)
+static IntegerType JsonToInteger(const nlohmann::json &value)
 {
   if (value.is_string())
   {
diff --git a/exporters/otlp/test/otlp_grpc_exporter_test.cc b/exporters/otlp/test/otlp_grpc_exporter_test.cc
@@ -407,7 +407,8 @@ TEST_F(OtlpGrpcExporterTestPeer, ConfigRetryGenericValuesFromEnv)
 #  ifdef ENABLE_OTLP_RETRY_PREVIEW
 struct TestTraceService : public opentelemetry::proto::collector::trace::v1::TraceService::Service
 {
-  TestTraceService(std::vector<grpc::StatusCode> status_codes) : status_codes_(status_codes) {}
+  TestTraceService(const std::vector<grpc::StatusCode> &status_codes) : status_codes_(status_codes)
+  {}
 
   inline grpc::Status Export(
       grpc::ServerContext *,
diff --git a/exporters/otlp/test/otlp_grpc_metric_exporter_test.cc b/exporters/otlp/test/otlp_grpc_metric_exporter_test.cc
@@ -67,15 +67,15 @@ class OtlpGrpcMetricExporterTestPeer : public ::testing::Test
 
   std::unique_ptr<sdk::metrics::PushMetricExporter> GetExporter(
       std::unique_ptr<proto::collector::metrics::v1::MetricsService::StubInterface> stub_interface,
-      std::shared_ptr<OtlpGrpcClient> client)
+      const std::shared_ptr<OtlpGrpcClient> &client)
   {
     return std::unique_ptr<sdk::metrics::PushMetricExporter>(
         new OtlpGrpcMetricExporter(std::move(stub_interface), std::move(client)));
   }
 
   std::unique_ptr<sdk::metrics::PushMetricExporter> GetExporter(
       const OtlpGrpcMetricExporterOptions &options,
-      std::shared_ptr<OtlpGrpcClient> client)
+      const std::shared_ptr<OtlpGrpcClient> &client)
   {
     return std::unique_ptr<sdk::metrics::PushMetricExporter>(
         new OtlpGrpcMetricExporter(options, std::move(client)));
diff --git a/exporters/otlp/test/otlp_http_metric_exporter_test.cc b/exporters/otlp/test/otlp_http_metric_exporter_test.cc
@@ -62,7 +62,7 @@ namespace otlp
 {
 
 template <class IntegerType>
-static IntegerType JsonToInteger(nlohmann::json value)
+static IntegerType JsonToInteger(const nlohmann::json &value)
 {
   if (value.is_string())
   {
diff --git a/exporters/prometheus/include/opentelemetry/exporters/prometheus/exporter_utils.h b/exporters/prometheus/include/opentelemetry/exporters/prometheus/exporter_utils.h
@@ -162,7 +162,7 @@ class PrometheusExporterUtils
    * Counter => Prometheus Counter
    */
   template <typename T>
-  static void SetData(std::vector<T> values,
+  static void SetData(const std::vector<T> &values,
                       const opentelemetry::sdk::metrics::PointAttributes &labels,
                       const opentelemetry::sdk::instrumentationscope::InstrumentationScope *scope,
                       ::prometheus::MetricType type,
@@ -175,7 +175,7 @@ class PrometheusExporterUtils
    * Histogram => Prometheus Histogram
    */
   template <typename T>
-  static void SetData(std::vector<T> values,
+  static void SetData(const std::vector<T> &values,
                       const std::vector<double> &boundaries,
                       const std::vector<uint64_t> &counts,
                       const opentelemetry::sdk::metrics::PointAttributes &labels,
diff --git a/exporters/prometheus/src/exporter_utils.cc b/exporters/prometheus/src/exporter_utils.cc
@@ -641,7 +641,7 @@ void PrometheusExporterUtils::SetTarget(
  */
 template <typename T>
 void PrometheusExporterUtils::SetData(
-    std::vector<T> values,
+    const std::vector<T> &values,
     const metric_sdk::PointAttributes &labels,
     const opentelemetry::sdk::instrumentationscope::InstrumentationScope *scope,
     prometheus_client::MetricType type,
@@ -661,7 +661,7 @@ void PrometheusExporterUtils::SetData(
  */
 template <typename T>
 void PrometheusExporterUtils::SetData(
-    std::vector<T> values,
+    const std::vector<T> &values,
     const std::vector<double> &boundaries,
     const std::vector<uint64_t> &counts,
     const metric_sdk::PointAttributes &labels,
diff --git a/ext/include/opentelemetry/ext/http/server/http_server.h b/ext/include/opentelemetry/ext/http/server/http_server.h
@@ -63,11 +63,11 @@ class HttpRequestCallback
 
   virtual ~HttpRequestCallback() = default;
 
-  HttpRequestCallback(CallbackFunction func) : callback(func) {}
+  HttpRequestCallback(CallbackFunction func) : callback(std::move(func)) {}
 
   HttpRequestCallback &operator=(CallbackFunction func)
   {
-    callback = func;
+    callback = std::move(func);
     return (*this);
   }
 
@@ -133,7 +133,7 @@ class HttpServer : private SocketTools::Reactor::SocketCallback
       second = nullptr;
     }
 
-    HttpRequestHandler &operator=(std::pair<std::string, HttpRequestCallback *> other)
+    HttpRequestHandler &operator=(const std::pair<std::string, HttpRequestCallback *> &other)
     {
       first  = other.first;
       second = other.second;
diff --git a/ext/include/opentelemetry/ext/http/server/socket_tools.h b/ext/include/opentelemetry/ext/http/server/socket_tools.h
@@ -439,7 +439,7 @@ struct SocketData
 
   SocketData() : socket(), flags(0) {}
 
-  bool operator==(Socket s) { return (socket == s); }
+  bool operator==(const Socket &s) { return (socket == s); }
 };
 
 /// <summary>
diff --git a/sdk/include/opentelemetry/sdk/instrumentationscope/scope_configurator.h b/sdk/include/opentelemetry/sdk/instrumentationscope/scope_configurator.h
@@ -3,6 +3,7 @@
 
 #pragma once
 #include <functional>
+#include <utility>
 
 #include "opentelemetry/sdk/instrumentationscope/instrumentation_scope.h"
 #include "opentelemetry/version.h"
@@ -48,7 +49,7 @@ class ScopeConfigurator
     Builder &AddCondition(std::function<bool(const InstrumentationScope &)> scope_matcher,
                           T scope_config)
     {
-      conditions_.emplace_back(scope_matcher, scope_config);
+      conditions_.emplace_back(std::move(scope_matcher), scope_config);
       return *this;
     }
 
@@ -135,7 +136,7 @@ class ScopeConfigurator
 private:
   // Prevent direct initialization of ScopeConfigurator objects.
   explicit ScopeConfigurator(std::function<T(const InstrumentationScope &)> configurator)
-      : configurator_(configurator)
+      : configurator_(std::move(configurator))
   {}
 
   std::function<T(const InstrumentationScope &)> configurator_;
diff --git a/sdk/include/opentelemetry/sdk/metrics/aggregation/default_aggregation.h b/sdk/include/opentelemetry/sdk/metrics/aggregation/default_aggregation.h
@@ -63,7 +63,7 @@ class DefaultAggregation
 
   static std::unique_ptr<Aggregation> CreateAggregation(
       AggregationType aggregation_type,
-      InstrumentDescriptor instrument_descriptor,
+      const InstrumentDescriptor &instrument_descriptor,
       const AggregationConfig *aggregation_config = nullptr)
   {
     switch (aggregation_type)
diff --git a/sdk/include/opentelemetry/sdk/metrics/data/exemplar_data.h b/sdk/include/opentelemetry/sdk/metrics/data/exemplar_data.h
@@ -26,7 +26,7 @@ using MetricAttributes = opentelemetry::sdk::metrics::FilteredOrderedAttributeMa
 class ExemplarData
 {
 public:
-  static ExemplarData Create(std::shared_ptr<opentelemetry::trace::SpanContext> context,
+  static ExemplarData Create(const std::shared_ptr<opentelemetry::trace::SpanContext> &context,
                              const opentelemetry::common::SystemTimestamp &timestamp,
                              const PointDataAttributes &point_data_attr)
   {
@@ -68,7 +68,7 @@ class ExemplarData
   static PointType CreateDropPointData() { return DropPointData{}; }
 
 private:
-  ExemplarData(std::shared_ptr<opentelemetry::trace::SpanContext> context,
+  ExemplarData(const std::shared_ptr<opentelemetry::trace::SpanContext> &context,
                opentelemetry::common::SystemTimestamp timestamp,
                const PointDataAttributes &point_data_attr)
       : context_(*context.get()), timestamp_(timestamp), point_data_attr_(point_data_attr)
diff --git a/sdk/include/opentelemetry/sdk/metrics/exemplar/aligned_histogram_bucket_exemplar_reservoir.h b/sdk/include/opentelemetry/sdk/metrics/exemplar/aligned_histogram_bucket_exemplar_reservoir.h
@@ -44,7 +44,7 @@ class AlignedHistogramBucketExemplarReservoir : public FixedSizeExemplarReservoi
 
   AlignedHistogramBucketExemplarReservoir(
       size_t size,
-      std::shared_ptr<ReservoirCellSelector> reservoir_cell_selector,
+      const std::shared_ptr<ReservoirCellSelector> &reservoir_cell_selector,
       MapAndResetCellType map_and_reset_cell)
       : FixedSizeExemplarReservoir(size + 1, reservoir_cell_selector, map_and_reset_cell)
   {}
diff --git a/sdk/include/opentelemetry/sdk/metrics/exemplar/fixed_size_exemplar_reservoir.h b/sdk/include/opentelemetry/sdk/metrics/exemplar/fixed_size_exemplar_reservoir.h
@@ -27,7 +27,7 @@ class FixedSizeExemplarReservoir : public ExemplarReservoir
 
 public:
   FixedSizeExemplarReservoir(size_t size,
-                             std::shared_ptr<ReservoirCellSelector> reservoir_cell_selector,
+                             const std::shared_ptr<ReservoirCellSelector> &reservoir_cell_selector,
                              MapAndResetCellType map_and_reset_cell)
       : storage_(size),
         reservoir_cell_selector_(reservoir_cell_selector),
diff --git a/sdk/include/opentelemetry/sdk/metrics/exemplar/reservoir.h b/sdk/include/opentelemetry/sdk/metrics/exemplar/reservoir.h
@@ -79,12 +79,12 @@ class ExemplarReservoir
 
   static nostd::shared_ptr<ExemplarReservoir> GetSimpleFixedSizeExemplarReservoir(
       size_t size,
-      std::shared_ptr<ReservoirCellSelector> reservoir_cell_selector,
+      const std::shared_ptr<ReservoirCellSelector> &reservoir_cell_selector,
       MapAndResetCellType map_and_reset_cell);
 
   static nostd::shared_ptr<ExemplarReservoir> GetAlignedHistogramBucketExemplarReservoir(
       size_t size,
-      std::shared_ptr<ReservoirCellSelector> reservoir_cell_selector,
+      const std::shared_ptr<ReservoirCellSelector> &reservoir_cell_selector,
       MapAndResetCellType map_and_reset_cell);
 
   static nostd::shared_ptr<ExemplarReservoir> GetNoExemplarReservoir();
diff --git a/sdk/include/opentelemetry/sdk/metrics/exemplar/simple_fixed_size_exemplar_reservoir.h b/sdk/include/opentelemetry/sdk/metrics/exemplar/simple_fixed_size_exemplar_reservoir.h
@@ -43,9 +43,10 @@ class SimpleFixedSizeExemplarReservoir : public FixedSizeExemplarReservoir
     return std::shared_ptr<ReservoirCellSelector>{new SimpleFixedSizeCellSelector{size}};
   }
 
-  SimpleFixedSizeExemplarReservoir(size_t size,
-                                   std::shared_ptr<ReservoirCellSelector> reservoir_cell_selector,
-                                   MapAndResetCellType map_and_reset_cell)
+  SimpleFixedSizeExemplarReservoir(
+      size_t size,
+      const std::shared_ptr<ReservoirCellSelector> &reservoir_cell_selector,
+      MapAndResetCellType map_and_reset_cell)
       : FixedSizeExemplarReservoir(size, reservoir_cell_selector, map_and_reset_cell)
   {}
 
diff --git a/sdk/include/opentelemetry/sdk/metrics/export/metric_filter.h b/sdk/include/opentelemetry/sdk/metrics/export/metric_filter.h
@@ -63,13 +63,13 @@ class MetricFilter
       const PointAttributes &attributes)>;
 
   // static
-  static std::unique_ptr<MetricFilter> Create(TestMetricFn test_metric_fn,
-                                              TestAttributesFn test_attributes_fn)
+  static std::unique_ptr<MetricFilter> Create(const TestMetricFn &test_metric_fn,
+                                              const TestAttributesFn &test_attributes_fn)
   {
     return std::make_unique<MetricFilter>(test_metric_fn, test_attributes_fn);
   }
 
-  MetricFilter(TestMetricFn test_metric_fn, TestAttributesFn test_attributes_fn)
+  MetricFilter(const TestMetricFn &test_metric_fn, const TestAttributesFn &test_attributes_fn)
       : test_metric_fn_(test_metric_fn), test_attributes_fn_(test_attributes_fn)
   {}
 
diff --git a/sdk/include/opentelemetry/sdk/metrics/state/async_metric_storage.h b/sdk/include/opentelemetry/sdk/metrics/state/async_metric_storage.h
@@ -34,7 +34,7 @@ namespace metrics
 class AsyncMetricStorage : public MetricStorage, public AsyncWritableMetricStorage
 {
 public:
-  AsyncMetricStorage(InstrumentDescriptor instrument_descriptor,
+  AsyncMetricStorage(const InstrumentDescriptor &instrument_descriptor,
                      const AggregationType aggregation_type,
 #ifdef ENABLE_METRICS_EXEMPLAR_PREVIEW
                      ExemplarFilterType exempler_filter_type,
diff --git a/sdk/include/opentelemetry/sdk/metrics/state/multi_metric_storage.h b/sdk/include/opentelemetry/sdk/metrics/state/multi_metric_storage.h
@@ -25,7 +25,7 @@ namespace metrics
 class SyncMultiMetricStorage : public SyncWritableMetricStorage
 {
 public:
-  void AddStorage(std::shared_ptr<SyncWritableMetricStorage> storage)
+  void AddStorage(const std::shared_ptr<SyncWritableMetricStorage> &storage)
   {
     storages_.push_back(storage);
   }
@@ -73,7 +73,7 @@ class SyncMultiMetricStorage : public SyncWritableMetricStorage
 class AsyncMultiMetricStorage : public AsyncWritableMetricStorage
 {
 public:
-  void AddStorage(std::shared_ptr<AsyncWritableMetricStorage> storage)
+  void AddStorage(const std::shared_ptr<AsyncWritableMetricStorage> &storage)
   {
     storages_.push_back(storage);
   }
diff --git a/sdk/include/opentelemetry/sdk/metrics/state/sync_metric_storage.h b/sdk/include/opentelemetry/sdk/metrics/state/sync_metric_storage.h
@@ -57,7 +57,7 @@ class SyncMetricStorage : public MetricStorage, public SyncWritableMetricStorage
 #endif  // ENABLE_METRICS_EXEMPLAR_PREVIEW
 
 public:
-  SyncMetricStorage(InstrumentDescriptor instrument_descriptor,
+  SyncMetricStorage(const InstrumentDescriptor &instrument_descriptor,
                     const AggregationType aggregation_type,
                     std::shared_ptr<const AttributesProcessor> attributes_processor,
 #ifdef ENABLE_METRICS_EXEMPLAR_PREVIEW
diff --git a/sdk/include/opentelemetry/sdk/metrics/sync_instruments.h b/sdk/include/opentelemetry/sdk/metrics/sync_instruments.h
@@ -23,7 +23,7 @@ namespace metrics
 class Synchronous
 {
 public:
-  Synchronous(InstrumentDescriptor instrument_descriptor,
+  Synchronous(const InstrumentDescriptor &instrument_descriptor,
               std::unique_ptr<SyncWritableMetricStorage> storage)
       : instrument_descriptor_(instrument_descriptor), storage_(std::move(storage))
   {}
diff --git a/sdk/include/opentelemetry/sdk/metrics/view/view.h b/sdk/include/opentelemetry/sdk/metrics/view/view.h
@@ -25,9 +25,9 @@ class View
 {
 public:
   View(const std::string &name,
-       const std::string &description                        = "",
-       AggregationType aggregation_type                      = AggregationType::kDefault,
-       std::shared_ptr<AggregationConfig> aggregation_config = nullptr,
+       const std::string &description                               = "",
+       AggregationType aggregation_type                             = AggregationType::kDefault,
+       const std::shared_ptr<AggregationConfig> &aggregation_config = nullptr,
        std::unique_ptr<opentelemetry::sdk::metrics::AttributesProcessor> attributes_processor =
            std::unique_ptr<opentelemetry::sdk::metrics::AttributesProcessor>(
                new opentelemetry::sdk::metrics::DefaultAttributesProcessor()))
diff --git a/sdk/include/opentelemetry/sdk/metrics/view/view_factory.h b/sdk/include/opentelemetry/sdk/metrics/view/view_factory.h
@@ -35,12 +35,12 @@ class OPENTELEMETRY_EXPORT ViewFactory
   static std::unique_ptr<View> Create(const std::string &name,
                                       const std::string &description,
                                       AggregationType aggregation_type,
-                                      std::shared_ptr<AggregationConfig> aggregation_config);
+                                      const std::shared_ptr<AggregationConfig> &aggregation_config);
 
   static std::unique_ptr<View> Create(const std::string &name,
                                       const std::string &description,
                                       AggregationType aggregation_type,
-                                      std::shared_ptr<AggregationConfig> aggregation_config,
+                                      const std::shared_ptr<AggregationConfig> &aggregation_config,
                                       std::unique_ptr<AttributesProcessor> attributes_processor);
 };
 
diff --git a/sdk/include/opentelemetry/sdk/trace/recordable.h b/sdk/include/opentelemetry/sdk/trace/recordable.h
@@ -123,7 +123,7 @@ class Recordable
    * Add a link to a span with default (empty) attributes.
    * @param span_context the span context of the linked span
    */
-  void AddLink(opentelemetry::trace::SpanContext span_context)
+  void AddLink(const opentelemetry::trace::SpanContext &span_context)
   {
     AddLink(span_context, opentelemetry::sdk::GetEmptyAttributes());
   }
diff --git a/sdk/include/opentelemetry/sdk/trace/span_data.h b/sdk/include/opentelemetry/sdk/trace/span_data.h
@@ -8,6 +8,7 @@
 #include <memory>
 #include <string>
 #include <unordered_map>
+#include <utility>
 #include <vector>
 
 #include "opentelemetry/common/attribute_value.h"
@@ -40,7 +41,7 @@ class SpanDataEvent
   SpanDataEvent(std::string name,
                 opentelemetry::common::SystemTimestamp timestamp,
                 const opentelemetry::common::KeyValueIterable &attributes)
-      : name_(name), timestamp_(timestamp), attribute_map_(attributes)
+      : name_(std::move(name)), timestamp_(timestamp), attribute_map_(attributes)
   {}
 
   /**
@@ -79,7 +80,7 @@ class SpanDataLink
 public:
   SpanDataLink(opentelemetry::trace::SpanContext span_context,
                const opentelemetry::common::KeyValueIterable &attributes)
-      : span_context_(span_context), attribute_map_(attributes)
+      : span_context_(std::move(span_context)), attribute_map_(attributes)
   {}
 
   /**
diff --git a/sdk/src/metrics/exemplar/reservoir.cc b/sdk/src/metrics/exemplar/reservoir.cc
diff --git a/sdk/src/metrics/view/view_factory.cc b/sdk/src/metrics/view/view_factory.cc
diff --git a/test_common/include/opentelemetry/test_common/ext/http/client/nosend/http_client_nosend.h b/test_common/include/opentelemetry/test_common/ext/http/client/nosend/http_client_nosend.h

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,8 @@ void CleanupTracer()`
`100`	`100`	`opentelemetry::sdk::trace::Provider::SetTracerProvider(none);`
`101`	`101`	`}`
`102`	`102`
`103`		`-opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(std::string tracer_name)`
	`103`	`+opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(`
	`104`	`+ const std::string &tracer_name)`
`104`	`105`	`{`
`105`	`106`	`auto provider = opentelemetry::trace::Provider::GetTracerProvider();`
`106`	`107`	`return provider->GetTracer(tracer_name);`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,8 @@ void CleanupTracer()`
`89`	`89`	`opentelemetry::sdk::trace::Provider::SetTracerProvider(none);`
`90`	`90`	`}`
`91`	`91`
`92`		`-opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(std::string tracer_name)`
	`92`	`+opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> get_tracer(`
	`93`	`+ const std::string &tracer_name)`
`93`	`94`	`{`
`94`	`95`	`auto provider = opentelemetry::trace::Provider::GetTracerProvider();`
`95`	`96`	`return provider->GetTracer(tracer_name);`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ class ProtobufGlobalSymbolGuard`
`56`	`56`	`};`
`57`	`57`
`58`	`58`	`template <class IntegerType>`
`59`		`-static IntegerType JsonToInteger(nlohmann::json value)`
	`59`	`+static IntegerType JsonToInteger(const nlohmann::json &value)`
`60`	`60`	`{`
`61`	`61`	`if (value.is_string())`
`62`	`62`	`{`
Original file line number	Diff line number	Diff line change
`@@ -407,7 +407,8 @@ TEST_F(OtlpGrpcExporterTestPeer, ConfigRetryGenericValuesFromEnv)`
`407`	`407`	`# ifdef ENABLE_OTLP_RETRY_PREVIEW`
`408`	`408`	`struct TestTraceService : public opentelemetry::proto::collector::trace::v1::TraceService::Service`
`409`	`409`	`{`
`410`		`- TestTraceService(std::vector<grpc::StatusCode> status_codes) : status_codes_(status_codes) {}`
	`410`	`+ TestTraceService(const std::vector<grpc::StatusCode> &status_codes) : status_codes_(status_codes)`
	`411`	`+ {}`
`411`	`412`
`412`	`413`	`inline grpc::Status Export(`
`413`	`414`	`grpc::ServerContext *,`
Original file line number	Diff line number	Diff line change
`@@ -67,15 +67,15 @@ class OtlpGrpcMetricExporterTestPeer : public ::testing::Test`
`67`	`67`
`68`	`68`	`std::unique_ptr<sdk::metrics::PushMetricExporter> GetExporter(`
`69`	`69`	`std::unique_ptr<proto::collector::metrics::v1::MetricsService::StubInterface> stub_interface,`
`70`		`- std::shared_ptr<OtlpGrpcClient> client)`
	`70`	`+ const std::shared_ptr<OtlpGrpcClient> &client)`
`71`	`71`	`{`
`72`	`72`	`return std::unique_ptr<sdk::metrics::PushMetricExporter>(`
`73`	`73`	`new OtlpGrpcMetricExporter(std::move(stub_interface), std::move(client)));`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`std::unique_ptr<sdk::metrics::PushMetricExporter> GetExporter(`
`77`	`77`	`const OtlpGrpcMetricExporterOptions &options,`
`78`		`- std::shared_ptr<OtlpGrpcClient> client)`
	`78`	`+ const std::shared_ptr<OtlpGrpcClient> &client)`
`79`	`79`	`{`
`80`	`80`	`return std::unique_ptr<sdk::metrics::PushMetricExporter>(`
`81`	`81`	`new OtlpGrpcMetricExporter(options, std::move(client)));`
Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ namespace otlp`
`62`	`62`	`{`
`63`	`63`
`64`	`64`	`template <class IntegerType>`
`65`		`-static IntegerType JsonToInteger(nlohmann::json value)`
	`65`	`+static IntegerType JsonToInteger(const nlohmann::json &value)`
`66`	`66`	`{`
`67`	`67`	`if (value.is_string())`
`68`	`68`	`{`
Original file line number	Diff line number	Diff line change
`@@ -63,11 +63,11 @@ class HttpRequestCallback`
`63`	`63`
`64`	`64`	`virtual ~HttpRequestCallback() = default;`
`65`	`65`
`66`		`- HttpRequestCallback(CallbackFunction func) : callback(func) {}`
	`66`	`+ HttpRequestCallback(CallbackFunction func) : callback(std::move(func)) {}`
`67`	`67`
`68`	`68`	`HttpRequestCallback &operator=(CallbackFunction func)`
`69`	`69`	`{`
`70`		`- callback = func;`
	`70`	`+ callback = std::move(func);`
`71`	`71`	`return (*this);`
`72`	`72`	`}`
`73`	`73`
`@@ -133,7 +133,7 @@ class HttpServer : private SocketTools::Reactor::SocketCallback`
`133`	`133`	`second = nullptr;`
`134`	`134`	`}`
`135`	`135`
`136`		`- HttpRequestHandler &operator=(std::pair<std::string, HttpRequestCallback *> other)`
	`136`	`+ HttpRequestHandler &operator=(const std::pair<std::string, HttpRequestCallback *> &other)`
`137`	`137`	`{`
`138`	`138`	`first = other.first;`
`139`	`139`	`second = other.second;`