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 780945 - GstPushSrc.do_create and Gst.BaseSrc.do_query are not introspected properly in bindings (Python)
GstPushSrc.do_create and Gst.BaseSrc.do_query are not introspected properly i...
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
unspecified
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2017-04-05 17:58 UTC by César Fabián Orccón Chipana
Modified: 2018-11-03 12:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add annotations to vfuncs in base sources (1.83 KB, patch)
2017-04-05 17:58 UTC, César Fabián Orccón Chipana
needs-work Details | Review

Description César Fabián Orccón Chipana 2017-04-05 17:58:00 UTC
Created attachment 349314 [details] [review]
Add annotations to vfuncs in base sources

For example,
When you use "do_create" vfunc in Python for a GstBaseSrc, you need to define the following method:

def do_create(self, buff):
    # Implementation here

But what you actually should do is
def do_create(self):
    # Implementation here
    return buff

I am uploading a patch that fixes that.


PD: I also think that all vfuncs should be annotated.
Comment 1 Nicolas Dufresne (ndufresne) 2017-04-05 18:04:52 UTC
Review of attachment 349314 [details] [review]:

That's interesting. Is that a new GI feature ? If so, which version is required ? Problem though is that you are duplicating some doc, so I'd say if we go that route, we should move all the doc to use comment blocks.
Comment 2 Sebastian Dröge (slomo) 2017-04-09 08:15:07 UTC
(In reply to Nicolas Dufresne (stormer) from comment #1)
> Review of attachment 349314 [details] [review] [review]:
> 
> That's interesting. Is that a new GI feature ? If so, which version is
> required ? Problem though is that you are duplicating some doc, so I'd say
> if we go that route, we should move all the doc to use comment blocks.

What are you talking about? These are normal GI annotations :)


(In reply to César Fabián Orccón Chipana from comment #0)
> 
> PD: I also think that all vfuncs should be annotated.

Why do they all need to be annoted? There should be default handling like for other functions, and what you added as annotations here should be the default for these vfuncs. This looks like something broken in GI.
Comment 3 GStreamer system administrator 2018-11-03 12:40:23 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org'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.freedesktop.org/gstreamer/gstreamer/issues/227.