On 02/19/2017 10:25 AM, Luca Barbato wrote:
> On 19/02/2017 17:15, John Stebbins wrote:
>> ---
>> libavformat/dv.c | 7 +++++--
>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavformat/dv.c b/libavformat/dv.c
>> index d4e5180..0cd60de 100644
>> --- a/libavformat/dv.c
>> +++ b/libavformat/dv.c
>> @@ -484,11 +484,14 @@ static int dv_read_packet(AVFormatContext *s, AVPacket
>> *pkt)
>> size = avpriv_dv_get_packet(c->dv_demux, pkt);
>>
>> if (size < 0) {
>> + int result;
>> +
>> if (!c->dv_demux->sys)
>> return AVERROR(EIO);
>> size = c->dv_demux->sys->frame_size;
>> - if (avio_read(s->pb, c->buf, size) <= 0)
>> - return AVERROR(EIO);
>> + result = avio_read(s->pb, c->buf, size);
>> + if (result <= 0)
>> + return result;
>>
>> size = avpriv_dv_produce_packet(c->dv_demux, pkt, c->buf, size);
>> }
>>
> Probably it is a stable candidate.
>
> nit: `ret` is the name normally used across the codebase for the
> variable instead of result.
>
>Yes, it would be nice for this to go into stable. result changed to ret locally. May I push? -- John GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01 83F0 49F1 D7B2 60D4 D0F7
signature.asc
Description: OpenPGP digital signature
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
