<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">Le dim. 20 janv. 2019 12 h 27, Sebastian Dröge <<a href="mailto:sebastian@centricular.com">sebastian@centricular.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, 2019-01-20 at 10:58 +1100, Matthew Waters wrote:<br>
> On Sun., 20 Jan. 2019, 10:28 Nicolas Dufresne <<a href="mailto:nicolas@ndufresne.ca" target="_blank" rel="noreferrer">nicolas@ndufresne.ca</a><br>
> wrote:<br>
> > <br>
> > Le sam. 19 janv. 2019 14 h 27, Sebastian Dröge <<br>
> > <a href="mailto:sebastian@centricular.com" target="_blank" rel="noreferrer">sebastian@centricular.com</a>> a écrit :<br>
> > > On Sat, 2019-01-19 at 11:38 -0500, Nicolas Dufresne wrote:<br>
> > > > Le samedi 19 janvier 2019 à 17:06 +0100, Philippe Lalevée a<br>
> > > écrit :<br>
> > > > > Hello I still have problems when using on-sending-rtcp/on-<br>
> > > receiving<br>
> > > > > signals (I would like to send RTCP packets of APP type). <br>
> > > > <br>
> > > > The RTPSession API isn't public, I'm not sure it is correct to<br>
> > > use<br>
> > > > this from an application. I believe you should better describe<br>
> > > what<br>
> > > > you are trying to do.<br>
> > <br>
> > I don't agree. The API being public would mean we install the<br>
> > appropriate header. These things are not event in GST namespace. As<br>
> > we expose the opaque object as a GObject, yes, we have made public<br>
> > the properties and signal, but we don't document it. I totally<br>
> > disagree with having to maintain API stability here for our<br>
> > internal RTP helpers.<br>
> <br>
> What makes you think the internal API as exposed through the GObject<br>
> properties and signals is not public?  My understanding is that it is<br>
> public API.  Just because there is no header, doesn't mean the API is<br>
> not public.  How do you explain element properties then or even<br>
> GObject properties in general which require no headers for use.<br>
> <br>
> Documentation of the internal rtpsession isn't exposed presumably due<br>
> to limitations in gtk-doc however it is written for public use.<br>
<br>
Also why would there otherwise be the "internal-session" GObject<br>
property on the rtpsession element, or the "get-internal-session"<br>
action signals on rtpbin?<br>
<br>
It's public API just like the properties and signals on the rtpsource<br>
objects, and even if it was not intended as such (which is not the case<br>
from what I understand) it would be too late now to change it in<br>
incompatible ways. There's too much code depending on them.<br>
<br>
The parts that are not public API are not exposed via GObject<br>
properties/signals and are only accessible internally via private C<br>
API.</blockquote></div></div><div dir="auto"><br></div><div dir="auto">Then I would say it's time to make a real effort and document these thing. I don't see any productive way we can support users of this here without. GTK doc limitation is a wrong argument, we have elements documentation, which does not have a C API.</div><div dir="auto"><br></div><div dir="auto">I strongly prefer saying it's not officially supported then repeating myself over and over due to lack of documentation / proper committment to make this un-ambiguously public.</div><div dir="auto"></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-- <br>
Sebastian Dröge, Centricular Ltd · <a href="https://www.centricular.com" rel="noreferrer noreferrer" target="_blank">https://www.centricular.com</a><br>
_______________________________________________<br>
gstreamer-devel mailing list<br>
<a href="mailto:gstreamer-devel@lists.freedesktop.org" target="_blank" rel="noreferrer">gstreamer-devel@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/gstreamer-devel</a><br>
</blockquote></div></div></div>