[mythtv] [mythtv-commits] Ticket #7167: Check the return value of function calls inside of mythfrontend

Erik Hovland erik at hovland.org
Fri Sep 25 23:46:13 UTC 2009


>> I'd sooner remove the return value of BuildFocusList() than add all those
>> checks. The only time we return false would is when we're missing
>> focusable widgets and that's perfectly valid if the screen works without
>> them. If the screen doesn't work without them then it's a code error, the
>> author has failed to check that the required widgets exist in the theme.
>> So altogether it's redundant for BuildFocusList() to return any value at
>> all.
>
> can you put in a pragma or something so that it is flagged as catered for by
> the scanning/check tool?

Don't do that. I can add annotations for the checker into my tree.

E

-- 
Erik Hovland
erik at hovland.org
http://hovland.org/


More information about the mythtv-dev mailing list