61 lines
1.6 KiB
INI
61 lines
1.6 KiB
INI
# filters
|
|
#
|
|
# 1. [-whitespace/braces]
|
|
# { should almost always be at the end of the previous line
|
|
#
|
|
# Allow:
|
|
# bool SocketUDP::pollin(uint32_t timeout_ms)
|
|
# {
|
|
#
|
|
# Instead of:
|
|
# bool SocketUDP::pollin(uint32_t timeout_ms) {
|
|
#
|
|
# 2. [-runtime/references]
|
|
# Is this a non-const reference? If so, make const or use a pointer
|
|
#
|
|
# Allow:
|
|
# void get_client_address(const char *&ip_addr, uint16_t &port);
|
|
#
|
|
# Instead of:
|
|
# void get_client_address(const char *&ip_addr, uint16_t *port);
|
|
#
|
|
# 3. [-whitespace/indent]
|
|
# private: should be indented +1 space
|
|
#
|
|
# 4. [-whitespace/blank_line]
|
|
# Do not leave a blank line after "private:"
|
|
#
|
|
# 3 and 4 are to allow Gazebo Sim class formatting where each
|
|
# function / method must have an access specifier.
|
|
#
|
|
#
|
|
# 5. [-whitespace/newline]
|
|
# An else should appear on the same line as the preceding }
|
|
#
|
|
# Allow:
|
|
# }
|
|
# else if (time > this->dataPtr->lastUpdateTime)
|
|
# {
|
|
#
|
|
# Instead of:
|
|
# } else if (time > this->dataPtr->lastUpdateTime) {
|
|
#
|
|
# 6. [-build/include_subdir]
|
|
# Include the directory when naming header files
|
|
#
|
|
# This prevent an error when including the plugin headers as:
|
|
# #include "GimbalSmall2dPlugin.hh"
|
|
#
|
|
# 7. [-build/c++11], [+build/c++14]
|
|
#
|
|
# This is to allow headers not approved for C++11 such as <chrono> etc.
|
|
#
|
|
# 8. [-legal/copyright]
|
|
# Do not require a copyright header in each file
|
|
#
|
|
|
|
set noparent
|
|
filter=-whitespace/braces,-runtime/references,-whitespace/indent,-whitespace/blank_line,-whitespace/newline,-build/include_subdir,-build/c++11,+build/c++14,-legal/copyright
|
|
linelength=80
|
|
root=include
|