Skip to content

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>
clalancette and others added 20 commits October 31, 2022 08:38
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
It doesn't actually work at the moment.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* fix memory leak for MultiLibraryClassLoader

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

* fix potential dead lock reported by thread sanitizer

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

* make the static variable in `ClassLoader` destroyed after `active_class_loaders_`

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

* revert MultipleClassLoader to allow create load same library multiple times

allow ClassLoader to unloadlibrary multiple times without throwing exception

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

* update doc and some minor fix

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

Signed-off-by: Chen Lihui <lihui.chen@sony.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
We can just use target_link_libraries instead.

Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
Signed-off-by: Marco A. Gutierrez <marcogg@marcogg.com>
Signed-off-by: Alejandro Hernández Cordero <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernandez Cordero <ahcorde@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.

8 participants