Coding conventions for max line length

[felixf4xu]

Code of Conduct

• I have read CODE OF CONDUCT and Support Guidelines before creating this Discussion post.

Contents

Hi,

It seems that there is a strict max line length rule in the coding conventions in autoware, for example, the code below

        const auto time_to_traj = 
          min_lat_dist_to_traj_poly / std::max(epsilon, projected_vel.second);

is separated into 2 lines, max line length is 79.
and the code

    const auto & current_obstacle_pose =
      obstacle_cruise_utils::getCurrentObjectPose(predicted_object, obj_stamp, now(), true);

max line length is 93.

Because the function names, namespaces, and parameters in autoware are long, so almost all function calls are written in 2 lines, but I would say the rule is too strict and make the code less readable.

Anyone else who is also annoyed by this rule?

[maxime-clem]

The current max line length is set to 100 characters (https://github.com/autowarefoundation/autoware/blob/main/CPPLINT.cfg#L7).
This appears to be a very typical limit for C++ projects and I personally think it is fine.