Replace std::string pointer by reference and remove unused function DLSym

SAtacker commented 5 months ago

Add test for DLOpen

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.87%. Comparing base (e61e7ca) to head (2315dea). Report is 11 commits behind head on main.

Additional details and impacted files

[![Impacted file tree graph](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193/graphs/tree.svg?width=650&height=150&src=pr&token=7UWTYSVVT5&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research)](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) ```diff @@ Coverage Diff @@ ## main #193 +/- ## ========================================== + Coverage 78.83% 78.87% +0.03% ========================================== Files 8 8 Lines 3048 3048 ========================================== + Hits 2403 2404 +1 + Misses 645 644 -1 ``` | [Files](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) | Coverage Δ | | |---|---|---| | [lib/Interpreter/DynamicLibraryManager.cpp](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-bGliL0ludGVycHJldGVyL0R5bmFtaWNMaWJyYXJ5TWFuYWdlci5jcHA=) | `75.12% <100.00%> (ø)` | | | [lib/Interpreter/Paths.cpp](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-bGliL0ludGVycHJldGVyL1BhdGhzLmNwcA==) | `37.75% <100.00%> (+2.34%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) | [Files](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research) | Coverage Δ | | |---|---|---| | [lib/Interpreter/DynamicLibraryManager.cpp](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-bGliL0ludGVycHJldGVyL0R5bmFtaWNMaWJyYXJ5TWFuYWdlci5jcHA=) | `75.12% <100.00%> (ø)` | | | [lib/Interpreter/Paths.cpp](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research#diff-bGliL0ludGVycHJldGVyL1BhdGhzLmNwcA==) | `37.75% <100.00%> (+2.34%)` | :arrow_up: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/compiler-research/CppInterOp/pull/193/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=compiler-research)