search

SSL-Roots / consai2r2

https://github.com/SSL-Roots/consai_ros2 へ開発を移行しています。
MIT License
8 stars 7 forks source link

lint everything #30

Closed MiraiHattori closed 4 years ago

MiraiHattori commented 4 years ago

colcon testが通るかの実験を兼ねて

MiraiHattori commented 4 years ago

https://github.com/cpplint/cpplint/issues/7

そういえば，#include "header.hpp"と#include は違うものとして認識されているようです．

MiraiHattori commented 4 years ago

ament_clang_formatがなぜか使えなかったのでament_uncrustifyでフォーマットしました。

MiraiHattori commented 4 years ago

""と<>の使い分けは https://github.com/ros2/examples/blob/master/rclcpp/minimal_composition/include/minimal_composition/subscriber_node.hpp などのexamplesを参考にしました

MiraiHattori commented 4 years ago 
$ cd ~/ros2_ws/src/
$ ament_uncrustify --reformat
ShotaAk commented 4 years ago

確認しました。 visibility_control.hのプリプロセッサがインデントされないのは可読性落ちちゃいますね。 エラーが出なければインデントしたいです。

MiraiHattori commented 4 years ago

インデントを入れました．どうやらこの作業の前に手動でclang-formatをかけたときに変わってしまったようです．

MiraiHattori commented 4 years ago

その他にも生のclang-formatが悪さをしていそうなので，まだcommitします

MiraiHattori commented 4 years ago

コミットしなおした方が楽そうなので，PRを出し直しました． https://github.com/SSL-Roots/consai2r2/pull/31 にあります．こちらはcloseしようと思います．