Re: [MPlayer-dev-eng] Fwd: Re: [PATCH] add optional device with dvd://

[Nico Sabbi <nicola.sabbi@xxxxxxxx>]

Il giorno ven, 04/07/2008 alle 17.53 +0200, Nico Sabbi ha scritto:
> 
> 
> ---------- Forwarded message ----------
> From: Nico Sabbi <Nicola.Sabbi@xxxxxxxx>
> Date: 2008/7/3
> Subject: Fwd: Re: [MPlayer-dev-eng] [PATCH] add optional device with
> dvd://
> To: Nico Sabbi <nico.sabbi@xxxxxxxxx>
> 
> 
> 
> 
> 
> ---------- Messaggio inoltrato ----------
> From: Mathieu SCHROETER <mathieu.schroeter@xxxxxxxxxxxx>
> To: mplayer-dev-eng@xxxxxxxxxxxx
> Date: Thu, 03 Jul 2008 12:47:49 +0200
> Subject: Re: [MPlayer-dev-eng] [PATCH] add optional device with dvd://
> Mathieu SCHROETER a écrit :
> >+  if(!p->device)
> >+    dvd_device_current = strdup(dvd_device ? dvd_device :
> DEFAULT_DVD_DEVICE);
> 
> Sorry previous patch is wrong because a leak exists on
> dvd_device_current. Now strdup() is only used on dvd_device
> (if necessary)...
> 
> 
> Regards,
> Mathieu SCHROETER
> 
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng@xxxxxxxxxxxx
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> 
> 

thanks, applied in pieces, but guy: you like to make
controls a bit complicated with all those ! conditions
(that I simplified)

_______________________________________________
MPlayer-dev-eng mailing list
MPlayer-dev-eng@xxxxxxxxxxxx
https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng