search

ros-industrial-consortium / descartes

ROS-Industrial Special Project: Cartesian Path Planner
Apache License 2.0
127 stars 92 forks source link

Auto format code? #166

Closed davetcoleman closed 8 years ago

davetcoleman commented 8 years ago

Would you guys be opposed to me running my ROS-customized clang-formatter to this code base? It will enforce ROS standard styles within a second, recursively. I've been reading through the code and am a little frustrated by lines of code that are, for example, 137 characters long. I prefer to have two windows side by side when I code and this only works with the ROS standard of 120.

I'd create a PR that changes the whole code base in tiny ways.

Jmeyer1292 commented 8 years ago

Go nuts. A lot of the "old code" is a mess in terms of style, convention, and readability. As long as a PR is just non-syntactic text changes, I don't care.