[vlc-devel] [PATCH] Refactored/re-enabled libvlc_vlm_show_media

Alexander Bethke abethke at oamk.fi
Mon Aug 10 10:03:00 CEST 2009


Rémi Denis-Courmont wrote:
> Le jeudi 6 août 2009 17:31:56 Alexander Bethke, vous avez écrit :
>   
>> +        if( psz_response )
>> +        {
>> +            free( psz_response );
>> +        }
>> +        psz_response = strdup( "" );
>>     
>
> free(NULL) is a no-op in C, so that condition is useless.
>
>   
Actually psz_response is initialised with strdup( "" ), nevertheless the 
condition is useless. I removed and it and made the initialisation of 
psz_response more readable. Resending the patch...

Regards, Alex



More information about the vlc-devel mailing list