Hi,

On Wed, Jun 29, 2011 at 3:04 PM, Diego Biurrun <di...@biurrun.de> wrote:
> On Wed, Jun 29, 2011 at 02:13:19PM -0700, Ronald S. Bultje wrote:
>> From: Ami Fischman <fisch...@chromium.org>
>>
>> if (EAGAIN) continue if (..) { .. } is smaller and does the
>> same thing.
>
> Patch OK, but the log msg could be improved; utils.c is an ambiguous
> file name in Libav and it's not necessary to repeat the body of the
> patch.  Suggestion:
>
> av_find_stream_info(): Slightly simplify control flow.

Changed to: "av_find_stream_info: simplify EGAIN handling."

Ronald
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to