[vlc-devel] [PATCH] better libvlc audio handling

Lukas Durfina lukas.durfina at gmail.com
Mon Nov 3 22:23:58 CET 2008


On Mon, 03 Nov 2008 19:49:16 +0100, Rémi Duraffort <ivoire at via.ecp.fr>  
wrote:

> Hi,
> just a little remark.
>
> In this function:
> VLC_PUBLIC_API void libvlc_audio_output_device_set(libvlc_instance_t

>
> Why not checking  that psz_device_id != NULL at the begining, so if it's
> NULL you don't do any malloc+free.
>
> Seams good to me.
>
> And thanks for redoing patch for every remarks !!
>

Hi,

  good point... and thanks for remarks, without them it wouldnt be
pretty code.

Lukas

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-libvlc-better-audio-handling.patch
Type: application/octet-stream
Size: 21373 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20081103/e8b7a389/attachment.obj>


More information about the vlc-devel mailing list