[Bug 705991] Adding support for DASH common encryption to qtdemux and dashdemux

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Sat Apr 18 04:35:57 PDT 2015


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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #301853|none                        |committed
             status|                            |

--- Comment #124 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Comment on attachment 301853
  --> https://bugzilla.gnome.org/attachment.cgi?id=301853
Creates gstprotection in GStreamer core

Pushed, with minor changes: (transfer-full) -> (transfer full) ; GST_DEBUG
strings don't need a newline at the end; added some gratuitious newlines in the
code. Also removed the "If the sample is not encrypted, @info may be %NULL."
from the gst_buffer_add_protection_meta() docs to make it consistent with the
function guards. Having said that, perhaps it does make sense to allow a NULL
info, but one could just as well signal an unencrypted sample with the absence
of the protection meta, no? Either way would work for me if you think it makes
something easier or is more elegant.

-- 
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