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

use rcutils allocator #127

Closed
Karsten1987 opened this issue Apr 18, 2017 · 1 comment
Closed

use rcutils allocator #127

Karsten1987 opened this issue Apr 18, 2017 · 1 comment
Assignees
@Karsten1987

Comments

@Karsten1987
Copy link
Contributor

Karsten1987 commented Apr 18, 2017 •
edited
Loading

rcl_lifecycle currently uses malloc and free natively. This should be changed to the allocators in rcutils.
@Karsten1987 Karsten1987 self-assigned this Apr 18, 2017
@mikaelarguedas
Copy link
Member

@Karsten1987 to link to other issue / close if needed

@Karsten1987 Karsten1987 added the in progress Actively being worked on (Kanban column) label May 10, 2017
@Karsten1987 Karsten1987 changed the title use c_utilities allocator use rcutils allocator May 10, 2017
@Karsten1987 Karsten1987 added in review Waiting for review (Kanban column) and removed in progress Actively being worked on (Kanban column) labels Jun 12, 2017
This was referenced Jun 14, 2017
Merged
Closed
Merged
@dirk-thomas dirk-thomas removed the in review Waiting for review (Kanban column) label Jun 16, 2017
emersonknapp pushed a commit to aws-ros-dev/rcl that referenced this issue Jun 3, 2019
@nburek @wjwwood
Logging (ros2#127)
86b5c24 
* Creating a logging configure function that can be used to select which output handlers you want to use and setting the framework to allow for multiple output handlers in the logging code.

cr https://code.amazon.com/reviews/CR-3734291

* refactoring so code for console sink can be reused

cr https://code.amazon.com/reviews/CR-3682772

* Moving the external logging library and rosout functions out to rcl. Also automatically uncrustifying and fixing unit tests.

* Switching back to va_list for output handlers.

* PR fixup

* use strncat_s on Windows

Signed-off-by: William Woodall <william@osrfoundation.org>

* derp

Signed-off-by: William Woodall <william@osrfoundation.org>
ivanpauno pushed a commit that referenced this issue Jan 2, 2020
@dirk-thomas
Merge pull request #127 from ros2/ignore_unavailable_rmw
7240d97 
ignore unavailable rmw implementations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Karsten1987 Karsten1987

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dirk-thomas @Karsten1987 @mikaelarguedas