After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 637215 - Don't skip vfuncs that have a corresponding signal
Don't skip vfuncs that have a corresponding signal
Status: RESOLVED FIXED
Product: gobject-introspection
Classification: Platform
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: gobject-introspection Maintainer(s)
gobject-introspection Maintainer(s)
Depends on:
Blocks: 635460
 
 
Reported: 2010-12-14 09:17 UTC by Tomeu Vizoso
Modified: 2015-02-07 16:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Don't skip vfuncs that have a corresponding signal (1.83 KB, patch)
2010-12-14 09:17 UTC, Tomeu Vizoso
committed Details | Review
Remove warning about a vfunc not having an invoker (2.54 KB, patch)
2010-12-14 09:36 UTC, Tomeu Vizoso
committed Details | Review

Description Tomeu Vizoso 2010-12-14 09:17:29 UTC
.
Comment 1 Tomeu Vizoso 2010-12-14 09:17:31 UTC
Created attachment 176391 [details] [review]
Don't skip vfuncs that have a corresponding signal

Otherwise subclasses don't have a mean to override the parent's
behavior.
Comment 2 Tomeu Vizoso 2010-12-14 09:36:50 UTC
Created attachment 176393 [details] [review]
Remove warning about a vfunc not having an invoker

As library authors don't want to add those invokers to their
public APIs. Bindings need to be able to register function wrappers
from the vfunc info.
Comment 3 Paolo Borelli 2010-12-18 11:14:32 UTC
Shouldn't this be filed against g-i? (the patches are for the scanner...)


Also it is probably the same problem reported in bug #635460 which tries to fix it on the pygobject side
Comment 4 Tomeu Vizoso 2010-12-18 12:43:55 UTC
(In reply to comment #3)
> Shouldn't this be filed against g-i? (the patches are for the scanner...)
> 
> 
> Also it is probably the same problem reported in bug #635460 which tries to fix
> it on the pygobject side

Oops, thanks for telling, I filed in the wrong component.
Comment 5 Colin Walters 2011-01-07 15:48:57 UTC
Review of attachment 176391 [details] [review]:

There's really no test updates for this one?  Did you run "make check"?

If there's no tests we should add one.
Comment 6 Colin Walters 2011-01-07 15:50:32 UTC
Review of attachment 176393 [details] [review]:

Oh I see, this one has the test updates for the previous one.

Can you squash to preserve bisectability?

I'm sad to see this check go; one tricky thing is that for some vfuncs the invoker has important default behavior (e.g. in gio).  But I guess we can't warn for all of them.
Comment 7 André Klapper 2015-02-07 16:50:37 UTC
[Mass-moving gobject-introspection tickets to its own Bugzilla product - see bug 708029. Mass-filter your bugmail for this message: introspection20150207 ]