Re: [evolution-patches] Various fixes for e-shell-importer.c and mail importers



Le lun 23/08/2004 à 05:22, Not Zed a écrit :
> On Fri, 2004-08-20 at 19:22 +0200, Frederic Crozat wrote: 
> > Le ven 20/08/2004 à 19:10, Not Zed a écrit :
> > > > -gtk_dialog_set_has_separator shouldn't be used on message dialog with
> > > > GTK+ >= 2.4.0 (it causes a warning)
> > > 
> > > Do we really want to do this?  We still want it to run on  < gtk 2.4.0
> > > i presume.
> > 
> > Yes, this is why I used GTK_CHECK_VERSION in the patch..
> Yeah but all the #ifdef crap is a real pain in the arse.
> 
> Should we just remove it anyway, doesn't it default to off now?

Only on GTK+ 2.4 and since Evo doesn't yet required 2.4, I had to use
such #ifdef. When 2.4 will be required, it will be just a matter of
removing this #ifdef code.

I've check GTK+ 2.2 code and this code was required to remove separator
for message dialog (or you had to use GTK_DIALOG_NO_SEPARATOR in the
constructor but in that case, GTK+ 2.4 also output a warning).

-- 
Frederic Crozat <fcrozat mandrakesoft com>
Mandrakesoft





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