Yes, I probably messed up. Merging code from very different 2 year old
code is not exactly easy. I don't know how you people do this...

On Tue, Apr 11, 2017 at 3:33 PM, Hendrik Leppkes <h.lepp...@gmail.com> wrote:
> On Tue, Apr 11, 2017 at 3:29 PM, Anton Khirnov <an...@khirnov.net> wrote:
>> Quoting Steve Lhomme (2017-04-07 14:27:39)
>>> From: Hendrik Leppkes <h.lepp...@gmail.com>
>>>
>>> Fixes ticket #4185.
>>>
>>> Reviewed-By: Mickael Raulet <mickael.rau...@insa-rennes.fr>
>>> Signed-off-by: Hendrik Leppkes <h.lepp...@gmail.com>
>>> ---
>>>  libavcodec/hevcdec.c | 3 +++
>>>  libavcodec/hevcdec.h | 1 +
>>>  2 files changed, 4 insertions(+)
>>>
>>> diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
>>> index 6a04858587..6fe05d1b01 100644
>>> --- a/libavcodec/hevcdec.c
>>> +++ b/libavcodec/hevcdec.c
>>> @@ -2421,6 +2421,8 @@ static int hevc_frame_start(HEVCContext *s)
>>>      s->is_decoded        = 0;
>>>      s->first_nal_type    = s->nal_unit_type;
>>>
>>> +    s->no_rasl_output_flag = IS_IDR(s) || IS_BLA(s);
>>> +
>>>      if (s->ps.pps->tiles_enabled_flag)
>>>          lc->end_of_tiles_x = s->ps.pps->column_width[0] << 
>>> s->ps.sps->log2_ctb_size;
>>>
>>> @@ -3007,6 +3009,7 @@ static int hevc_update_thread_context(AVCodecContext 
>>> *dst,
>>>      s->seq_output = s0->seq_output;
>>>      s->pocTid0    = s0->pocTid0;
>>>      s->max_ra     = s0->max_ra;
>>> +    s->no_rasl_output_flag = s0->no_rasl_output_flag;
>>>
>>>      s->is_nalff        = s0->is_nalff;
>>>      s->nal_length_size = s0->nal_length_size;
>>> diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h
>>> index ff192f67ae..64089bde75 100644
>>> --- a/libavcodec/hevcdec.h
>>> +++ b/libavcodec/hevcdec.h
>>> @@ -484,6 +484,7 @@ typedef struct HEVCContext {
>>>      int bs_height;
>>>
>>>      int is_decoded;
>>> +    int no_rasl_output_flag;
>>>
>>>      HEVCPredContext hpc;
>>>      HEVCDSPContext hevcdsp;
>>> --
>>> 2.11.1
>>
>> Eh? This seems to be write-only.
>>
>
> For some reason he skipped one of the hunks (and also modified the
> actual setting of the flag?)
> http://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=0118158efa8e45761f9f65a3bb74f33907bd2aec
>
> Also, a follow up for tsan happyness:
> http://git.videolan.org/?p=ffmpeg.git;a=commitdiff;h=bddabfaab65808e40605181d579ffcd85bfe4c26
>
> - Hendrik
> _______________________________________________
> libav-devel mailing list
> libav-devel@libav.org
> https://lists.libav.org/mailman/listinfo/libav-devel
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to