On Fri, 29 Apr 2016, Diego Biurrun wrote:

On Fri, Apr 29, 2016 at 04:18:21PM +0300, Martin Storsjö wrote:
We still only support one single layer though, but this allows
receiving streams that has this structure present even for

have

Fixed locally


--- a/libavformat/rtpdec_vp9.c
+++ b/libavformat/rtpdec_vp9.c
@@ -222,8 +222,64 @@ static int vp9_handle_packet(AVFormatContext *ctx, 
PayloadContext *rtp_vp9_ctx,
      *   X: 1 if this layer index has an extended relative Picture ID.
      */
     if (has_ss_data) {
-        avpriv_report_missing_feature(ctx, "VP9 scalability structure data");
-        return AVERROR(ENOSYS);
+        if (n_s > 0) {
+            avpriv_report_missing_feature(ctx, "VP9 scalability structure with 
multiple layers");
+            return AVERROR(ENOSYS);

AVERROR_PATCHWELCOME?

Yes, you have had AVERROR(ENOSYS) before, but we have mostly
PATCHWELCOME throughout the codebase.

It practically doesn't matter here, since the rtpdec framework just checks the return value for <0, 0 or >0, and the value never gets forwarded to the caller outside of the demuxer. But I guess I can change if you want to...

// Martin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to