Skip to content

Commit db8c018

Browse files
chusitoochusitoo
committed
Fix warnings as errors
1 parent feb6a68 commit db8c018

File tree

8 files changed

+60
-60
lines changed

8 files changed

+60
-60
lines changed

‎exporters/otlp/test/otlp_grpc_exporter_test.cc‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -359,9 +359,9 @@ TEST_F(OtlpGrpcExporterTestPeer, ConfigRetryDefaultValues)
359359
std::unique_ptr<OtlpGrpcExporter> exporter(new OtlpGrpcExporter());
360360
const auto options = GetOptions(exporter);
361361
ASSERT_EQ(options.retry_policy_max_attempts, 5);
362-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0);
363-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0);
364-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5);
362+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0f);
363+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0f);
364+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5f);
365365
}
366366

367367
TEST_F(OtlpGrpcExporterTestPeer, ConfigRetryValuesFromEnv)
@@ -374,9 +374,9 @@ TEST_F(OtlpGrpcExporterTestPeer, ConfigRetryValuesFromEnv)
374374
std::unique_ptr<OtlpGrpcExporter> exporter(new OtlpGrpcExporter());
375375
const auto options = GetOptions(exporter);
376376
ASSERT_EQ(options.retry_policy_max_attempts, 123);
377-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5);
378-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7);
379-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9);
377+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5f);
378+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7f);
379+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9f);
380380

381381
unsetenv("OTEL_CPP_EXPORTER_OTLP_TRACES_RETRY_MAX_ATTEMPTS");
382382
unsetenv("OTEL_CPP_EXPORTER_OTLP_TRACES_RETRY_INITIAL_BACKOFF");
@@ -394,9 +394,9 @@ TEST_F(OtlpGrpcExporterTestPeer, ConfigRetryGenericValuesFromEnv)
394394
std::unique_ptr<OtlpGrpcExporter> exporter(new OtlpGrpcExporter());
395395
const auto options = GetOptions(exporter);
396396
ASSERT_EQ(options.retry_policy_max_attempts, 321);
397-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4);
398-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6);
399-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8);
397+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4f);
398+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6f);
399+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8f);
400400

401401
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_MAX_ATTEMPTS");
402402
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_INITIAL_BACKOFF");

‎exporters/otlp/test/otlp_grpc_log_record_exporter_test.cc‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -472,9 +472,9 @@ TEST_F(OtlpGrpcLogRecordExporterTestPeer, ConfigRetryDefaultValues)
472472
std::unique_ptr<OtlpGrpcLogRecordExporter> exporter(new OtlpGrpcLogRecordExporter());
473473
const auto options = GetOptions(exporter);
474474
ASSERT_EQ(options.retry_policy_max_attempts, 5);
475-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0);
476-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0);
477-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5);
475+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0f);
476+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0f);
477+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5f);
478478
}
479479

480480
TEST_F(OtlpGrpcLogRecordExporterTestPeer, ConfigRetryValuesFromEnv)
@@ -487,9 +487,9 @@ TEST_F(OtlpGrpcLogRecordExporterTestPeer, ConfigRetryValuesFromEnv)
487487
std::unique_ptr<OtlpGrpcLogRecordExporter> exporter(new OtlpGrpcLogRecordExporter());
488488
const auto options = GetOptions(exporter);
489489
ASSERT_EQ(options.retry_policy_max_attempts, 123);
490-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5);
491-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7);
492-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9);
490+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5f);
491+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7f);
492+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9f);
493493

494494
unsetenv("OTEL_CPP_EXPORTER_OTLP_LOGS_RETRY_MAX_ATTEMPTS");
495495
unsetenv("OTEL_CPP_EXPORTER_OTLP_LOGS_RETRY_INITIAL_BACKOFF");
@@ -507,9 +507,9 @@ TEST_F(OtlpGrpcLogRecordExporterTestPeer, ConfigRetryGenericValuesFromEnv)
507507
std::unique_ptr<OtlpGrpcLogRecordExporter> exporter(new OtlpGrpcLogRecordExporter());
508508
const auto options = GetOptions(exporter);
509509
ASSERT_EQ(options.retry_policy_max_attempts, 321);
510-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4);
511-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6);
512-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8);
510+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4f);
511+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6f);
512+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8f);
513513

514514
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_MAX_ATTEMPTS");
515515
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_INITIAL_BACKOFF");

‎exporters/otlp/test/otlp_grpc_metric_exporter_test.cc‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -225,9 +225,9 @@ TEST_F(OtlpGrpcMetricExporterTestPeer, ConfigRetryDefaultValues)
225225
std::unique_ptr<OtlpGrpcMetricExporter> exporter(new OtlpGrpcMetricExporter());
226226
const auto options = GetOptions(exporter);
227227
ASSERT_EQ(options.retry_policy_max_attempts, 5);
228-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0);
229-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0);
230-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5);
228+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0f);
229+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0f);
230+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5f);
231231
}
232232

233233
TEST_F(OtlpGrpcMetricExporterTestPeer, ConfigRetryValuesFromEnv)
@@ -240,9 +240,9 @@ TEST_F(OtlpGrpcMetricExporterTestPeer, ConfigRetryValuesFromEnv)
240240
std::unique_ptr<OtlpGrpcMetricExporter> exporter(new OtlpGrpcMetricExporter());
241241
const auto options = GetOptions(exporter);
242242
ASSERT_EQ(options.retry_policy_max_attempts, 123);
243-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5);
244-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7);
245-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9);
243+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5f);
244+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7f);
245+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9f);
246246

247247
unsetenv("OTEL_CPP_EXPORTER_OTLP_METRICS_RETRY_MAX_ATTEMPTS");
248248
unsetenv("OTEL_CPP_EXPORTER_OTLP_METRICS_RETRY_INITIAL_BACKOFF");
@@ -260,9 +260,9 @@ TEST_F(OtlpGrpcMetricExporterTestPeer, ConfigRetryGenericValuesFromEnv)
260260
std::unique_ptr<OtlpGrpcMetricExporter> exporter(new OtlpGrpcMetricExporter());
261261
const auto options = GetOptions(exporter);
262262
ASSERT_EQ(options.retry_policy_max_attempts, 321);
263-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4);
264-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6);
265-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8);
263+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4f);
264+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6f);
265+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8f);
266266

267267
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_MAX_ATTEMPTS");
268268
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_INITIAL_BACKOFF");

‎exporters/otlp/test/otlp_http_exporter_test.cc‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -640,9 +640,9 @@ TEST_F(OtlpHttpExporterTestPeer, ConfigRetryDefaultValues)
640640
std::unique_ptr<OtlpHttpExporter> exporter(new OtlpHttpExporter());
641641
const auto options = GetOptions(exporter);
642642
ASSERT_EQ(options.retry_policy_max_attempts, 5);
643-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0);
644-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0);
645-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5);
643+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0f);
644+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0f);
645+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5f);
646646
}
647647

648648
TEST_F(OtlpHttpExporterTestPeer, ConfigRetryValuesFromEnv)
@@ -655,9 +655,9 @@ TEST_F(OtlpHttpExporterTestPeer, ConfigRetryValuesFromEnv)
655655
std::unique_ptr<OtlpHttpExporter> exporter(new OtlpHttpExporter());
656656
const auto options = GetOptions(exporter);
657657
ASSERT_EQ(options.retry_policy_max_attempts, 123);
658-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5);
659-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7);
660-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9);
658+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5f);
659+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7f);
660+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9f);
661661

662662
unsetenv("OTEL_CPP_EXPORTER_OTLP_TRACES_RETRY_MAX_ATTEMPTS");
663663
unsetenv("OTEL_CPP_EXPORTER_OTLP_TRACES_RETRY_INITIAL_BACKOFF");
@@ -675,9 +675,9 @@ TEST_F(OtlpHttpExporterTestPeer, ConfigRetryGenericValuesFromEnv)
675675
std::unique_ptr<OtlpHttpExporter> exporter(new OtlpHttpExporter());
676676
const auto options = GetOptions(exporter);
677677
ASSERT_EQ(options.retry_policy_max_attempts, 321);
678-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4);
679-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6);
680-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8);
678+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4f);
679+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6f);
680+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8f);
681681

682682
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_MAX_ATTEMPTS");
683683
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_INITIAL_BACKOFF");

‎exporters/otlp/test/otlp_http_log_record_exporter_test.cc‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -762,9 +762,9 @@ TEST_F(OtlpHttpLogRecordExporterTestPeer, ConfigRetryDefaultValues)
762762
std::unique_ptr<OtlpHttpLogRecordExporter> exporter(new OtlpHttpLogRecordExporter());
763763
const auto options = GetOptions(exporter);
764764
ASSERT_EQ(options.retry_policy_max_attempts, 5);
765-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0);
766-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0);
767-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5);
765+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0f);
766+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0f);
767+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5f);
768768
}
769769

770770
TEST_F(OtlpHttpLogRecordExporterTestPeer, ConfigRetryValuesFromEnv)
@@ -777,9 +777,9 @@ TEST_F(OtlpHttpLogRecordExporterTestPeer, ConfigRetryValuesFromEnv)
777777
std::unique_ptr<OtlpHttpLogRecordExporter> exporter(new OtlpHttpLogRecordExporter());
778778
const auto options = GetOptions(exporter);
779779
ASSERT_EQ(options.retry_policy_max_attempts, 123);
780-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5);
781-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7);
782-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9);
780+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5f);
781+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7f);
782+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9f);
783783

784784
unsetenv("OTEL_CPP_EXPORTER_OTLP_LOGS_RETRY_MAX_ATTEMPTS");
785785
unsetenv("OTEL_CPP_EXPORTER_OTLP_LOGS_RETRY_INITIAL_BACKOFF");
@@ -797,9 +797,9 @@ TEST_F(OtlpHttpLogRecordExporterTestPeer, ConfigRetryGenericValuesFromEnv)
797797
std::unique_ptr<OtlpHttpLogRecordExporter> exporter(new OtlpHttpLogRecordExporter());
798798
const auto options = GetOptions(exporter);
799799
ASSERT_EQ(options.retry_policy_max_attempts, 321);
800-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4);
801-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6);
802-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8);
800+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4f);
801+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6f);
802+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8f);
803803

804804
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_MAX_ATTEMPTS");
805805
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_INITIAL_BACKOFF");

‎exporters/otlp/test/otlp_http_metric_exporter_test.cc‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1017,9 +1017,9 @@ TEST_F(OtlpHttpMetricExporterTestPeer, ConfigRetryDefaultValues)
10171017
std::unique_ptr<OtlpHttpMetricExporter> exporter(new OtlpHttpMetricExporter());
10181018
const auto options = GetOptions(exporter);
10191019
ASSERT_EQ(options.retry_policy_max_attempts, 5);
1020-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0);
1021-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0);
1022-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5);
1020+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 1.0f);
1021+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 5.0f);
1022+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 1.5f);
10231023
}
10241024

10251025
TEST_F(OtlpHttpMetricExporterTestPeer, ConfigRetryValuesFromEnv)
@@ -1032,9 +1032,9 @@ TEST_F(OtlpHttpMetricExporterTestPeer, ConfigRetryValuesFromEnv)
10321032
std::unique_ptr<OtlpHttpMetricExporter> exporter(new OtlpHttpMetricExporter());
10331033
const auto options = GetOptions(exporter);
10341034
ASSERT_EQ(options.retry_policy_max_attempts, 123);
1035-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5);
1036-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7);
1037-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9);
1035+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 4.5f);
1036+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 6.7f);
1037+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 8.9f);
10381038

10391039
unsetenv("OTEL_CPP_EXPORTER_OTLP_METRICS_RETRY_MAX_ATTEMPTS");
10401040
unsetenv("OTEL_CPP_EXPORTER_OTLP_METRICS_RETRY_INITIAL_BACKOFF");
@@ -1052,9 +1052,9 @@ TEST_F(OtlpHttpMetricExporterTestPeer, ConfigRetryGenericValuesFromEnv)
10521052
std::unique_ptr<OtlpHttpMetricExporter> exporter(new OtlpHttpMetricExporter());
10531053
const auto options = GetOptions(exporter);
10541054
ASSERT_EQ(options.retry_policy_max_attempts, 321);
1055-
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4);
1056-
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6);
1057-
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8);
1055+
ASSERT_FLOAT_EQ(options.retry_policy_initial_backoff.count(), 5.4f);
1056+
ASSERT_FLOAT_EQ(options.retry_policy_max_backoff.count(), 7.6f);
1057+
ASSERT_FLOAT_EQ(options.retry_policy_backoff_multiplier, 9.8f);
10581058

10591059
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_MAX_ATTEMPTS");
10601060
unsetenv("OTEL_CPP_EXPORTER_OTLP_RETRY_INITIAL_BACKOFF");

‎ext/src/http/client/curl/http_client_curl.cc‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -475,7 +475,7 @@ bool HttpClient::MaybeSpawnBackgroundThread()
475475
#endif /* ENABLE_THREAD_INSTRUMENTATION_PREVIEW */
476476

477477
// curl_multi_poll is added from libcurl 7.66.0, before 7.68.0, we can only wait until
478-
// timeout to do the rest jobs
478+
// timeout to do the remaining jobs
479479
#if LIBCURL_VERSION_NUM >= 0x074200
480480
/* wait for activity, timeout or "nothing" */
481481
mc = curl_multi_poll(self->multi_handle_, nullptr, 0,
@@ -646,16 +646,16 @@ void HttpClient::ScheduleAddSession(uint64_t session_id)
646646
void HttpClient::ScheduleAbortSession(uint64_t session_id)
647647
{
648648
{
649-
std::lock_guard<std::mutex> lock_guard{sessions_m_};
649+
std::lock_guard<std::mutex> sessions_lock{sessions_m_};
650650
auto session = sessions_.find(session_id);
651651
if (session == sessions_.end())
652652
{
653-
std::lock_guard<std::recursive_mutex> lock_guard{session_ids_m_};
653+
std::lock_guard<std::recursive_mutex> session_ids_lock{session_ids_m_};
654654
pending_to_add_session_ids_.erase(session_id);
655655
}
656656
else
657657
{
658-
std::lock_guard<std::recursive_mutex> lock_guard{session_ids_m_};
658+
std::lock_guard<std::recursive_mutex> session_ids_lock{session_ids_m_};
659659
pending_to_abort_sessions_[session_id] = std::move(session->second);
660660
pending_to_add_session_ids_.erase(session_id);
661661

‎ext/src/http/client/curl/http_operation_curl.cc‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -464,7 +464,7 @@ std::chrono::system_clock::time_point HttpOperation::NextRetryTime()
464464
{
465465
static std::random_device rd;
466466
static std::mt19937 gen(rd());
467-
static std::uniform_real_distribution<float> dis(0.8, 1.2);
467+
static std::uniform_real_distribution<float> dis(0.8f, 1.2f);
468468

469469
// The initial retry attempt will occur after initialBackoff * random(0.8, 1.2)
470470
auto backoff = retry_policy_.initial_backoff;
@@ -1043,7 +1043,7 @@ CURLcode HttpOperation::Setup()
10431043
// TODO: control local port to use
10441044
// curl_easy_setopt(curl, CURLOPT_LOCALPORT, dcf_port);
10451045

1046-
rc = SetCurlLongOption(CURLOPT_TIMEOUT_MS, http_conn_timeout_.count());
1046+
rc = SetCurlLongOption(CURLOPT_TIMEOUT_MS, static_cast<long>(http_conn_timeout_.count()));
10471047
if (rc != CURLE_OK)
10481048
{
10491049
return rc;

0 commit comments

Comments
 (0)