Merge commit '728d90a0c1973661a9e73da697bf4f90c9d19577'
authorClément Bœsch <u@pkh.me>
Sat, 18 Jun 2016 12:04:17 +0000 (14:04 +0200)
committerClément Bœsch <u@pkh.me>
Sat, 18 Jun 2016 12:04:55 +0000 (14:04 +0200)
commit5584f019b5ebb943cd45f848e88f9e59c03f2b41
tree1ada491eab70f85d6df4c594a7d2a262b1735f5d
parent403a53c60e7c6046467ca078ca78210312879e14
parent728d90a0c1973661a9e73da697bf4f90c9d19577
Merge commit '728d90a0c1973661a9e73da697bf4f90c9d19577'

* commit '728d90a0c1973661a9e73da697bf4f90c9d19577':
  h264: decouple h264_sei from the h264 decoder

Main changes:

- SEI decoding doesn't have access to the debug flag in the codec context so a
  few logging are dropped.

- naming of quincunx_sampling_flag and frame_packing_arrangement_type are kept
  as they are in FFmpeg instead of respectively quincunx_subsampling and
  arrangement_type used in Libav because the former match the specifications.

- don't reset the x264 build info once read in order to fix
  fate-h264-lossless (change by Hendrik)

- H264Context.has_recovery_point and deprecated
  AVCodecContext.dtg_active_format are set after ff_h264_sei_decode()
  based on the SEI state since ff_h264_sei_decode() doesn't have access
  to H264Context anymore.

- frame_packing_arrangement_type is not checked against <= 0 in
  decode_postinit() since it is always read as a positive value with
  get_bits(). This fixes a -Wtype-limits warning by GCC spotted by
  Michael.

Side Notes:

- tested that ffprobe on the file from ticket #3652 still returns 4
  keyframes
- tested that playback from ticket #3063 still works

Merged-by: Clément Bœsch <clement@stupeflix.com>
Signed-off-by: Hendrik Leppkes <h.leppkes@gmail.com>
libavcodec/h264.c
libavcodec/h264.h
libavcodec/h264_direct.c
libavcodec/h264_mb.c
libavcodec/h264_parser.c
libavcodec/h264_sei.c
libavcodec/h264_sei.h
libavcodec/h264_slice.c