Remove Cylinder::SetLength const method

[ahcorde]

Signed-off-by: ahcorde [email protected]

🦟 Bug fix

Summary

Remove SetLength() const method. I think this method does not make sense. It's imposible to set the Lenght in a const method.

There is no implementation for this method.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

[codecov]

Codecov Report

Merging #259 (7ad391a) into ign-math6 (4c1259a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           ign-math6     #259   +/-   ##
==========================================
  Coverage      99.41%   99.41%           
==========================================
  Files             67       67           
  Lines           6347     6347           
==========================================
  Hits            6310     6310           
  Misses            37       37           

Impacted Files	Coverage Δ
include/ignition/math/Cylinder.hh	100.00% <ø> (ø)

---

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 4c1259a...7ad391a. Read the comment docs.

[chapulina]

Interesting that the ABI checker didn't complain. Is it because it's not implemented?

[azeey]

Interesting that the ABI checker didn't complain. Is it because it's not implemented?

I think it's because Cylinder is a template.