Skip to content

Fix deprecated benchmark::internal::Benchmark warnings in performance tests#34

Open
miguelgonrod wants to merge 2 commits into
ros2:rollingfrom
miguelgonrod:miguelgonrod/benchmark_warning
Open

Fix deprecated benchmark::internal::Benchmark warnings in performance tests#34
miguelgonrod wants to merge 2 commits into
ros2:rollingfrom
miguelgonrod:miguelgonrod/benchmark_warning

Conversation

@miguelgonrod
Copy link
Copy Markdown

Description

Fixes #33

Is this user-facing behavior change?

No

Did you use Generative AI?

No

Additional Information

As the issue says, new version of Google bechmark updated call from benchmark::internal::Benchmark to benchmark::Benchmark

Signed-off-by: miguelgonrod <miguelgonrod2004@gmail.com>
@miguelgonrod miguelgonrod self-assigned this May 25, 2026
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented May 25, 2026

Pulls: #34
Gist: https://gist.githubusercontent.com/ahcorde/01faff3b0168ec0995bdb8cdde65c9db/raw/9eaf137376453d80c3dee3e3776571cac58b1053/ros2.repos
BUILD args: --packages-above-and-dependencies performance_test_fixture
TEST args: --packages-above performance_test_fixture
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/19364

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

Signed-off-by: miguelgonrod <miguelgonrod2004@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🧑‍🌾 Deprecated use of benchmark::internal::Benchmark triggers warnings with newer google-benchmark versions

2 participants