Please trim your quotes, thank you.

On Wed, May 04, 2011 at 01:25:29PM -0700, Aℓex Converse wrote:
> On Wed, May 4, 2011 at 12:21 PM, Diego Biurrun <[email protected]> wrote:
> > From: Baptiste Coudurier <[email protected]>
> >
> > --- /dev/null
> > +++ b/libavcodec/s302m.c
> > @@ -0,0 +1,141 @@
> > +static int s302m_decode_frame(AVCodecContext *avctx,
> > +                              void *data, int *data_size,
> > +                              AVPacket *avpkt)
> > +{
> > +    const uint8_t *buf = avpkt->data;
> > +    int buf_size       = avpkt->size;
> > +
> > +    if (s302m_parse_frame_header(avctx, buf, buf_size) < 0)
> > +        return AVERROR_INVALIDDATA;
> 
> Why not return the error you get like my patch does.

I missed an updated patch from you where you return it and just
saw one where you return -1.  Updated patch coming up...

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to