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.

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

Add apis to retrieve service/client QoS #27

Open
wants to merge 16 commits into
base: irobot/add-events-executor
Choose a base branch
from

Conversation

mauropasse
Copy link

Signed-off-by: Mauro Passerino mpasserino@irobot.com

Mauro and others added 16 commits April 5, 2021 11:14
add constness

Move set callback APIs

Put unread events in queue when setting callback

Put unread events in queue when setting callback

Use or discard previous events: Guard conditions

Unset callbacks support

Rename EventHandle to ExecutorEventHandle

Update name

Add support for events

void return on set_events_executor_callback

Revert "void return on set_events_executor_callback"

Add mutexes

set to nullptr

Remove use_executor_callback_, not needed

Rename ExecutorEventCallback -> EventsExecutorCallback

Rename set_events_executor_callback->set_listener_callback

Use data types when setting callbacks

Move rcutils/executor_event_types.h to rmw/

rename event types

Rename executor_context->callback_context

Rename callback_context->user_data

Reorder APIs arguments

rename rmw_listener_cb_t->rmw_listener_callback_t

Only check callback pointer validity

This is the only item that is used in the RMW
layer, while the others are simply forwarded.

use void * to pass executor ptr

Rework executor callback data

Use RMW renamed file

Signed-off-by: Alberto Soragna <asoragna@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: William Woodall <william@osrfoundation.org>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
…ecutor

Mauro/rebase add events executor
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Mauro Passerino <mpasserino@irobot.com>
Signed-off-by: Chen Lihui <Lihui.Chen@sony.com>

Co-authored-by: Miguel Company <MiguelCompany@eprosima.com>
@mauropasse mauropasse force-pushed the mauro/add-events-executor-get-cli-serv-qos branch from 62a3edf to bea69b8 Compare October 18, 2021 09:30
@wjwwood wjwwood force-pushed the irobot/add-events-executor branch from e486ede to 1d643e3 Compare January 12, 2022 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants