Skip to content

Fix loop leaks#60

Merged
tonybaloney merged 4 commits into
mainfrom
fix_loop_leaks3
Aug 10, 2022
Merged

Fix loop leaks#60
tonybaloney merged 4 commits into
mainfrom
fix_loop_leaks3

Conversation

@tonybaloney

Copy link
Copy Markdown
Collaborator
  • Fixes a leak in the asctime field of log record
  • Fixes a leak in stream writer write() and flush() calls
  • Fixes a leak in stream close()

@codecov-commenter

codecov-commenter commented Aug 10, 2022

Copy link
Copy Markdown

Codecov Report

Merging #60 (fb5df0c) into main (b08df4b) will decrease coverage by 0.00%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   86.32%   86.31%   -0.01%     
==========================================
  Files          10       10              
  Lines        1894     1900       +6     
==========================================
+ Hits         1635     1640       +5     
- Misses        259      260       +1     
Impacted Files Coverage Δ
src/picologging/formatter.cxx 89.04% <50.00%> (-0.62%) ⬇️
src/picologging/logger.cxx 82.33% <100.00%> (+0.04%) ⬆️
src/picologging/streamhandler.cxx 94.20% <100.00%> (+0.35%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tonybaloney
tonybaloney merged commit a0aa8c6 into main Aug 10, 2022
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.

2 participants