Re: [Evolution-hackers] [PATCH 3/3] Add some missing includes

From: Pavel Roskin <proski gnu org>
To: Tor Lillqvist <tml novell com>
Cc: evolution-hackers gnome org
Subject: Re: [Evolution-hackers] [PATCH 3/3] Add some missing includes
Date: Tue, 15 Aug 2006 10:58:25 -0400

Hello, Tor!

On Tue, 2006-08-15 at 08:50 +0000, Tor Lillqvist wrote:
> må 2006-08-14 klockan 19:58 -0400 skrev Pavel Roskin:
> > +#include <gtk/gtkeventbox.h>
> > +#include <gtk/gtkimage.h>
> 
> Isn't the correct thing here to just include <gtk/gtk.h> and not attempt
> to hand-pick individual gtk headers?

Being quite new to all this, I'm trying to follow the existing style
rather than fix warnings and do The Right Thing at once.

It should be easy (for anyone knowing Perl, that is) to replace
gtk/gtk*.h with gtk/gtk.h across the board and then remove duplicate
headers.

-- 
Regards,
Pavel Roskin

References:
- [Evolution-hackers] [PATCH 1/3] Provide declaration for get_font_options()
  - From: Pavel Roskin
- [Evolution-hackers] [PATCH 3/3] Add some missing includes
  - From: Pavel Roskin
- Re: [Evolution-hackers] [PATCH 3/3] Add some missing includes
  - From: Tor Lillqvist

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