Remove visibility macros

[bmagyar]

No description provided.

[saikishor]

LGTM. Just a minor comment.
parallel_gripper_controller is missing the set(CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS ON)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.83%. Comparing base (f23b7d8) to head (da014e9).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1451   +/-   ##
=======================================
  Coverage   83.83%   83.83%           
=======================================
  Files         122      122           
  Lines       11121    11121           
  Branches      945      945           
=======================================
  Hits         9323     9323           
+ Misses       1490     1489    -1     
- Partials      308      309    +1     

Flag	Coverage Δ
unittests	83.83% <ø> (ø)

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

Files with missing lines	Coverage Δ
...de/diff_drive_controller/diff_drive_controller.hpp	100.00% <ø> (ø)
...rd_command_controller/forward_controllers_base.hpp	100.00% <ø> (ø)
...jectory_controller/joint_trajectory_controller.hpp	100.00% <ø> (ø)
...include/joint_trajectory_controller/trajectory.hpp	75.00% <ø> (ø)
...nclude/tricycle_controller/tricycle_controller.hpp	100.00% <ø> (ø)

... and 6 files with indirect coverage changes