Satisfy ament linters

[quarkytale]

Would go after #9.

Used [ament_code_style.cfg]
Copyright, cpplint and uncrustify cleared!

Flake is failing but seems unrelated:

4: -- run_test.py: invoking following command in '/github/home/ws/src/buoy_sim/buoy_gazebo':
4:  - /opt/ros/galactic/bin/ament_flake8 --xunit-file /github/home/ws/build/buoy_gazebo/test_results/buoy_gazebo/flake8.xunit.xml
4: 
4: ./launch/electrohydraulic_pto.launch.py:48:100: E501 line too long (116 > 99 characters)
4:             '/model/Hydraulics_Test/joint/HydraulicRam/UserCommandedCurr@std_msgs/msg/Float64]ignition.msgs.Double',

[chapulina]

Looking good to me so far! Just a small comment.

[quarkytale]

This is too messy to fix as well as review. Working on separate PRs for individual test-failures fixes.

[chapulina]

Flake is failing but seems unrelated:

Ah that was introduced by me on #13 , I'll take care of it on another PR

[chapulina]

@andermi , was there a reason not to use int16_t here before? We're about to change it to int16_t because of this warning:

Use int16/int64/etc, rather than the C type short  [runtime/int] [4]

[chapulina]

Thanks for iterating, LGTM! Let's keep the short change for now, let's just keep in mind that this was done in case it creates problems.