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
clangtidy gives warnings about narrowing when constructing persistent classes (ie in *DataProducts constructors). Those narrowings are conscious choices to reduce persistent payload. Can we turn off those warnings?
narrowing conversion from 'double' to 'float' [cppcoreguidelines-narrowing-conversions]
The text was updated successfully, but these errors were encountered:
brownd1978
changed the title
clangtidy flags narrowing in persistent classes
clangtidy warnings about narrowing in persistent classes
Apr 13, 2024
clangy-tidy wants you to write myfloat = float(mydouble), so declaring the narrowing was intentional. If you don't like that, do you want the warning turned off globally, for the three *Products directories, or by file or instance (which would be very tedious)?
I don't see much value in this test in this *Products directory, if it could be turned off that would be good. We can also just update to make the explicit casts, its just tedious.
clangtidy gives warnings about narrowing when constructing persistent classes (ie in *DataProducts constructors). Those narrowings are conscious choices to reduce persistent payload. Can we turn off those warnings?
narrowing conversion from 'double' to 'float' [cppcoreguidelines-narrowing-conversions]
The text was updated successfully, but these errors were encountered: