Re: [Pingus-CVS] CVS: Games/Pingus/src/gui gui_manager.cxx,1.9,1.10

[Ingo Ruhnke]

address@hidden writes:

>  GUIManager::add (Component* c, bool delete_component) 
>  { 
>    components.push_back(c); 
> +  if(delete_component);
>  }

I find these empty 'if's rather ugly. We should probally replace them
with something like:

#define UNUSED_ARGUMENT (x) (void)(x)

#define UNUSED_ARGUMENT (x) if(x);else;

#ifdef GCC
#define UNUSED_ARGUMENT (x) __attribute__ ((unused)) x
#endif

or what ever the compiler likes most.

The last one could be used like:

void foobar (float x, int UNUSED_ARGUMENT (y))... 

while the other ones would be placed in the function body:

void foobar (float x, int y) {
        UNUSED_ARGUMENT (y);
        ...
}

Not sure if windows provides something simalar to the __attribute__
thing of GCC.

-- 
WWW:      http://pingus.seul.org/~grumbel/ 
Games:    http://pingus.seul.org/~grumbel/gamedesigns/
JabberID: address@hidden 
ICQ:      59461927