[MPlayer-cvslog] r36944 - in trunk/gui/win32: interface.c playlist.c playlist.h
Ingo Brückl
ib at wupperonline.de
Tue Feb 25 19:39:10 CET 2014
Reimar Döffinger wrote on Tue, 25 Feb 2014 19:01:27 +0100:
>> @@ -501,6 +501,8 @@ static unsigned __stdcall GuiThread(void
>> {
>> MSG msg;
>>
>> + (void) param;
> This is a bit ugly and might not work in all compilers.
I'd say it'll work on most (all?) compilers. Yes, it's ugly.
> I think using av_unused on the parameter would be nicer,
I don't mind using it, but it would be nice if gcc would warn if such a
parameter *will* be used later. (This is the only thing I dislike about the
(void) solution. It'll look strange if you forget to remove it later. With
av_unused it'll even look stranger.)
Ingo
More information about the MPlayer-cvslog
mailing list