Closed taavit closed 1 year ago
Patch coverage: 66.66
% and project coverage change: -0.59
:warning:
Comparison is base (
512fd1a
) 80.31% compared to head (86b50fe
) 79.73%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
During experiments with c bindings for nmea library, found that multiple MAX_LEN consts are used around. All of the same value (64). This causes cbindgen to generate single const with ambiguous meaning. When I look closer, found that I've used manual conversion waypoint ids to ArrayString, while proper array_string method is already done. This PR fixes this as well, and because array_string is used just for parameters, I've updated error with more proper one (ParameterLength).