GNOME Bugzilla – Bug 683941
g_thread_pool_new() is not introspectable
Last modified: 2018-05-24 14:33:07 UTC
The constructor doesn't take any arguments and accessing func directly aborts: py> from gi.repository import GLib py> a = GLib.ThreadPool(func=callable) Traceback (most recent call last):
+ Trace 230847
py> a = GLib.ThreadPool() py> a.func = (lambda *args: None) ** ERROR:/home/martin/debian/pkg-gnome/build-area/pygobject-3.3.91/gi/pygi-argument.c:1202:_pygi_argument_from_object: code should not be reached Aborted
Indeed g_thread_pool_new() is shown as not introspectable in GLib-2.0.gir. Reassigning to glib for this. I'm afraid using the GObject constructor (GLib.ThreadPool(func=callable)) won't get you very far, as g_thread_pool_new() does nontrivial things (i. e. more than just setting properties). It may or may not be possible to move that functionality to g_thread_pool_init(), so that _new() becomes the usual small shim and the GObject constructor works.
I'm afraid this is not just an annotation problem. The user_data argument has a scope beyond a simple (call) or (async), so there needs to be a new constructor with a GDestroyNotify argument for user_data. However, even when I annotate this with a wrong scope, it still doesn't become introspectable. Out of interest, what's the use case for using the GLib implementation instead of Python's native threading module?
@glib developers: Feel free to close this as "won't fix" if you do not intend to make this API introspectable.
The idea was to use glib functions where possible to see how far this carries me. I'm fine with using Python's native threading but currently it's sometimes a bit hard to guess what should work and what shouldn't (without looking at the introspection data itself).
-- GitLab Migration Automatic Message -- This bug has been migrated to GNOME's GitLab instance and has been closed from further activity. You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/glib/issues/601.