"Ronald S. Bultje" <[email protected]> writes: > Hi, > > On Wed, Jun 29, 2011 at 3:04 PM, Diego Biurrun <[email protected]> wrote: >> On Wed, Jun 29, 2011 at 02:13:19PM -0700, Ronald S. Bultje wrote: >>> From: Ami Fischman <[email protected]> >>> >>> 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."
Much better. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
