[vlc-devel] [PATCH 3/3] Add microdns services_advertisement submodule
remi at remlab.net
Tue Sep 3 17:19:04 CEST 2019
Le maanantaina 2. syyskuuta 2019, 19.34.58 EEST Thomas Guillem a écrit :
> One other concern:
> SA modules should not annonce anything by default but the core should ask
> for it.
> I propose the following changes:
> - The sa module initialize from Open but doesn't annonce anything, it just
> registers an annonce callback. - Add vlc_sa_annonce( libvlc, const char
> *name, const char *type, unsigned port ), that will call the annonce
> callback of the main SA.
What makes you think a port number describe any and all types of service
announcements? The only currently supported announce types is SDP, and that's
definitely carrying a hell of a lot more data than a port number.
I rather think that the activation callback should start advertising straight
away, because I don't see how you can make a generic "add" or "start" callback
More information about the vlc-devel