[MPlayer-cvslog] r32624 - in trunk: TOOLS/vfw2menc.c gui/app.c gui/interface.c gui/mplayer/gtk/pl.c gui/mplayer/gtk/sb.c gui/mplayer/play.c gui/skin/font.c gui/win32/dialogs.c gui/win32/gui.c gui/win32/playlist.c g...

Clément Bœsch ubitux at gmail.com
Mon Nov 15 23:42:34 CET 2010


On Mon, Nov 15, 2010 at 11:19:39PM +0100, Diego Biurrun wrote:
> On Sun, Nov 14, 2010 at 10:12:35AM +0100, cboesch wrote:
> > 
> > Log:
> > Remove most of the NULL pointer check before free all over the code
> 
> I'd appreciate if you could be a tad more verbose in commit messages.
> In general a commit message should answer two questions: What and why.
> This commit message only answers the former question.
> 
> NULL pointer checks before free() are unnecessary, but very common.
> A few words that explain why it is safe to remove them would enlighten
> the people that come after you, many of which possibly won't know that
> it is safe to remove those checks.
> 

Edited. Should be better now.

> Diego

-- 
Clément B.
Not sent from a jesusPhone.


More information about the MPlayer-cvslog mailing list