search

busstoptaktik / geodesy

Rust geodesy
Apache License 2.0
63 stars 6 forks source link

`CoordinateSet` trait: `get_coord/set_coord` should be `get/set` #104

Closed busstoptaktik closed 2 months ago

busstoptaktik commented 3 months ago

According to the Rust API guidelines, when only one thing could be reasonably gotten/sat, the corresponding methods should be named get and set respectively.

Hence the CoordinateSet methods get_coord(i) and set_coord(i, value) should be deprecated, replaced by get(i) and set(i, value), and eventually removed.

busstoptaktik commented 2 months ago

Closing, since this is only marginally relevant with the introduction of more accessors in PRs #103 and #107