On Thu, Jan 17, 2013 at 4:09 PM, Diego Biurrun <di...@biurrun.de> wrote:
>> >> On Thu, Jan 17, 2013 at 11:57 AM, Jindřich Makovička <makov...@gmail.com> 
>> >> wrote:
>> >> --- a/libavformat/avidec.c
>> >> +++ b/libavformat/avidec.c
>> >> @@ -1015,8 +1015,9 @@ static int avi_read_packet(AVFormatContext *s, 
>> >> AVPacket *pkt)
>> >>                  best_stream_index= i;
>> >>              }
>> >>          }
>> >> -        if(!best_st)
>> >> -            return -1;
>> >> +        if(!best_st) {
>> >> +            return AVERROR_EOF;
>> >> +        }
>> >
>> > Don't add {} and squash into 2/2.
>>
>> Fixed.
>
> Sort of - you added a third patch to the set instead of squashing them
> together.  Please squash into a single patch.

Ok. I somehow missed the second part.

--
Jindřich Makovička

Attachment: 0001-avidec-use-sensible-error-codes-instead-of-1.patch
Description: Binary data

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

Reply via email to