* free valid_transitions for all states
transition_map->states[i].valid_transitions wasn't being release which was
leaking about 800 bytes per instantiation.
See https://github.com/aliasrobotics/RVD/issues/333
Signed-off-by: Víctor Mayoral Vilches <v.mayoralv@gmail.com>
* Fixups.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Treat __name the same as __node
This modifies the lexer so that it will accept __name and treat it
exactly the same as __node. __node still works in order to preserve
backwards compatibility.
Fixes#258
Distribution Statement A; OPSEC #2893
Signed-off-by: P. J. Reed <preed@swri.org>
* Update dot graph for lexer
Distribution Statement A; OPSEC #2893
Signed-off-by: P. J. Reed <preed@swri.org>
It wasn't preventing any allocations from happening, so it
doesn't seem to serve much purpose. Also remove the tests
for the maximum string size.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Add localhost option to node creation
Signed-off-by: Brian Ezequiel Marchi <brian.marchi65@gmail.com>
* Bring localhost function from rmw
Signed-off-by: Brian Ezequiel Marchi <brian.marchi65@gmail.com>
* Satisfy uncrustify and address pr comments
Signed-off-by: Brian Ezequiel Marchi <brian.marchi65@gmail.com>
* add optional rmw payload to rcl options for pub and sub
Signed-off-by: William Woodall <william@osrfoundation.org>
* move ignore_local_publications into rmw options structure for subs
Signed-off-by: William Woodall <william@osrfoundation.org>
* reset error message before setting a new one, embed the original one
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
* fix max line length
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
* Bring back deprecated CLI arguments.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Remove extra vertical space.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Promote special CLI rules to flags.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Improve argument parsing logging and error messages.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Remove some short form CLI flags.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Fail fast on invalid ROS arguments.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Address peer review comments.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Fix warning on Windows.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Enforce -r/--remap flags.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Cast size_t to int explicitly.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Support rcl_params_t copies.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Parse parameter overrides from command line.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Parameter overrides' tests passing.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Test rcl_yaml_node_struct_copy() function
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Export rcl_yaml_param_parser as rcl dependency.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Zero initialize parameter overrides before rcl arguments copy.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Initialize local variables early enough.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Simplify rcl package.xml
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Assert arguments sanity in rcl args parsing internal functions.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Extend rcl_yaml_param_parser tests to all parameter types.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Address peer review comments.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Support --remap/-r flags.
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
* Please cpplint
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
