[Glade-devel] [glade3, patch] load library module before calling *_get_type ()
Joaquin Cuenca Abela
e98cuenc@yahoo.com
Fri, 9 Jan 2004 06:51:13 -0800 (PST)
Hi!
--- Tommi Komulainen <tommi.komulainen@nokia.com>
wrote:
> On Thu, 2004-01-08 at 22:45, ext Joaquin Cuenca
> Abela wrote:
>
> [the "libglade" + foo -library instead of "libfoo"]
>
> > That's the library that it's loaded by
> g_module_open in the code that you
> > move with your patch, and it's unrelated to the
> library that contains the
> > real Foo widget.
>
> Is it really unrelated? It's using the widgets in
> the widget library
> and thus needs to be linked with it, doesn't it?
Indeed, yes. I missed that loading the helping lib
(libgladefoo) will automatically open libfoo.
[snip]
> In essence the whole module loading could actually
> be as simple as the
> following (unless I'm missing some fundamental
> reason for constructing a
> strictly formatted library name - I think it's just
> unnecessary.)
> "library" is verbatim the attribute value, error
> checking and all
> ignored for brevity:
>
> module = g_module_load (g_module_build_path
> (MODULES, library));
> if (!module)
> module = g_module_load (g_module_build_path (NULL,
> library));
That makes perfect sense to me. If none else
disagrees, feel free to commit a patch with your
suggested changes, which should equal to your earliest
patch + this change.
Thank you for your collaboration!
Cheers,
>
>
> At least that's how I've understood g_module_open
> works, please correct
> me if I'm wrong.
=====
Joaquin Cuenca Abela
e98cuenc at yahoo dot com
__________________________________
Do you Yahoo!?
Yahoo! Hotjobs: Enter the "Signing Bonus" Sweepstakes
http://hotjobs.sweepstakes.yahoo.com/signingbonus