"Ronald S. Bultje" <[email protected]> writes:

> Hi,
>
> On Sat, Jul 2, 2011 at 6:57 PM, Mans Rullgard <[email protected]> wrote:
>> Signed-off-by: Mans Rullgard <[email protected]>
>> ---
>>  libavcodec/mjpegdec.c |   40 +++++++++++++++++++---------------------
>>  1 files changed, 19 insertions(+), 21 deletions(-)
>
> OK.
>
>> +                    goto the_end;
>>                     break;
>
> Also feel free to remove the "break;" here.

That's in the next patch.  Splitting this made it much easier to read.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to