[vlc-devel] [PATCH] Add MPEG audio layer I, II, III decoder based on mpg123

[Tristan Matthews]

> > +

> > > +    /* Feed mpg123 with raw data */
> > > +    i_err = mpg123_feed( p_sys->p_handle, p_block->p_buffer,
> > > +                         p_block->i_buffer );
> > > +
> > > +    if( i_err != MPG123_OK )
> > > +    {
> > > +        msg_Err( p_dec, "mpg123_feed failed: %s",
> mpg123_plain_strerror(
> > > i_err ) );
> > > +        block_Release( p_block );
> > > +        return NULL;
> > > +    }
> > > +
> > > +    /* Get details about the stream */
> > > +    i_err = mpg123_info( p_sys->p_handle, &p_sys->frame_info );
> > > +
> > > +    if( i_err == MPG123_NEED_MORE )
> > > +    {
> > > +        /* Need moar data */
> > > +        return NULL;
>

Should block_Release( p_block )  be called here or am I missing something?

Best,
Tristan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20141203/7acc89c1/attachment.html>