Skip to content

Fix move assignment of bounded vector #496

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jun 3, 2020
Merged

Fix move assignment of bounded vector #496

merged 1 commit into from
Jun 3, 2020

Conversation

@jacobperron
Copy link
Member

@jacobperron jacobperron commented May 28, 2020

Reapplying the patch in #445 to the file that was moved in #442.

Testing test_communication and rosidl_runtime_cpp:

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

@jacobperron jacobperron mentioned this pull request May 28, 2020
Closed
@jacobperron jacobperron self-assigned this May 28, 2020
@jacobperron jacobperron added the bug Something isn't working label May 28, 2020
@jacobperron jacobperron requested a review from dirk-thomas June 3, 2020 06:45
@jacobperron jacobperron merged commit a334c0a into master Jun 3, 2020
@jacobperron jacobperron deleted the jacob/fix_move_assignment_bounded_vector branch June 3, 2020 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ivanpauno ivanpauno ivanpauno approved these changes

+1 more reviewer

@dirk-thomas dirk-thomas dirk-thomas approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@jacobperron jacobperron

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jacobperron @dirk-thomas @ivanpauno @carlos-apex