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

Diego Biurrun diego at biurrun.de
Sat Jul 5 13:25:59 CEST 2008


On Thu, Jul 03, 2008 at 02:35:56PM +0200, Nico Sabbi wrote:
> On Thursday 03 July 2008 12:47:49 Mathieu SCHROETER wrote:
> > 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)...
> 
> thanks, I'll review it tonight

So you reviewed the patch and committed, but why FOR FUCK'S SAKE did you
have to leave out the documentation part?

How often do I have to repeat this?  Never ever split off the
documentation part from a patch.  It causes extra work for *you* as well
as for everybody else.

Oh, and this violates II.8 and II.11 of the policy, in case you care.

Diego



More information about the MPlayer-dev-eng mailing list