Skip to content

[foxy] Fix runtime error: reference binding to null pointer of type (#1547)#1548

Merged
fujitatomoya merged 1 commit intoros2:foxyfrom
fujitatomoya:foxy-bugfix-20210209-issue-1544
Feb 11, 2021
Merged

[foxy] Fix runtime error: reference binding to null pointer of type (#1547)#1548
fujitatomoya merged 1 commit intoros2:foxyfrom
fujitatomoya:foxy-bugfix-20210209-issue-1544

Conversation

@fujitatomoya
Copy link
Copy Markdown
Collaborator

@fujitatomoya fujitatomoya commented Feb 10, 2021

foxy backport of #1547 (original issue is #1544)

)

* Fix runtime error: reference binding to null pointer of type

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>

* delete cppcheck v1.89 workaround

Signed-off-by: Tomoya.Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

CI:

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

@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

@clalancette @wjwwood friendly ping

Copy link
Copy Markdown
Contributor

@clalancette clalancette left a comment

Choose a reason for hiding this comment

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

Looks good to me. @jacobperron FYI.

@wjwwood
Copy link
Copy Markdown
Member

wjwwood commented Feb 10, 2021

Just making sure @fujitatomoya, but you based the CI off of the Foxy ros2.repos file yes?

If so, I'd say it's good for merge.

@fujitatomoya
Copy link
Copy Markdown
Collaborator Author

@wjwwood

but you based the CI off of the Foxy ros2.repos file yes?

that is yes, based on https://github.com/ros2/ros2/blob/foxy/ros2.repos. thanks for checking!

@fujitatomoya fujitatomoya merged commit a2004f8 into ros2:foxy Feb 11, 2021
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.

4 participants