The configuration macro is changed to a registry that includes types
and default values next to the configuration names.

The macro must be used for any new configuration added and a CI
job verifies that all entries exist and that no environment variable
is accessed directly.

The CI will also verify that the generated file is up to date and
matches the remote registry. It would fail, if either is not the
case.

---------

Co-authored-by: Ruben Bridgewater <ruben.bridgewater@datadoghq.com>

Browse the repository at this point in the history

Browse the repository at this point in the history

…ing zeros (DataDog#288)

* Fix _dd.p.ksr formatting to use 6 significant digits without trailing zeros

Replace std::to_string() (which uses sprintf "%f" producing 6 trailing
decimal places) with snprintf "%.6g" which produces up to 6 significant
digits with no trailing zeros. This matches the behavior of Python's
f"{rate:.6g}" and Go's strconv.FormatFloat(rate, 'g', 6, 64).

Examples: 1.0 -> "1", 0.5 -> "0.5", 0.0 -> "0"

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix clang-format violation in trace_segment.cpp

Reformat the emplace_back call to match clang-format's expected style
(arguments on one line with alignment) to fix the verify CI job.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Do not set _dd.p.ksr when using the DEFAULT sampling mechanism

The ksr trace tag should only be set when the sampling decision comes
from an explicit source (agent rate, rule, or remote rule). When the
DEFAULT mechanism is used — meaning no agent configuration has been
received yet — the rate is a hardcoded 100% and ksr is meaningless.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Replace snprintf with std::to_chars for _dd.p.ksr formatting

Address PR feedback: remove format_rate function and use
std::to_chars with std::chars_format::general inline instead
of snprintf.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Update src/datadog/trace_segment.cpp

Co-authored-by: Damien Mehala <damien.mehala@datadoghq.com>

* Apply suggestion from @bm1549

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

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 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: Damien Mehala <damien.mehala@datadoghq.com>

3 people authored

Browse the repository at this point in the history

Browse the repository at this point in the history

Browse the repository at this point in the history

* feat: add C language binding

* refactor: address PR review feedback for C binding

* refactor: address second round of PR review feedback for C binding

* refactor: address third round of PR review feedback for C binding

* fix: simplify dd_error_t doc comment

Browse the repository at this point in the history

Browse the repository at this point in the history

Browse the repository at this point in the history

Browse the repository at this point in the history