gst/gstbuffer.c: Update some comments.

Original commit message from CVS:
* gst/gstbuffer.c:
Update some comments.
* tools/gst-inspect.c: (print_element_properties_info):
Improve printing of flags.
This commit is contained in:
Wim Taymans 2008-01-09 12:19:31 +00:00
parent 58338a041b
commit c7d40b0a25
3 changed files with 15 additions and 7 deletions

View file

@ -1,3 +1,11 @@
2008-01-09 Wim Taymans <wim.taymans@collabora.co.uk>
* gst/gstbuffer.c:
Update some comments.
* tools/gst-inspect.c: (print_element_properties_info):
Improve printing of flags.
2008-01-08 Tim-Philipp Müller <tim at centricular dot net> 2008-01-08 Tim-Philipp Müller <tim at centricular dot net>
* libs/gst/base/gstbasetransform.c: * libs/gst/base/gstbasetransform.c:

View file

@ -380,8 +380,9 @@ gst_buffer_try_new_and_alloc (guint size)
* Returns: a reference to the #GstCaps. unref after usage. * Returns: a reference to the #GstCaps. unref after usage.
* Returns NULL if there were no caps on this buffer. * Returns NULL if there were no caps on this buffer.
*/ */
/* FIXME can we make this threadsafe without a lock on the buffer? /* this is not made atomic because if the buffer were reffed from multiple
* We can use compare and swap and atomic reads. */ * threads, it would have a refcount > 2 and thus be immutable.
*/
GstCaps * GstCaps *
gst_buffer_get_caps (GstBuffer * buffer) gst_buffer_get_caps (GstBuffer * buffer)
{ {
@ -406,10 +407,9 @@ gst_buffer_get_caps (GstBuffer * buffer)
* be increased and any previous caps on the buffer will be * be increased and any previous caps on the buffer will be
* unreffed. * unreffed.
*/ */
/* FIXME can we make this threadsafe without a lock on the buffer? /* this is not made atomic because if the buffer were reffed from multiple
* We can use compare and swap and atomic reads. Another idea is to * threads, it would have a refcount > 2 and thus be immutable.
* not attach the caps to the buffer but use an event to signal a caps */
* change. */
void void
gst_buffer_set_caps (GstBuffer * buffer, GstCaps * caps) gst_buffer_set_caps (GstBuffer * buffer, GstCaps * caps)
{ {

View file

@ -511,7 +511,7 @@ print_element_properties_info (GstElement * element)
} }
n_print n_print
("%-23.23s Flags \"%s\" Default: %d, \"%s\" Current: %d, \"%s\"", ("%-23.23s Flags \"%s\" Default: 0x%08x, \"%s\" Current: 0x%08x, \"%s\"",
"", g_type_name (G_VALUE_TYPE (&value)), pflags->default_value, "", g_type_name (G_VALUE_TYPE (&value)), pflags->default_value,
(def_flags ? def_flags->str : "(none)"), flags_value, (def_flags ? def_flags->str : "(none)"), flags_value,
(cur_flags ? cur_flags->str : "(none)")); (cur_flags ? cur_flags->str : "(none)"));