Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix warning maybe-uninitialized #16

Merged
merged 1 commit into from
Jun 1, 2021
Merged

Fix warning maybe-uninitialized #16

merged 1 commit into from
Jun 1, 2021

Conversation

homalozoa
Copy link

Fix warning "may be used uninitialized" of googletest when compiling with GCC 11.1

Signed-off-by: Homalozoa [email protected]

@homalozoa
Copy link
Author

And please cherry pick this PR to other branch to avoid compiling error(some project would treat warning as error).

@clalancette
Copy link

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@clalancette
Copy link

For the record, the upstream PR that fixes this is google#3024

@clalancette clalancette merged commit 32b0a10 into ament:ros2 Jun 1, 2021
@clalancette
Copy link

@Mergifyio backport galactic foxy

mergify bot pushed a commit that referenced this pull request Jun 1, 2021
Signed-off-by: Homalozoa <[email protected]>
(cherry picked from commit 32b0a10)
mergify bot pushed a commit that referenced this pull request Jun 1, 2021
Signed-off-by: Homalozoa <[email protected]>
(cherry picked from commit 32b0a10)
@mergify
Copy link

mergify bot commented Jun 1, 2021

Command backport galactic foxy: success

Backports have been created

clalancette pushed a commit that referenced this pull request Jun 4, 2021
Signed-off-by: Homalozoa <[email protected]>
(cherry picked from commit 32b0a10)

Co-authored-by: Homalozoa X <[email protected]>
jacobperron pushed a commit that referenced this pull request Jun 10, 2021
Signed-off-by: Homalozoa <[email protected]>
(cherry picked from commit 32b0a10)

Co-authored-by: Homalozoa X <[email protected]>
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.

2 participants