Fix TPE codecheck errors

[claireyywang]

Add tpe to codecheck and fix errors. Remove white spaces and shorten line length. Closes #58

Uses sh tools/code_check.sh to conduct code check on tpe

[codecov]

Codecov Report

Merging #62 into ign-physics2 will increase coverage by 0.01%.
The diff coverage is 93.75%.

@@               Coverage Diff                @@
##           ign-physics2      #62      +/-   ##
================================================
+ Coverage         81.52%   81.53%   +0.01%     
================================================
  Files               103      103              
  Lines              3432     3434       +2     
================================================
+ Hits               2798     2800       +2     
  Misses              634      634              

Impacted Files	Coverage Δ
tpe/lib/src/Engine.hh	100.00% <ø> (ø)
tpe/lib/src/Link.hh	100.00% <ø> (ø)
tpe/lib/src/Model.hh	100.00% <ø> (ø)
tpe/lib/src/Shape.hh	100.00% <ø> (ø)
tpe/lib/src/World.hh	100.00% <ø> (ø)
tpe/plugin/src/Base.hh	98.00% <ø> (ø)
tpe/plugin/src/SDFFeatures.cc	84.41% <ø> (ø)
tpe/plugin/src/ShapeFeatures.cc	64.34% <ø> (ø)
tpe/plugin/src/SimulationFeatures.cc	39.13% <ø> (ø)
tpe/plugin/src/EntityManagementFeatures.cc	78.23% <75.00%> (+0.12%)	⬆️
... and 7 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c7f99de...d8099e0. Read the comment docs.

[chapulina]

There are still some codecheck errors on Jenkins:

https://build.osrfoundation.org/job/ignition_physics-ci-pr_any-ubuntu_auto-amd64/684/cppcheckResult/

Those are checked running make codecheck. If you're in a colcon workspace: Actually, Jenkins is running tools/code_check.sh, see gazebo-tooling/release-tools#148

colcon build --cmake-target codecheck --packages-select-regex .*physics

[iche033]

I see the same code check errors running sh tools/code_check.sh.

make codecheck gives a couple of errors related to to tpe:

/usr/bin/find: ‘/home/osrf/code/ign_c_ws/src/ign-physics/tpe/include’: No such file or directory

Not sure if something that could be fixed in ign-physics or we have to modify ign-cmake. We don't install any headers and the directory layout is different so that msg does not apply. We could just ignore it.

[chapulina]

We don't install any headers

TIL that ign-cmake installs some for us, see gazebo-release/gz-physics2-release#7 (comment)

[iche033]

ah yes the auto-generated headers..

[claireyywang]

I have fixed the warning. Ready for another review @iche033 🙇‍♀️