Fix Windows Compilation Issues in joint_trajectory_controller and pid_controller (master branch) (backport #1330)

[mergify]

Description:

This pull request addresses compilation issues encountered in the joint_trajectory_controller and pid_controller when building on Windows. The changes ensure compatibility with the Windows environment and improve overall stability.

Changes Made:

• joint_trajectory_controller:

  • Modifies the lambda capture in is_erase_value from [ ] to [=], which captures all outer scope variables by value.

• pid_controller:

  • Added compile definitions for Windows to minimize namespace collisions and define math constants:

    if(WIN32)
      add_compile_definitions(
        # For math constants
        _USE_MATH_DEFINES
        # Minimize Windows namespace collision
        NOMINMAX
        WIN32_LEAN_AND_MEAN
      )
    endif()

Testing:

• Local tests have been run successfully, confirming that all modifications do not break existing functionality.

Please let me know if you have any questions or need further modifications. I appreciate your feedback!

---

This is an automatic backport of pull request #1330 done by Mergify.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.19%. Comparing base (b625c50) to head (a52ad83).
Report is 1 commits behind head on humble.

Additional details and impacted files

@@            Coverage Diff             @@
##           humble    #1332      +/-   ##
==========================================
+ Coverage   65.17%   65.19%   +0.01%     
==========================================
  Files         103      103              
  Lines       11997    11997              
  Branches     7399     7400       +1     
==========================================
+ Hits         7819     7821       +2     
  Misses       1550     1550              
+ Partials     2628     2626       -2     

Flag	Coverage Δ
unittests	65.19% <100.00%> (+0.01%)	⬆️

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

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

... and 1 file with indirect coverage changes