Unused variable warnings when building

[sea-bass]

When building this package (ROS 2), I see these warnings.

/home/runner/work/moveit2/moveit2/.work/upstream_ws/src/geometric_shapes/src/aabb.cpp:43:26: warning: unused variable 't' [-Wunused-variable]
     43 |   const Eigen::Vector3d& t = transform.translation();
        |                          ^
  /home/runner/work/moveit2/moveit2/.work/upstream_ws/src/geometric_shapes/src/aabb.cpp:45:10: warning: unused variable 'x_range' [-Wunused-variable]
     45 |   double x_range = 0.5 * (fabs(r(0, 0) * box[0]) + fabs(r(0, 1) * box[1]) + fabs(r(0, 2) * box[2]));
        |          ^~~~~~~
  /home/runner/work/moveit2/moveit2/.work/upstream_ws/src/geometric_shapes/src/aabb.cpp:46:10: warning: unused variable 'y_range' [-Wunused-variable]
     46 |   double y_range = 0.5 * (fabs(r(1, 0) * box[0]) + fabs(r(1, 1) * box[1]) + fabs(r(1, 2) * box[2]));
        |          ^~~~~~~
  /home/runner/work/moveit2/moveit2/.work/upstream_ws/src/geometric_shapes/src/aabb.cpp:47:10: warning: unused variable 'z_range' [-Wunused-variable]
     47 |   double z_range = 0.5 * (fabs(r(2, 0) * box[0]) + fabs(r(2, 1) * box[1]) + fabs(r(2, 2) * box[2]));
        |          ^~~~~~~

Leaving as a reminder to fix this later, but if someone gets to it first that's cool.

[sea-bass]

This might have been a source control issue in #239

In ROS 1, this block of code is in a preprocessor conditional with the alternative implementation, but in ROS 2 it's just doubling up on the code...