[vlc-devel] [PATCH] New mpg123 audio converter

Ludovic Fauvet etix at videolan.org
Tue Sep 3 14:26:38 CEST 2013


On 09/03/2013 02:12 PM, Ludovic Fauvet wrote:
>>> +/*****************************************************************************
>>> + * ExitMPG123 : close down the mpg123 library (reentrant)
>>> + *****************************************************************************/
>>> +static void ExitMPG123( void )
>>> +{
>>> +    vlc_mutex_lock( &mpg123_mutex );
>>> +    if( mpg123_refcount > 1 )
>>> +    {
>>> +        mpg123_refcount--;
>>> +        vlc_mutex_unlock( &mpg123_mutex );
>>> +        return;
>>> +    }
>>
>> Missing mpg123_refcount--
> 
> We don't want to decrease the refcount if we're already at 0, do we? :)

Nevermind, you're right.

>>> +    mpg123_exit();
>>> +    vlc_mutex_unlock( &mpg123_mutex );
>>> +}
>>>

-- 
Ludovic Fauvet
www.videolan.org




More information about the vlc-devel mailing list