On 7/31/20, Andreas Rheinhardt <[email protected]> wrote:
> smacker_decode_header_tree() uses different variables for return values
> (res) and for errors (err) leading to code like
> res = foo(bar);
> if (res < 0) {
>     err = res;
>     goto error;
> }
> Given that no positive return value is ever used at all one can simplify
> the above by removing the intermediate res.
>

LGTM
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to