[Bug 793221] Add support for ONVIF backchannel

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Tue Feb 6 17:49:29 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=793221

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |t.i.m at zen.co.uk

--- Comment #1 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Shouldn't there be padding in the new class and instance structures for the
onvif subclasses? We might want to add more vfuncs in future, for example, no?

The vfunc extension hack is a bit unfortunate, but still seems better than
adding 50 new vfuncs just for that extra context parameter. I wonder what
g-i/bindings make of the #ifdef #else there. The only alternative I can think
of is to push the context into thread-local storage before ->vfunc and pop it
off after.

Does the new vfunc define also need to be set in the meson build?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list