Re: A cross-platform status icon api
- From: Matthias Clasen <mclasen redhat com>
- To: Hans Breuer <hans breuer org>
- Cc: gtk-devel-list gnome org
- Subject: Re: A cross-platform status icon api
- Date: Tue, 06 Sep 2005 17:08:21 +0400
On Mon, 2005-09-05 at 23:25 +0200, Hans Breuer wrote:
> On 04.09.2005 22:59, Matthias Clasen wrote:
> > On Sun, 2005-09-04 at 15:27 +0200, Hans Breuer wrote:
> >>On 30.08.2005 00:37, Matthias Clasen wrote:
> >>>cvs as of today has
> >>Trying to make it compile on win32 implies that two files
> >>should be renamed :
> >>gtkstatusicon-x11.c has nothing X11 specific in it. At least
> >>it compiles on win32 without any change. So it probably should
> >>be named gtkstatusicon.c
> >>gtktrayicon.c is highly X11 dependent so it should be
> >>called gtktrayicon-x11.c
> >>I have done the renaming in my local tree and also created
> >>a currently empty implementation of gtktrayicon-win32.c.
> >>With it HEAD could be compiled again on win32.
> > No, this is backwards. gtkstatusicon-x11.c is named like it is because
> > it is the X11-specific implementation of the statusicon api.
> Yeah, trying to implement something with it I noticed that, too.
> > gtktrayicon.c may compile on win32, but that is completely irrelevant,
> gtkstatusicon-x11 does, gtktrayicon.c does *not*. How do you suggest to
> call the win32 specific file ?
I thought it would be gtkstatusicon-win32.c, following the example of
how the plug/socket code is organized.
> Also gtkstatusicon-x11.c not only implements the fdo spec but it also
> defines a bunch of status icon api. Not only 21 functions but additionally
> quite some properties. Should all this be duplicated to get the
> cross-platform status api ?
Hmm, good point. When I did that code I thought a clean separation
between the X and win32 implementations would be better, but looking at
the code in gtkstatusicon-x11.c, only a few lines deal with the tray
icon. Do you think it would be feasible to #ifdef out the
platform-specifc parts without too much pain ? If so, then we should
probably do the cvs surgery to rename gtktrayicon.c to gtktrayicon-x11.c
and gtkstatusicon-x11.c gtkstatusicon.c.
] [Thread Prev