Fix polling manager thread shut down

[christophebedard]

Part of #247

This doesn't solve everything. It fixes the shutdown of the polling thread, but then there's another issue.

Signed-off-by: Christophe Bedard [email protected]

[codecov]

Codecov Report

Merging #248 (1032515) into master (9db2a56) will decrease coverage by 0.02%.
The diff coverage is 6.25%.

@@            Coverage Diff             @@
##           master     #248      +/-   ##
==========================================
- Coverage   15.95%   15.93%   -0.03%     
==========================================
  Files          78       77       -1     
  Lines        1999     2008       +9     
  Branches      971      980       +9     
==========================================
+ Hits          319      320       +1     
- Misses       1489     1496       +7     
- Partials      191      192       +1     

Flag	Coverage Δ
unittests	15.93% <6.25%> (-0.03%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
email/src/context.cpp	0.00% <0.00%> (ø)
email/src/email/polling_manager.cpp	0.00% <0.00%> (ø)
rmw_email_cpp/src/rmw_init.cpp	0.00% <0.00%> (ø)
email/src/log.cpp	65.97% <50.00%> (-0.34%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9db2a56...1032515. Read the comment docs.