Re: [GnomeMeeting-devel-list] [PATCH] compilation warnings

[PUYDT Julien <julien puydt laposte net>]

On ven, 2004-01-30 at 16:25, Stefan Bruens wrote:
> I think the part below ist wrong, should not it be "} else {"?

Hmmm... no, I added the braces because it wasn't clear what the external
if applied too (and the compiler complained), but for the internal if,
things are quite clear.

If you ask, then the compiler was most certainly right to complain ;-)

Snark

> @@ -294,13 +294,14 @@
>                              menu [i].widget);      
>  
>      
> -    if (menu [i].id)
> +    if (menu [i].id) {
>        if (menu [i].type != MENU_SUBMENU_NEW)
>         g_object_set_data (G_OBJECT (menubar), menu [i].id,
>                            menu [i].widget);
>        else
>         g_object_set_data (G_OBJECT (menubar), menu [i].id,
>                            menu_widget);
> +    }
>        
> Greets,
> 
> Stefan