Fix atomic variables in JTC

[christophfroehlich]

Explicitly initialize them, should fix the error reported with #1720 (comment)

I don't know why the tests did not cover this?

Furthermore, I saw that subscriber_is_active_ should be atomic too, as it is used in both threads.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.29%. Comparing base (e00d4cb) to head (22c8aa0).
Report is 7 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1749      +/-   ##
==========================================
- Coverage   86.30%   86.29%   -0.01%     
==========================================
  Files         123      123              
  Lines       11940    11941       +1     
  Branches      994      995       +1     
==========================================
  Hits        10305    10305              
  Misses       1333     1333              
- Partials      302      303       +1     

Flag	Coverage Δ
unittests	86.29% <ø> (-0.01%)	⬇️

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

Files with missing lines	Coverage Δ
...jectory_controller/joint_trajectory_controller.hpp	100.00% <ø> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[bmagyar]

@lqiang2003cn could you please confirm is this fixes the issue you are currently seeing?

[saikishor]

Looks good