[FFmpeg-devel] [PATCH v2 0/9] Removing HEVCContext depencencies

[Aaron Levinson]

The entire patch set LGTM.

Aaron Levinson

On 5/3/2017 6:41 PM, James Almer wrote:
> On 5/3/2017 9:39 PM, Aaron Levinson wrote:
>> James,
>>
>> Can you document which patches depend on other patches in this patch set
>> to make it easier to review this patch set in chunks?
>
> Patch 2 depends on 1. Patches 3, 4 and 5 are functionally standalone but
> need previous patches to apply cleanly. Patches 7 and 6 depend on every
> patch before themselves. Patch 8 depends on patches 1 and 2 (and maybe
> others to apply cleanly).
>
>>
>> Thanks,
>> Aaron Levinson
>>
>> On 5/2/2017 2:55 PM, James Almer wrote:
>>> Some changes and fixes suggested on IRC. Some patches are v3 as they have
>>> been sent before outside the previous set.
>>>
>>> James Almer (9):
>>>   hevcdec: remove HEVCContext usage from hevc_sei
>>>   hevcdec: move SEI message parsing into a separate header
>>>   hevcdec: remove HEVCContext usage from ff_hevc_compute_poc()
>>>   hevcdec: move SliceHeader struct definition to hevc_ps
>>>   hevc_parser: use ff_h2645_packet_split() to parse NAL units
>>>   hevc_parser: remove HEVCContext usage
>>>   hevc_parser: move slice header parsing to its own function
>>>   hevc_parse: decode SEI message NALUs in extradata
>>>   doc/libav_merge: remove line about ADVANCED_PARSER
>>>
>>>  doc/libav-merge.txt        |   1 -
>>>  libavcodec/Makefile        |   4 +-
>>>  libavcodec/hevc_parse.c    |  21 ++-
>>>  libavcodec/hevc_parse.h    |   7 +-
>>>  libavcodec/hevc_parser.c   | 415
>>> ++++++++++++++++-----------------------------
>>>  libavcodec/hevc_ps.c       |  23 +++
>>>  libavcodec/hevc_ps.h       |  88 ++++++++++
>>>  libavcodec/hevc_refs.c     |  27 +--
>>>  libavcodec/hevc_sei.c      | 193 +++++++++------------
>>>  libavcodec/hevc_sei.h      | 125 ++++++++++++++
>>>  libavcodec/hevcdec.c       |  94 +++++-----
>>>  libavcodec/hevcdec.h       | 137 +--------------
>>>  libavcodec/mediacodecdec.c |   4 +-
>>>  13 files changed, 532 insertions(+), 607 deletions(-)
>>>  create mode 100644 libavcodec/hevc_sei.h
>>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>