search

apache / celix

Apache Celix is a framework for C and C++14 to develop dynamic modular software applications using component and in-process service-oriented programming.
https://celix.apache.org/
Apache License 2.0
160 stars 85 forks source link

Refactor scheduled event use count to use celix_ref. #648

Closed PengZheng closed 10 months ago

PengZheng commented 10 months ago

Considering we have https://github.com/apache/celix/issues/390, reference count should be implemented use atomic ops. Since macOS does not use glibc, I'm not sure if macOS is affected by this issue. Let's go the safest route.

codecov-commenter commented 10 months ago

Codecov Report

Merging #648 (429e52f) into master (91ff81c) will decrease coverage by 0.04%. The diff coverage is 100.00%.

:exclamation: Current head 429e52f differs from pull request most recent head 45320b6. Consider uploading reports for the commit 45320b6 to get more accurate results

@@            Coverage Diff             @@
##           master     #648      +/-   ##
==========================================
- Coverage   81.59%   81.55%   -0.04%     
==========================================
  Files         260      260              
  Lines       34679    34672       -7     
==========================================
- Hits        28295    28277      -18     
- Misses       6384     6395      +11
Files Changed Coverage Δ
libs/framework/src/celix_scheduled_event.c 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more