[Bug 770107] qtdemux: reports only one encryption system even if it can support more than one

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=770107

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #352274|none                        |needs-work
             status|                            |

--- Comment #26 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 352274
  --> https://bugzilla.gnome.org/attachment.cgi?id=352274
1/3 protection: added function to filter system ids (review suggestions
applied)

>+/**
>+ * gst_protection_filter_systems_by_available_decryptors

This needs a colon at the end I believe?

>+ * Returns: (transfer full): A null terminated array containing all the
>+ * @system_identifiers supported by the set of available decryptors.

Apologies if I've asked this before already, but it seems if there are not
matches we currently return an array with str[0] = NULL. Wouldn't it be better
to return NULL in case there are no matches?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.