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

Fix inheritance for ComponentException #676

Merged
merged 1 commit into from
May 24, 2022

Conversation

jeffdiclemente
Copy link
Member

Not inheriting through public inheritance means that this exception type can't be caught by the std::exception nor std::runtime_error types in a catch.

Signed-off-by: The MathWorks, Inc. jdicleme@mathworks.com

Not inheriting through public inheritance means that this exception type can't be caught by the std::exception nor std::runtime_error types in a catch.

Signed-off-by: The MathWorks, Inc. <jdicleme@mathworks.com>
@codecov-commenter
Copy link

codecov-commenter commented May 24, 2022

Codecov Report

Merging #676 (9c68df7) into development (d9f4d5c) will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #676      +/-   ##
===============================================
- Coverage        90.42%   90.32%   -0.10%     
===============================================
  Files              249      230      -19     
  Lines            13532    12465    -1067     
===============================================
- Hits             12236    11259     -977     
+ Misses            1296     1206      -90     
Impacted Files Coverage Δ
compendium/LogService/src/LogService.cpp 0.00% <0.00%> (-100.00%) ⬇️
...endium/ServiceComponent/src/ComponentException.cpp 0.00% <0.00%> (-100.00%) ⬇️
...m/ServiceComponent/src/ServiceComponentRuntime.cpp 0.00% <0.00%> (-100.00%) ⬇️
...mework/src/service/ServiceReferenceBasePrivate.cpp 86.16% <0.00%> (-5.93%) ⬇️
...pendium/tools/SCRCodeGen/ManifestParserFactory.hpp 83.33% <0.00%> (-5.56%) ⬇️
framework/src/service/ServiceListeners.cpp 95.10% <0.00%> (-4.90%) ⬇️
...iveServices/src/metadata/MetadataParserFactory.hpp 66.66% <0.00%> (-3.34%) ⬇️
...ndium/ConfigurationAdmin/src/CMBundleExtension.cpp 85.71% <0.00%> (-3.18%) ⬇️
framework/src/service/ServiceRegistry.cpp 90.76% <0.00%> (-2.57%) ⬇️
framework/src/bundle/BundleRegistry.cpp 89.95% <0.00%> (-2.01%) ⬇️
... and 100 more

@jeffdiclemente jeffdiclemente merged commit 2adce66 into development May 24, 2022
@jeffdiclemente jeffdiclemente deleted the bugfix-compexception branch May 24, 2022 16:43
insi-eb pushed a commit to insi-eb/CppMicroServices-cpp14 that referenced this pull request Oct 17, 2022
Not inheriting through public inheritance means that this exception type can't be caught by the std::exception nor std::runtime_error types in a catch.

Signed-off-by: The MathWorks, Inc. <jdicleme@mathworks.com>

Co-authored-by: Jeff <DiClemente>
jeffdiclemente added a commit that referenced this pull request Oct 18, 2022
Not inheriting through public inheritance means that this exception type can't be caught by the std::exception nor std::runtime_error types in a catch.

Signed-off-by: The MathWorks, Inc. <jdicleme@mathworks.com>

Co-authored-by: Jeff <DiClemente>

Co-authored-by: Jeff DiClemente <jeffdiclemente@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants