capability.c function headers to say 'TRUE' instead of 'true'
i was looking on old tickets to see if any interested me and this seemed too easy to just leave undone.
I assume you did this ticket to generate discussion, in my opinion we should be consistent with style and leave all function headers in all caps when they refer to the macros
Reply To dark-ether
in my opinion we should be consistent with style and leave all function headers in all caps when they refer to the macros
It's not only a style issue, but the fact that 'TRUE' (the macro) is not the same thing as boolean concept (or C++ / newer C boolean value) 'true'.
Typically I open this kind of tickets when I'm working on something else that I don't want to interrupt (maybe also having modifications for that work in my source tree) and don't want to forget what I've noticed.
Likely there's similar issues in other headers too.
The patch does not apply to S3_0. Just targeting the ticket to S3_1 instead of waiting for separate S3_0 patch.
When referring to value of type bool, capability.c function headers should say 'TRUE' instead of 'true'