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 731102 - PROP_SESSION_POOL and PROP_MOUNT_POINTS were not documented.
PROP_SESSION_POOL and PROP_MOUNT_POINTS were not documented.
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: documentation
1.x
Other All
: Normal normal
: NONE
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-06-02 11:07 UTC by Anuj
Modified: 2015-02-13 16:50 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
PROP_SESSION_POOL and PROP_MOUNT_POINTS Documented. (1.59 KB, patch)
2014-06-02 11:08 UTC, Anuj 		needs-work Details | Review

Description Anuj 2014-06-02 11:07:15 UTC 
gst-rtsp-server/gst/rtsp-server/rtspclient.c
 
In gst_rtsp_client_class_init()

PROP_SESSION_POOL and PROP_MOUNT_POINTS were not documented.
Comment 1 Anuj 2014-06-02 11:08:27 UTC 
Created attachment 277714 [details] [review]
PROP_SESSION_POOL and PROP_MOUNT_POINTS Documented.
Comment 2 Tim-Philipp Müller 2015-02-13 16:49:57 UTC 
Thanks, but these properties should be documented automatically via gtk-doc, and it seems they are:

http://gstreamer.freedesktop.org/data/doc/gstreamer/head/gst-rtsp-server/html/GstRTSPClient.html#GstRTSPClient.properties

The only reason to add explicit doc blurbs is if one needs to add a 'Since: 1.N' tag to it, or if one wants to add some longer description than the usual property description. I'm not sure if that's needed in this case?

Besides, your patch is not correct. The property is called "session-pool", not "session", and there seems to be an indentation issue too. Feel free to update the patch though if you think the longer property explanation is desirable. Thanks!