Re: [GnomeMeeting-devel-list] [PATCH] (src/)menu.h and (lib/)gtk_menu_extensions.h



Le lun, 05/04/2004 à 14:43 +0200, PUYDT Julien a écrit :

> Hi,
> 
> those where always both included, while in some cases, only one was
> really used directly. That patch fixes that.
> 

I don't understand what you mean.


> Of course, I kept both included even if menu.h includes
> gtk_menu_extensions.h: it is good to see direct dependancies at the
> beginning of the files! 
> 

My policy is that if a function from a header is included in a .cpp
file, the header must be included even though it is already included
through another file. If your patch doesn't break this, you can commit
it as cleanup.


> Snark

-- 
 _      Damien Sandras
(o-     GnomeMeeting: http://www.gnomemeeting.org/
//\     FOSDEM:  http://www.fosdem.org
v_/_    
        

Attachment: signature.asc
Description: Ceci est une partie de message =?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]