weixin_39946239 2020-11-29 10:12
浏览 0

Rename all C++ files to match coding standard.

We should rename all C++ files to match coding standard.

By my count we have this work ahead of us (excludes thirdParty): - Good extensions: - 93 .h - 3 .cc - Bad extensions: - [x] 1 .hpp - [x] ~207 .cpp (originally)~ - [ ] 34 .cpp now

We also have to rename myStudlyFile to my_under_file for all C++ code (not MATLAB files!), no matter the extension; that will be painful. - Good basenames: - [x] 21 are all lowercase letters - Bad basenames: - [ ] 294 contain at least 1 uppercase letter

While we're playing the massive-rename game we should also - [ ] Rename all programs' main filenames to match the program name, e.g., runQuadrotorLQR program should be built from runQuadrotorLQR.cc not runLQR.cpp (or run_quadrotor_lqr.cc if we go with lowercase).

This some chance of causing messy conflicts across everyone's own personal branches. I will endeavour to fire a flare to the relevant mailing lists as merge time approaches.

I advise that nobody do these renames except as part of this PR, so that we stop breaking other folks' PRs in the meantime with merge conflicts.

该提问来源于开源项目:RobotLocomotion/drake

  • 写回答

5条回答 默认 最新

  • weixin_39946239 2020-11-29 10:12
    关注

    Possibly this issues is simpler once #3278 is fixed: we could enforce the rename on the remaining non-matlab C++ files, and leave the matlab-only C++ files alone.

    评论

报告相同问题?