Skip to content

Commit 3b42db5

Browse files
authored
[CodeHealth] Fix clang-tidy warnings part 5 (#3506)
1 parent dff39f3 commit 3b42db5

16 files changed

+38
-28
lines changed

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_client.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,10 @@ class OtlpFileClient
4040
OtlpFileClientRuntimeOptions &&runtime_options);
4141

4242
~OtlpFileClient();
43+
OtlpFileClient(const OtlpFileClient &) = delete;
44+
OtlpFileClient &operator=(const OtlpFileClient &) = delete;
45+
OtlpFileClient(OtlpFileClient &&) = delete;
46+
OtlpFileClient &operator=(OtlpFileClient &&) = delete;
4347

4448
/**
4549
* Sync export

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_client_options.h

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ struct OtlpFileClientFileSystemOptions
6868
// Maximum file count
6969
std::size_t rotate_size = 3;
7070

71-
inline OtlpFileClientFileSystemOptions() noexcept {}
71+
OtlpFileClientFileSystemOptions() = default;
7272
};
7373

7474
/**
@@ -77,7 +77,12 @@ struct OtlpFileClientFileSystemOptions
7777
class OtlpFileAppender
7878
{
7979
public:
80-
virtual ~OtlpFileAppender() = default;
80+
OtlpFileAppender() = default;
81+
virtual ~OtlpFileAppender() = default;
82+
OtlpFileAppender(const OtlpFileAppender &) = default;
83+
OtlpFileAppender &operator=(const OtlpFileAppender &) = default;
84+
OtlpFileAppender(OtlpFileAppender &&) = default;
85+
OtlpFileAppender &operator=(OtlpFileAppender &&) = default;
8186

8287
virtual void Export(opentelemetry::nostd::string_view data, std::size_t record_count) = 0;
8388

@@ -96,12 +101,17 @@ using OtlpFileClientBackendOptions =
96101
*/
97102
struct OtlpFileClientOptions
98103
{
104+
OtlpFileClientOptions() = default;
105+
virtual ~OtlpFileClientOptions() = default;
106+
OtlpFileClientOptions(const OtlpFileClientOptions &) = default;
107+
OtlpFileClientOptions &operator=(const OtlpFileClientOptions &) = default;
108+
OtlpFileClientOptions(OtlpFileClientOptions &&) = default;
109+
OtlpFileClientOptions &operator=(OtlpFileClientOptions &&) = default;
110+
99111
// Whether to print the status of the FILE client in the console
100112
bool console_debug = false;
101113

102114
OtlpFileClientBackendOptions backend_options;
103-
104-
inline OtlpFileClientOptions() noexcept {}
105115
};
106116
} // namespace otlp
107117
} // namespace exporter

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_client_runtime_options.h

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,13 @@ namespace otlp
1919
*/
2020
struct OtlpFileClientRuntimeOptions
2121
{
22+
OtlpFileClientRuntimeOptions() = default;
23+
virtual ~OtlpFileClientRuntimeOptions() = default;
24+
OtlpFileClientRuntimeOptions(const OtlpFileClientRuntimeOptions &) = default;
25+
OtlpFileClientRuntimeOptions &operator=(const OtlpFileClientRuntimeOptions &) = default;
26+
OtlpFileClientRuntimeOptions(OtlpFileClientRuntimeOptions &&) = default;
27+
OtlpFileClientRuntimeOptions &operator=(OtlpFileClientRuntimeOptions &&) = default;
28+
2229
std::shared_ptr<sdk::common::ThreadInstrumentation> thread_instrumentation =
2330
std::shared_ptr<sdk::common::ThreadInstrumentation>(nullptr);
2431
};

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_exporter.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,9 @@ class OPENTELEMETRY_EXPORT OtlpFileExporter final : public opentelemetry::sdk::t
7676

7777
private:
7878
// The configuration options associated with this exporter.
79-
const OtlpFileExporterOptions options_;
79+
OtlpFileExporterOptions options_;
8080
// The runtime options associated with this exporter.
81-
const OtlpFileExporterRuntimeOptions runtime_options_;
81+
OtlpFileExporterRuntimeOptions runtime_options_;
8282

8383
// Object that stores the file context.
8484
std::unique_ptr<OtlpFileClient> file_client_;

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_exporter_options.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ namespace otlp
2222
struct OPENTELEMETRY_EXPORT OtlpFileExporterOptions : public OtlpFileClientOptions
2323
{
2424
OtlpFileExporterOptions();
25-
~OtlpFileExporterOptions();
2625
};
2726

2827
} // namespace otlp

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_exporter_runtime_options.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@ namespace otlp
1818
*/
1919
struct OPENTELEMETRY_EXPORT OtlpFileExporterRuntimeOptions : public OtlpFileClientRuntimeOptions
2020
{
21-
OtlpFileExporterRuntimeOptions() = default;
22-
~OtlpFileExporterRuntimeOptions() = default;
21+
OtlpFileExporterRuntimeOptions() = default;
2322
};
2423

2524
} // namespace otlp

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_log_record_exporter.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,9 +77,9 @@ class OtlpFileLogRecordExporter final : public opentelemetry::sdk::logs::LogReco
7777

7878
private:
7979
// Configuration options for the exporter
80-
const OtlpFileLogRecordExporterOptions options_;
80+
OtlpFileLogRecordExporterOptions options_;
8181
// Runtime options for the exporter
82-
const OtlpFileLogRecordExporterRuntimeOptions runtime_options_;
82+
OtlpFileLogRecordExporterRuntimeOptions runtime_options_;
8383

8484
// Object that stores the file context.
8585
std::unique_ptr<OtlpFileClient> file_client_;

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_log_record_exporter_options.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ namespace otlp
2222
struct OPENTELEMETRY_EXPORT OtlpFileLogRecordExporterOptions : public OtlpFileClientOptions
2323
{
2424
OtlpFileLogRecordExporterOptions();
25-
~OtlpFileLogRecordExporterOptions();
2625
};
2726

2827
} // namespace otlp

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_log_record_exporter_runtime_options.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,7 @@ namespace otlp
1818
struct OPENTELEMETRY_EXPORT OtlpFileLogRecordExporterRuntimeOptions
1919
: public OtlpFileClientRuntimeOptions
2020
{
21-
OtlpFileLogRecordExporterRuntimeOptions() = default;
22-
~OtlpFileLogRecordExporterRuntimeOptions() = default;
21+
OtlpFileLogRecordExporterRuntimeOptions() = default;
2322
};
2423

2524
} // namespace otlp

exporters/otlp/include/opentelemetry/exporters/otlp/otlp_file_metric_exporter.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,12 +69,12 @@ class OtlpFileMetricExporter final : public opentelemetry::sdk::metrics::PushMet
6969
friend class OtlpFileMetricExporterTestPeer;
7070

7171
// Configuration options for the exporter
72-
const OtlpFileMetricExporterOptions options_;
72+
OtlpFileMetricExporterOptions options_;
7373
// Runtime options for the exporter
74-
const OtlpFileMetricExporterRuntimeOptions runtime_options_;
74+
OtlpFileMetricExporterRuntimeOptions runtime_options_;
7575

7676
// Aggregation Temporality Selector
77-
const sdk::metrics::AggregationTemporalitySelector aggregation_temporality_selector_;
77+
sdk::metrics::AggregationTemporalitySelector aggregation_temporality_selector_;
7878

7979
// Object that stores the file context.
8080
std::unique_ptr<OtlpFileClient> file_client_;

0 commit comments

Comments
 (0)