Merge commit '03a80925effc2698d21dc0b00290eecf42dd9e68'
authorJames Almer <jamrial@gmail.com>
Sun, 23 Apr 2017 02:07:59 +0000 (23:07 -0300)
committerJames Almer <jamrial@gmail.com>
Sun, 23 Apr 2017 02:27:05 +0000 (23:27 -0300)
* commit '03a80925effc2698d21dc0b00290eecf42dd9e68':
  lavc: add a bitstream filter for splitting VP9 superframes

Merged-by: James Almer <jamrial@gmail.com>
1  2 
doc/bitstream_filters.texi
libavcodec/Makefile
libavcodec/bitstream_filters.c
libavcodec/vp9_superframe_split_bsf.c

@@@ -237,40 -93,10 +237,44 @@@ applies the modification to every byte
  @section null
  This bitstream filter passes the packets through unchanged.
  
 -@section remove_extradata
 +@section remove_extra
 +
 +Remove extradata from packets.
 +
 +It accepts the following parameter:
 +@table @option
 +@item freq
 +Set which frame types to remove extradata from.
 +
 +@table @samp
 +@item k
 +Remove extradata from non-keyframes only.
 +
 +@item keyframe
 +Remove extradata from keyframes only.
 +
 +@item e, all
 +Remove extradata from all frames.
 +
 +@end table
 +@end table
 +
 +@anchor{text2movsub}
 +@section text2movsub
 +
 +Convert text subtitles to MOV subtitles (as used by the @code{mov_text}
 +codec) with metadata headers.
 +
 +See also the @ref{mov2textsub} filter.
 +
 +@section vp9_superframe
 +
 +Merge VP9 invisible (alt-ref) frames back into VP9 superframes. This
 +fixes merging of split/segmented VP9 streams where the alt-ref frame
 +was split from its visible counterpart.
  
+ @section vp9_superframe_split
+ Split VP9 superframes into single frames.
  @c man end BITSTREAM FILTERS
@@@ -998,7 -767,7 +998,8 @@@ OBJS-$(CONFIG_NOISE_BSF
  OBJS-$(CONFIG_NULL_BSF)                   += null_bsf.o
  OBJS-$(CONFIG_REMOVE_EXTRADATA_BSF)       += remove_extradata_bsf.o
  OBJS-$(CONFIG_TEXT2MOVSUB_BSF)            += movsub_bsf.o
 +OBJS-$(CONFIG_VP9_SUPERFRAME_BSF)         += vp9_superframe_bsf.o
+ OBJS-$(CONFIG_VP9_SUPERFRAME_SPLIT_BSF)   += vp9_superframe_split_bsf.o
  
  # thread libraries
  OBJS-$(HAVE_LIBC_MSVCRT)               += file_open.o
@@@ -34,14 -33,12 +34,15 @@@ extern const AVBitStreamFilter ff_hevc_
  extern const AVBitStreamFilter ff_imx_dump_header_bsf;
  extern const AVBitStreamFilter ff_mjpeg2jpeg_bsf;
  extern const AVBitStreamFilter ff_mjpega_dump_header_bsf;
 +extern const AVBitStreamFilter ff_mp3_header_decompress_bsf;
 +extern const AVBitStreamFilter ff_mpeg4_unpack_bframes_bsf;
  extern const AVBitStreamFilter ff_mov2textsub_bsf;
 -extern const AVBitStreamFilter ff_null_bsf;
 -extern const AVBitStreamFilter ff_text2movsub_bsf;
  extern const AVBitStreamFilter ff_noise_bsf;
 +extern const AVBitStreamFilter ff_null_bsf;
  extern const AVBitStreamFilter ff_remove_extradata_bsf;
 +extern const AVBitStreamFilter ff_text2movsub_bsf;
 +extern const AVBitStreamFilter ff_vp9_superframe_bsf;
+ extern const AVBitStreamFilter ff_vp9_superframe_split_bsf;
  
  #include "libavcodec/bsf_list.c"
  
index 0000000,4e635a3..6d6d8e6
mode 000000,100644..100644
--- /dev/null
@@@ -1,0 -1,146 +1,146 @@@
 - * This file is part of Libav.
+ /*
 - * Libav is free software; you can redistribute it and/or
++ * This file is part of FFmpeg.
+  *
 - * Libav is distributed in the hope that it will be useful,
++ * FFmpeg is free software; you can redistribute it and/or
+  * modify it under the terms of the GNU Lesser General Public
+  * License as published by the Free Software Foundation; either
+  * version 2.1 of the License, or (at your option) any later version.
+  *
 - * License along with Libav; if not, write to the Free Software
++ * FFmpeg is distributed in the hope that it will be useful,
+  * but WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+  * Lesser General Public License for more details.
+  *
+  * You should have received a copy of the GNU Lesser General Public
 -#include "bitstream.h"
++ * License along with FFmpeg; if not, write to the Free Software
+  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+  */
+ /**
+  * @file
+  * This bitstream filter splits VP9 superframes into packets containing
+  * just one frame.
+  */
+ #include <stddef.h>
+ #include "avcodec.h"
+ #include "bsf.h"
 -        BitstreamContext bc;
+ #include "bytestream.h"
++#include "get_bits.h"
+ typedef struct VP9SFSplitContext {
+     AVPacket *buffer_pkt;
+     int nb_frames;
+     int next_frame;
+     size_t next_frame_offset;
+     int sizes[8];
+ } VP9SFSplitContext;
+ static int vp9_superframe_split_filter(AVBSFContext *ctx, AVPacket *out)
+ {
+     VP9SFSplitContext *s = ctx->priv_data;
+     AVPacket *in;
+     int i, j, ret, marker;
+     int is_superframe = !!s->buffer_pkt;
+     if (!s->buffer_pkt) {
+         ret = ff_bsf_get_packet(ctx, &s->buffer_pkt);
+         if (ret < 0)
+             return ret;
+         in = s->buffer_pkt;
+         marker = in->data[in->size - 1];
+         if ((marker & 0xe0) == 0xc0) {
+             int length_size = 1 + ((marker >> 3) & 0x3);
+             int   nb_frames = 1 + (marker & 0x7);
+             int    idx_size = 2 + nb_frames * length_size;
+             if (in->size >= idx_size && in->data[in->size - idx_size] == marker) {
+                 GetByteContext bc;
+                 int total_size = 0;
+                 bytestream2_init(&bc, in->data + in->size + 1 - idx_size,
+                                  nb_frames * length_size);
+                 for (i = 0; i < nb_frames; i++) {
+                     int frame_size = 0;
+                     for (j = 0; j < length_size; j++)
+                         frame_size |= bytestream2_get_byte(&bc) << (j * 8);
+                     total_size += frame_size;
+                     if (total_size > in->size - idx_size) {
+                         av_log(ctx, AV_LOG_ERROR,
+                                "Invalid frame size in a superframe: %d\n", frame_size);
+                         ret = AVERROR(EINVAL);
+                         goto fail;
+                     }
+                     s->sizes[i] = frame_size;
+                 }
+                 s->nb_frames         = nb_frames;
+                 s->next_frame        = 0;
+                 s->next_frame_offset = 0;
+                 is_superframe        = 1;
+             }
+         }
+     }
+     if (is_superframe) {
 -        ret = bitstream_init8(&bc, out->data, out->size);
++        GetBitContext gb;
+         int profile, invisible = 0;
+         ret = av_packet_ref(out, s->buffer_pkt);
+         if (ret < 0)
+             goto fail;
+         out->data += s->next_frame_offset;
+         out->size  = s->sizes[s->next_frame];
+         s->next_frame_offset += out->size;
+         s->next_frame++;
+         if (s->next_frame >= s->nb_frames)
+             av_packet_free(&s->buffer_pkt);
 -        bitstream_read(&bc, 2); // frame_marker
 -        profile  = bitstream_read(&bc, 1);
 -        profile |= bitstream_read(&bc, 1) << 1;
++        ret = init_get_bits8(&gb, out->data, out->size);
+         if (ret < 0)
+             goto fail;
 -            bitstream_read(&bc, 1);
 -        if (!bitstream_read(&bc, 1)) {
 -            bitstream_read(&bc, 1);
 -            invisible = !bitstream_read(&bc, 1);
++        get_bits(&gb, 2); // frame_marker
++        profile  = get_bits1(&gb);
++        profile |= get_bits1(&gb) << 1;
+         if (profile == 3)
++            get_bits1(&gb);
++        if (!get_bits1(&gb)) {
++            get_bits1(&gb);
++            invisible = !get_bits1(&gb);
+         }
+         if (invisible)
+             out->pts = AV_NOPTS_VALUE;
+     } else {
+         av_packet_move_ref(out, s->buffer_pkt);
+         av_packet_free(&s->buffer_pkt);
+     }
+     return 0;
+ fail:
+     av_packet_free(&s->buffer_pkt);
+     return ret;
+ }
+ static void vp9_superframe_split_uninit(AVBSFContext *ctx)
+ {
+     VP9SFSplitContext *s = ctx->priv_data;
+     av_packet_free(&s->buffer_pkt);
+ }
+ const AVBitStreamFilter ff_vp9_superframe_split_bsf = {
+     .name = "vp9_superframe_split",
+     .priv_data_size = sizeof(VP9SFSplitContext),
+     .close          = vp9_superframe_split_uninit,
+     .filter         = vp9_superframe_split_filter,
+     .codec_ids      = (const enum AVCodecID []){ AV_CODEC_ID_VP9, AV_CODEC_ID_NONE },
+ };