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

Thread and Address Sanitizer CI #1

Open
wants to merge 5 commits into
base: ros2
Choose a base branch
from
Open

Thread and Address Sanitizer CI #1

wants to merge 5 commits into from

Conversation

tylerjw
Copy link
Owner

@tylerjw tylerjw commented Jun 12, 2022

Signed-off-by: Tyler Weaver tylerjw@gmail.com

paudrow and others added 5 commits June 28, 2022 09:17
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
…os#201)

* make the meta-object alive only in the lifecycle of the registered plugin

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* remove unuseful variable passed in the lambda

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* add extra tests to make test cases exists in both unique ptr test and utest

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* use the original mutex rather than import a new mutex

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* update document comment

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* symbol visibility for Windows

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* add extra document comment

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

* update doc

Signed-off-by: Chen Lihui <lihui.chen@sony.com>

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
Signed-off-by: Tyler Weaver <tylerjw@gmail.com>
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.

3 participants