On 7/31/20, Andreas Rheinhardt <andreas.rheinha...@gmail.com> 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
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
