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

702 ServiceTracker GetTrackingCount() when opened and then closed #714

Conversation

shane-riley
Copy link
Contributor

Ensure that the ServiceTracker::GetTrackingCount() method returns -1 if the tracker has been opened and then closed.

@shane-riley shane-riley changed the title 702 tracker trackingcount 702 ServiceTracker GetTrackingCount() when opened and then closed Aug 11, 2022
@codecov-commenter
Copy link

codecov-commenter commented Aug 11, 2022

Codecov Report

Merging #714 (c137373) into development (a34977c) will decrease coverage by 0.06%.
The diff coverage is 89.09%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #714      +/-   ##
===============================================
- Coverage        89.86%   89.80%   -0.07%     
===============================================
  Files              230      231       +1     
  Lines            13639    13823     +184     
===============================================
+ Hits             12257    12414     +157     
- Misses            1382     1409      +27     
Impacted Files Coverage Δ
...include/cppmicroservices/detail/ServiceTracker.tpp 86.31% <89.09%> (+0.96%) ⬆️
...mework/include/cppmicroservices/ServiceException.h 0.00% <0.00%> (-100.00%) ⬇️
framework/include/cppmicroservices/ServiceEvent.h 75.00% <0.00%> (-25.00%) ⬇️
...ramework/include/cppmicroservices/ServiceObjects.h 87.50% <0.00%> (-12.50%) ⬇️
...nfigurationAdmin/src/ConfigurationAdminPrivate.hpp 88.88% <0.00%> (-11.12%) ⬇️
...iveServices/src/metadata/MetadataParserFactory.hpp 70.00% <0.00%> (-5.00%) ⬇️
...mework/include/cppmicroservices/ServiceReference.h 95.83% <0.00%> (-4.17%) ⬇️
...ork/include/cppmicroservices/detail/CounterLatch.h 93.54% <0.00%> (-3.33%) ⬇️
...larativeServices/src/ConfigurationListenerImpl.cpp 72.22% <0.00%> (-3.29%) ⬇️
...dium/ConfigurationAdmin/src/CMAsyncWorkService.cpp 81.63% <0.00%> (-1.53%) ⬇️
... and 46 more

@shane-riley
Copy link
Contributor Author

The BundleTracker PR has been updated to match the resolution to 702 in both implementation and in behavior.

@jeffdiclemente jeffdiclemente marked this pull request as ready for review August 12, 2022 19:15
@jeffdiclemente jeffdiclemente merged commit f024d14 into CppMicroServices:development Sep 22, 2022
insi-eb pushed a commit to insi-eb/CppMicroServices-cpp14 that referenced this pull request Feb 8, 2023
…pMicroServices#714)

* Update CXX_STANDARD to 17 for doc

* TrackingCount=-1 when ServiceTracker is closed

* Add branch to workflow

* Remove branch from workflows

* ServiceTracker grab lock only once

* Update ServiceTracker.tpp

* Update ServiceTracker.tpp

Co-authored-by: Alexander Christoforides <38366659+achristoforides@users.noreply.github.com>
Co-authored-by: shane-riley (The Mathworks Inc.) <sriley@mathworks.com>
Co-authored-by: Jeff DiClemente <jeffdiclemente@users.noreply.github.com>
jeffdiclemente added a commit that referenced this pull request Feb 8, 2023
…) (#800)

* Update CXX_STANDARD to 17 for doc

* TrackingCount=-1 when ServiceTracker is closed

* Add branch to workflow

* Remove branch from workflows

* ServiceTracker grab lock only once

* Update ServiceTracker.tpp

* Update ServiceTracker.tpp

Co-authored-by: Shane Riley <52603694+shane-riley@users.noreply.github.com>
Co-authored-by: Alexander Christoforides <38366659+achristoforides@users.noreply.github.com>
Co-authored-by: shane-riley (The Mathworks Inc.) <sriley@mathworks.com>
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants