[MPlayer-dev-eng] [PATCH] add checkmark for menuitem for Debug Console

Reimar Döffinger Reimar.Doeffinger at gmx.de
Tue Aug 14 21:07:22 CEST 2012


On Tue, Aug 14, 2012 at 08:55:16PM +0200, Ingo Brückl wrote:
> Reimar Döffinger wrote on Tue, 14 Aug 2012 20:04:46 +0200:
> 
> > It should basically do the same thing as before, since those are
> > supposed to be NULL there.
> 
> Using NULL makes the code easier to understand. Like Stephen wrote, it's
> counter-intuitive to pass in the handle that we were creating.

Yes, of course, I very much agree. Just pointing out that knowing Windows
API is rather irrelevant to that change :-)


More information about the MPlayer-dev-eng mailing list