Skip to content

Conversation

@tylerjw
Copy link
Owner

@tylerjw tylerjw commented Jun 12, 2022

Signed-off-by: Tyler Weaver [email protected]

paudrow and others added 5 commits June 28, 2022 09:17
Signed-off-by: Chris Lalancette <[email protected]>
…os#201)

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

Signed-off-by: Chen Lihui <[email protected]>

* remove unuseful variable passed in the lambda

Signed-off-by: Chen Lihui <[email protected]>

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

Signed-off-by: Chen Lihui <[email protected]>

* use the original mutex rather than import a new mutex

Signed-off-by: Chen Lihui <[email protected]>

* update document comment

Signed-off-by: Chen Lihui <[email protected]>

* symbol visibility for Windows

Signed-off-by: Chen Lihui <[email protected]>

* add extra document comment

Signed-off-by: Chen Lihui <[email protected]>

* update doc

Signed-off-by: Chen Lihui <[email protected]>

Signed-off-by: Chen Lihui <[email protected]>
clalancette and others added 20 commits October 31, 2022 08:38
Signed-off-by: Chris Lalancette <[email protected]>

Signed-off-by: Chris Lalancette <[email protected]>
It doesn't actually work at the moment.

Signed-off-by: Chris Lalancette <[email protected]>

Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
* fix memory leak for MultiLibraryClassLoader

Signed-off-by: Chen Lihui <[email protected]>

* fix potential dead lock reported by thread sanitizer

Signed-off-by: Chen Lihui <[email protected]>

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

Signed-off-by: Chen Lihui <[email protected]>

* revert MultipleClassLoader to allow create load same library multiple times

allow ClassLoader to unloadlibrary multiple times without throwing exception

Signed-off-by: Chen Lihui <[email protected]>

* update doc and some minor fix

Signed-off-by: Chen Lihui <[email protected]>

Signed-off-by: Chen Lihui <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
We can just use target_link_libraries instead.

Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Chris Lalancette <[email protected]>
Signed-off-by: Marco A. Gutierrez <[email protected]>
Signed-off-by: Alejandro Hernandez Cordero <[email protected]>
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