You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With uncrustify v0.68 installed (default for macOS) we get errors for packages that use rosidl generators related to the code_width variable. E.g.:
Option<UNUM>: at /var/folders/cl/kf4w4g450n14r3mvm95yt9tw0000gn/T/uncrustify_bc3goxjd:2241: requested value 999 for option code_width is greater than the maximum value 0
Option<UNUM>: at /var/folders/cl/kf4w4g450n14r3mvm95yt9tw0000gn/T/uncrustify_bc3goxjd:2241: Expected unsigned number for code_width; got '999'
Output prefix: /var/folders/cl/kf4w4g450n14r3mvm95yt9tw0000gn/T/uncrustify_1dbjacdr/
Output suffix: .uncrustify
Expected behavior
No uncrustify errors.
Actual behavior
Uncrustify errors and test fails.
Additional information
An upper bound of 256 for the uncrustify option, code_width, was introduced in uncrustify/uncrustify#1852, specifically:
Bug report
Required Info:
Steps to reproduce issue
With uncrustify v0.68 installed (default for macOS) we get errors for packages that use rosidl generators related to the
code_width
variable. E.g.:Expected behavior
No uncrustify errors.
Actual behavior
Uncrustify errors and test fails.
Additional information
An upper bound of
256
for the uncrustify option,code_width
, was introduced in uncrustify/uncrustify#1852, specifically:https://github.com/uncrustify/uncrustify/pull/1852/files#diff-0799bb9998b90a7d6cbaf9f6679b6938R2461
We are attempting to bypass any line length issues by passing a value of
999
in several places:rosidl/rosidl_typesupport_introspection_c/cmake/rosidl_typesupport_introspection_c_generate_interfaces.cmake
Line 204 in 43eee85
rosidl/rosidl_typesupport_introspection_cpp/cmake/rosidl_typesupport_introspection_cpp_generate_interfaces.cmake
Line 196 in 43eee85
Along with other rosidl repos.
Looks like a solution for ignoring line length for generated files is to set
code_width
to0
.The text was updated successfully, but these errors were encountered: