[Bug 797239] qtdemux: Crash when playing smooth-streaming
GStreamer (GNOME Bugzilla)
bugzilla at gnome.org
Fri Oct 19 18:18:47 UTC 2018
https://bugzilla.gnome.org/show_bug.cgi?id=797239
Tim-Philipp Müller <t.i.m at zen.co.uk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #373975|none |reviewed
status| |
--- Comment #19 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 373975
--> https://bugzilla.gnome.org/attachment.cgi?id=373975
qtdemux: Don't use g_ptr_array_find_with_equal_func()
>+/* g_ptr_array_find_with_equal_func is available since 2.54,
>+ * replacement until we can depend unconditionally on the real one in GLib */
>+static gboolean
>+qtdemux_ptr_array_find_with_equal_func (GPtrArray * haystack,
>+ gconstpointer needle, GEqualFunc equal_func, guint * index_)
>+{
>...
>+}
I think this should be done conditionally, like this:
#if !GLIB_CHECK_VERSION(2,56,0)
#define g_ptr_array_find_with_equal_func qtdemux_ptr_array_find_with_equal_func
>+static gboolean
>+g_ptr_array_find_with_equal_func (GPtrArray * haystack,
>+ gconstpointer needle, GEqualFunc equal_func, guint * index_)
>+{
>...
>+}
#endif
which makes it easier to grep for it later when we bump the GLib req, and
avoids the duplication if GLib is new enough.
--
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