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.

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

Fix memory faults when registering subscribers, services and timers #675

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Fix memory faults when registering subscribers, services and timers #675

wants to merge 6 commits into from

Conversation

Affonso-Gui
Copy link
Member

@Affonso-Gui Affonso-Gui commented Jun 22, 2021 •
edited
Loading

Fixes #668

  • Add tests
  • Consider garbage collecting old subscribers to avoid memory leak?

Affonso-Gui added a commit to Affonso-Gui/jsk_roseus that referenced this pull request Oct 28, 2022
@Affonso-Gui
Add tests for jsk-ros-pkg#675
add0271
@Affonso-Gui Affonso-Gui changed the title [WIP] Fix possible memory faults when registering subscribers Fix memory faults when registering subscribers, services and timers Oct 28, 2022
Affonso-Gui added a commit to Affonso-Gui/jsk_roseus that referenced this pull request Nov 3, 2022
@Affonso-Gui
Add tests for jsk-ros-pkg#675
a0a4581
Affonso-Gui added a commit to Affonso-Gui/jsk_roseus that referenced this pull request Nov 4, 2022
@Affonso-Gui
Register jsk-ros-pkg#675 tests in CMakeLists
1518839
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR/PleaseAddTest
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Memory faults on subscribers
2 participants
@Affonso-Gui @k-okada