Ethereal-dev: Re: [Ethereal-dev] Re:[Ethereal-cvs] cvs commit: ethereal Makefile.am configure.

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Olivier Abad <oabad@xxxxxxx>
Date: Mon, 4 Nov 2002 08:16:17 +0100
On dim, nov 03, 2002 at 11:51:38 -0800, Guy Harris wrote:
> On Sun, Nov 03, 2002 at 07:06:53PM +0100, Joerg Mayer wrote:
> > Looking at the amount of resulting #ifdefs (GTK_MAJOR...) I'm not sure
> > that this was such a good idea.
> 
> Perhaps the next step should be to create "gtk12" and "gtk2"
> subdirectories and move stuff that has lots of #ifdefs there, and leave
> the rest of the stuff in "gtk".  For routines that are mostly the same,
> perhaps helper routines or macros could be put in "gtk12" and "gtk2",
> with the stuff in "gtk" calling it (e.g.:
> 
>     gtk12:
> 
> 	#define SIGNAL_CONNECT(widget, name, callback, arg) \
> 		gtk_signal_connect(GTK_OBJECT(widget), name, \
>                      GTK_SIGNAL_FUNC(callback), \
> 		     (arg) == NULL ? NULL : GTK_OBJECT(arg));
> 
>     gtk2:
> 
> 	#define SIGNAL_CONNECT(widget, name, callback, arg) \
> 		g_signal_connect(G_OBJECT(widget), name, \
>                    G_CALLBACK(callback), \
>                    (arg) == NULL ? NULL : G_OBJECT(arg));

I should have done this while merging. I'll work on it later.

Olivier
-- 
Humor in the Court:
Q: Did you tell your lawyer that your husband had offered you indignities?
A: He didn't offer me nothing; he just said I could have the furniture.