Fix clang-format violation and logic bug in ksr error handling

bm1549 · claude · bm1549 · commit d9275b8df761 · 2026-03-12T11:30:51.000-04:00
Fix two issues introduced by GitHub suggestion commits:
1. Correct the error condition: `ec == std::errc()` means success, not
   failure — changed to `ec != std::errc()` to properly detect errors.
2. Fix indentation from 4-space to 2-space to match project style and
   pass clang-format checks.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/src/datadog/trace_segment.cpp b/src/datadog/trace_segment.cpp
@@ -331,11 +331,11 @@ void TraceSegment::make_sampling_decision_if_null() {
     const auto [ptr, ec] = std::to_chars(buf.data(), buf.data() + buf.size(),
                                          *sampling_decision_->configured_rate,
                                          std::chars_format::general, 6);
-    if (ec == std::errc()) {
-        std::string error{"string conversion failed: "};
-        error += std::make_error_code(ec).message();
-        logger_->log_error(error);
-        return;
+    if (ec != std::errc()) {
+      std::string error{"string conversion failed: "};
+      error += std::make_error_code(ec).message();
+      logger_->log_error(error);
+      return;
     }
     trace_tags_.emplace_back(tags::internal::ksr, std::string(buf.data(), ptr));
   }
