Re: GdkLibrary

Sven Neumann <sven gimp org> writes:

> Hi,
> Owen Taylor <otaylor redhat com> writes:
> > > while you are changing this part of GDK anyway, would it hurt to to
> > > allow GDK backends to modify the argv array just like gtk_init() and
> > > gdk_init()? This would remove the need for some patching we had to do
> > > to integrate DirectFBInit (int *argc, char **argv[]).
> > > 
> > > I haven't found the time to look at the HEAD branch, but if you think
> > > this makes sense, I could try to come up with a patch soon.
> > 
> > Did you miss the _gdk_windowing_args[] functionality? The only
> > reason that wouldn't work is if the directfb support is taking
> > arguments that don't match that format of the rest of the
> > GTK+ args. (which seems like a bad idea to me.)
> I didn't miss it but I didn't want to add all DirectFB command-line
> options by hand since there are a _lot_ of them. In case you're
> interested, I'm posting the patch we use at the moment.  It adds the
> new type GDK_ARG_WINDOWING_OPTION which is detected by matching the
> start of the argument string against a backend supplied prefix. This
> works well for DirectFB since all our command-line arguments are
> prefixed with 'dfb:' but I don't consider this hack overly nice and
> would prefer if _gdk_windowing_init_check() would be able to
> manipulate its arguments just like gtk_init() and gdk_init().

Oh, you are saying, you don't parse them yourself, you need to pass them 
all through to directfb?

If you provide the patch to add argc, argv, gtk-windowing-init(), I'll
check it over; I'd rather see this than GDK_ARG_WINDOWING_OPTION;
GDK_ARG_WINDOWING_OPTION seems to be very specific to the DirectFB way
of doing things, and confusing because most of the time it _won't_ be
used for windowing system specific options


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