For quite some time now ctidy has been
reporting:
"warning: statement should be inside braces
[readability-braces-around-statements]"
and various codes in the project have been
updated accordingly.
HOWEVER, in the coding standards
documentation at:
https://www.mythtv.org/wiki/Coding_Standards
it explicitly says:
"One-line if or else bodies do not need braces."
Since this would appear to be a change in the
projects coding standards (it now does need
the braces?), the person who made that change
should likely update the wiki (or the ctidy checks
if the braces are still not a hard requirement) so
that future contributors know how to properly
write their code.
Thanks.
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org
reporting:
"warning: statement should be inside braces
[readability-braces-around-statements]"
and various codes in the project have been
updated accordingly.
HOWEVER, in the coding standards
documentation at:
https://www.mythtv.org/wiki/Coding_Standards
it explicitly says:
"One-line if or else bodies do not need braces."
Since this would appear to be a change in the
projects coding standards (it now does need
the braces?), the person who made that change
should likely update the wiki (or the ctidy checks
if the braces are still not a hard requirement) so
that future contributors know how to properly
write their code.
Thanks.
_______________________________________________
mythtv-dev mailing list
mythtv-dev@mythtv.org
http://lists.mythtv.org/mailman/listinfo/mythtv-dev
http://wiki.mythtv.org/Mailing_List_etiquette
MythTV Forums: https://forum.mythtv.org