On Sat, 21 Jan 2017 22:53:56 +0100 Moritz Barsnick <[email protected]> wrote:
> When adding new code, please stick to "if (" with a space. (Also in
> some of your other patches.)
>...
> Two different ways of doing the same thing. ;-) (Assignment to ret
> embedded in the "if"-clause versus before it.)
Thanks for pointing that out.
The mess with "ret" exists already in the old code, so I'm not sure, whether I
should fix it only in lines I touch or at all occurances.
Another question: the patcheck tool "complains" about "non doxy comments".
What is the correct way of annotating segments of code? Or is it not recommend
at all?
Regards,
Peter
pgpMYwLWNzyvM.pgp
Description: OpenPGP digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
