Re: Possible source error, 2.4.0

[Geoffrey Leach <geoff hughes net>]

On 06/25/2009 06:14:34 AM, Peter Bloomfield wrote:
> On 06/25/2009 01:10:40 AM Thu, Pawel Salek wrote:
> > On 06/25/2009 01:25:56 AM, Geoffrey Leach wrote:
> >> In the file spell-check.c, the position of
> >>   #include "spell-check.h"
> >> leads to compile failure, as GType is not defined at that point
> > 
> > Does the attached patch fix the problem?
> > 
> > Pawel
> 
> ------quoted attachment "spell-check-include.patch"------
> > diff --git a/src/spell-check.h b/src/spell-check.h
> > index c203c30..599f1ff 100644
> > --- a/src/spell-check.h
> > +++ b/src/spell-check.h
> > @@ -22,6 +22,8 @@
> >  #ifndef __BALSA_SPELL_CHECK_H__
> >  #define __BALSA_SPELL_CHECK_H__
> > 
> > +#include <glib-object.h>
> > +
> >  #define USE_ORIGINAL_MANAGER_FUNCS
> > 
> >  #ifdef __cplusplus
> 
> Pawel: I had to include <gtk/gtk.h> instead of <glib-object.h> to get
> it  
> to build.
> 
> Geoffrey: Any reason you're not building with-gtkspell?  I realize 
> the
>  
> default is without (perhaps we should change that), but it makes a
> much  
> better spell-checking experience.
> 
> Peter

You said it :-) I tend to avoid optional includes unless I know what 
I'm doing. Life tends to be simpler that way. However, I'm always up 
for a better experience! 

Geoffrey