ament / ament_lint

Apache License 2.0
37 stars 107 forks source link

[ament_cpplint] recognizes operator()(int) as C-cast #314

Open gleichdick opened 3 years ago

gleichdick commented 3 years ago
struct A
{
    void operator()(int /*unused*/)
    {
    }
};

results in

Using C-style cast.  Use static_cast<int>(...) instead  [readability/casting] [4]

Version:

root@ac488d647508:/workspaces/ros2/src/warehouse_ros_sqlite# dpkg -l | grep cpplint
ii  ros-rolling-ament-cmake-cpplint                    0.10.4-1focal.20210319.043534        amd64        The CMake API for ament_cpplint to lint C / C++ code using cpplint.
ii  ros-rolling-ament-cpplint                          0.10.6-1focal.20210506.150525        amd64        The ability to check code against
audrow commented 3 years ago

Hi @gleichdick, thanks for making this issue. Do you know if this issue is fixed in a newer version of cpplint?