Heiko, On 05.09.2016 06:59, Heiko Schocher wrote: > fix the following code: > > -ret = expression; > -if (ret) > - return ret; > -return 0; > +return expression; "Fix"? ;-) What was broken? I agree that we can write the expression in a different way, but is it really worth it? Is this part of a tree-wide cleanup? Thanks, //richard