xrobot-org/XRobot

Remove excess semicolons at the end of a function

EricChoLiu opened this issue ยท 1 comments

Describe the bug

  • ๐Ÿ› There's a ';' after some function end '}' that doesn't make any sense.
  • โš ๏ธ This doesn't have any effect, but it can be detected by linter (I use cpplint ).
  • ๐Ÿค” Eliminating the above effects can make the code more formatted. ๐Ÿ˜„

Here are the output of cpplint:

XRobot/src/component/comp_actuator.hpp:40: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_cmd.cpp:34: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_cmd.cpp:57: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_pid.cpp:151: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_pid.cpp:153: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_pid.cpp:155: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_pid.cpp:157: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/component/comp_utils.cpp:118: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/device/bmi088/dev_bmi088.cpp:350: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/device/simulator/dev_motor.hpp:38: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/system/FreeRTOS/timer.cpp:32: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/system/Linux/timer.cpp:32: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/system/Linux_Webots/timer.cpp:32: warning: You don't need a ; after a }  [readability/braces] [4]
XRobot/src/system/None/timer.cpp:26: warning: You don't need a ; after a }  [readability/braces] [4]

Screenshots

  • The last line of code XRobot/src/system/None/timer.cpp:26: warning: You don't need a ; after a } [readability/braces] [4] above seems like this,

image
image

fixed by ea297e9