On Wed, Jun 29, 2011 at 11:13:42PM +0100, Måns Rullgård wrote: > "Ronald S. Bultje" <[email protected]> writes: > > > 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.
Only if you fix the typo - notice the missing 'A'. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
