ability to disable a GraphEventDispatcher temporarily

envire / envire-envire_core

Core part for the Environment Representation library

BSD 2-Clause "Simplified" License

7 stars 13 forks source link

ability to disable a GraphEventDispatcher temporarily #48

Closed planthaber closed 5 years ago

arneboe commented 5 years ago

Can you add a test case? :)

edit: also this seems to break existing tests: test_envire_graph.cpp(727): error in "envire_graph_publish_current_state_test": check d.itemAddedEvents.size() == 2 failed

test_envire_graph.cpp(732): error in "envire_graph_publish_current_state_test": check d.itemRemovedEvents.size() == 2 failed

planthaber commented 5 years ago

used valgrind on the test suite, missed to initialize the enabled bool when the constructor without parameter was called