Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-21 Thread Diego Biurrun
On Fri, Jan 18, 2013 at 11:51:30PM +0100, Jindřich Makovička wrote: > On Fri, Jan 18, 2013 at 10:38 PM, Diego Biurrun wrote: > > On Thu, Jan 17, 2013 at 04:26:34PM +0100, Jindřich Makovička wrote: > >> > >> From 37eb9b56534b8ffae296510feaf37a8d42e764aa Mon Sep 17 00:00:00 2001 > >> From: Jindrich

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-18 Thread Jindřich Makovička
On Fri, Jan 18, 2013 at 10:38 PM, Diego Biurrun wrote: > On Thu, Jan 17, 2013 at 04:26:34PM +0100, Jindřich Makovička wrote: >> >> From 37eb9b56534b8ffae296510feaf37a8d42e764aa Mon Sep 17 00:00:00 2001 >> From: Jindrich Makovicka >> Date: Thu, 17 Jan 2013 16:24:28 +0100 >> Subject: [PATCH] avidec

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-18 Thread Diego Biurrun
On Thu, Jan 17, 2013 at 04:26:34PM +0100, Jindřich Makovička wrote: > > From 37eb9b56534b8ffae296510feaf37a8d42e764aa Mon Sep 17 00:00:00 2001 > From: Jindrich Makovicka > Date: Thu, 17 Jan 2013 16:24:28 +0100 > Subject: [PATCH] avidec: use sensible error codes instead of -1 > > Use AVERROR_INVA

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Jindřich Makovička
On Thu, Jan 17, 2013 at 4:09 PM, Diego Biurrun wrote: >> >> On Thu, Jan 17, 2013 at 11:57 AM, Jindřich Makovička >> >> wrote: >> >> --- a/libavformat/avidec.c >> >> +++ b/libavformat/avidec.c >> >> @@ -1015,8 +1015,9 @@ static int avi_read_packet(AVFormatContext *s, >> >> AVPacket *pkt) >> >>

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Diego Biurrun
On Thu, Jan 17, 2013 at 01:56:17PM +0100, Jindřich Makovička wrote: > On Thu, Jan 17, 2013 at 12:37 PM, Diego Biurrun wrote: > > On Thu, Jan 17, 2013 at 12:22:49PM +0100, Jindřich Makovička wrote: > >> On Thu, Jan 17, 2013 at 11:57 AM, Jindřich Makovička > >> wrote: > >> --- a/libavformat/avidec

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Jindřich Makovička
On Thu, Jan 17, 2013 at 12:37 PM, Diego Biurrun wrote: > On Thu, Jan 17, 2013 at 12:22:49PM +0100, Jindřich Makovička wrote: >> On Thu, Jan 17, 2013 at 11:57 AM, Jindřich Makovička >> wrote: >> > >> > these patches change return -1 in avidec to return AVERROR(EINVAL), >> > except one case where

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Diego Biurrun
On Thu, Jan 17, 2013 at 12:22:49PM +0100, Jindřich Makovička wrote: > On Thu, Jan 17, 2013 at 11:57 AM, Jindřich Makovička > wrote: > > > > these patches change return -1 in avidec to return AVERROR(EINVAL), > > except one case where the return -1 corresponds to an EOF condition in > > an interle

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Luca Barbato
On 17/01/13 12:22, Jindřich Makovička wrote: > I forgot to amend a lost semicolon before sending, here are fixed patches. Thank you, looks fine to me. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Jindřich Makovička
I forgot to amend a lost semicolon before sending, here are fixed patches. On Thu, Jan 17, 2013 at 11:57 AM, Jindřich Makovička wrote: > Hi, > > these patches change return -1 in avidec to return AVERROR(EINVAL), > except one case where the return -1 corresponds to an EOF condition in > an interl

[libav-devel] [PATCH] sensible error codes for avidec

2013-01-17 Thread Jindřich Makovička
Hi, these patches change return -1 in avidec to return AVERROR(EINVAL), except one case where the return -1 corresponds to an EOF condition in an interleaved AVI file. Regards, -- Jindřich Makovička 0001-avidec-return-EOF-when-no-more-frames-are-available.patch Description: Binary data 0002-a