Skip to content

Commit

Permalink
Fix #1705 (#1731)
Browse files Browse the repository at this point in the history
  • Loading branch information
marcalff authored Nov 4, 2022
1 parent ac179dd commit 7812651
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions api/include/opentelemetry/nostd/span.h
Original file line number Diff line number Diff line change
Expand Up @@ -240,9 +240,6 @@ class span<T, dynamic_extent>
T *end() const noexcept { return data_ + extent_; }

private:
// Note: matches libstdc++'s layout for std::span
// See
// https://github.com/gcc-mirror/gcc/blob/a60701e05b3878000ff9fdde1aecbc472b9dec5a/libstdc%2B%2B-v3/include/std/span#L402-L403
size_t extent_;
T *data_;
};
Expand Down

1 comment on commit 7812651

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'OpenTelemetry-cpp sdk Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 7812651 Previous: ac179dd Ratio
BM_BaselineBuffer/1 8915591.2399292 ns/iter 571726.3221740723 ns/iter 15.59

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.