[Webkit-unassigned] [Bug 26103] [GTK] Remove gtk/cairo/pango/freetype/fontconfig includes from the header files in WebCore

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jun 2 06:57:54 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=26103





------- Comment #8 from zecke at selfish.org  2009-06-02 06:57 PDT -------
(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #5)
> > > Disclaimer: I am not a WebKit developer.
> > > 
> > > First of all, sorry for misunderstanding the purpose of your patch.
> > > 
> > > But I do dislike your patch since it breaks the proper abstractions.
> > > An example:
> > 
> > Okay, I think you are slightly off again + my description not being too good.
> > My goal is to not have any glib/glib.h, cairo.h in WebCore header files as this
> > is increasing compile times (see this simple patch can save quite some time).
> > 
> > Way to get there include:
> >   - use forward declarations... and they are typesafe, e.g. if I declare
> > typedef struct _GtkEntry GtkWidget the compiler will shout at me.
> > 
> >   - do not use Glib types when not necessary. Using unsigned long long was a
> > bit too much to store the value 8192, so an updated patch will use unsigned.
> > See how I just didn't do typedef unsigned long gsize?
> >...
> 
> OK, so this seems to work.
> 
> But what would e.g. happen if with a future (ABI-incompatible) version of glib
> "typedef gint gboolean;" would no longer be true?

then your compiler will give you an error about incompatible types.

> Overally, my personal opinion is that your patch is going into the wrong
> direction - even if my suspicion that it might also be wrong on some technical
> details turns out to be wrong.

Please elaborate on it. Do you disagree with forward declarations in general?
for glib types? I know it is a bigger difference from C to C++. But forward
declarations are a pretty common way to speed compilation up. We use it
extensively in WebKit. Or in a more dramatic way... why do you think
MediaQueryEvaluator.cpp needs to know about GIO?


-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list