qxmpp-project / qxmpp

Cross-platform C++ XMPP client and server library
410 stars 197 forks source link

Use unique_ptrs #443

Closed lnjX closed 2 years ago

lnjX commented 2 years ago

PR check list:

codecov[bot] commented 2 years ago

Codecov Report

Merging #443 (83e5467) into master (6bd74d6) will decrease coverage by 0.00%. The diff coverage is 80.00%.

@@            Coverage Diff             @@
##           master     #443      +/-   ##
==========================================
- Coverage   69.29%   69.28%   -0.01%     
==========================================
  Files         254      261       +7     
  Lines       23377    23362      -15     
==========================================
- Hits        16198    16187      -11     
+ Misses       7179     7175       -4     
Impacted Files Coverage Δ
src/client/QXmppAtmTrustMemoryStorage.h 100.00% <ø> (ø)
src/client/QXmppAttentionManager.cpp 80.64% <ø> (-0.61%) :arrow_down:
src/client/QXmppAttentionManager.h 100.00% <ø> (ø)
src/client/QXmppBookmarkManager.cpp 0.00% <ø> (ø)
src/client/QXmppBookmarkManager.h 0.00% <ø> (ø)
src/client/QXmppClient.cpp 59.80% <ø> (-0.27%) :arrow_down:
src/client/QXmppClient.h 93.75% <ø> (+0.41%) :arrow_up:
src/client/QXmppClientExtension.cpp 78.94% <ø> (-2.01%) :arrow_down:
src/client/QXmppClientExtension.h 100.00% <ø> (ø)
src/client/QXmppDiscoveryManager.cpp 56.03% <ø> (-0.75%) :arrow_down:
... and 29 more

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