Re: [FFmpeg-devel] [PATCH] avcodec/hevcdec: fix non-ref frame judgement
On 10/23/2018 1:49 AM, Mark Wu wrote: > After inspecting the source code of x265, mpv and ffmpeg, I've found that > ffmpeg mistakenly regards EVC_NAL_BLA_N_LP and HEVC_NAL_IDR_N_LP as non- > reference frames, which are acutally reference frames according to the > specification in x265, and drops them. > > This patch should address the problem. I have tested it with mpv. > > Signed-off-by: Mark Wu > --- > libavcodec/hevcdec.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h > index f0f588f2b8..dd951aae06 100644 > --- a/libavcodec/hevcdec.h > +++ b/libavcodec/hevcdec.h > @@ -559,8 +559,6 @@ static av_always_inline int ff_hevc_nal_is_nonref(enum > HEVCNALUnitType type) > case HEVC_NAL_VCL_N10: > case HEVC_NAL_VCL_N12: > case HEVC_NAL_VCL_N14: > -case HEVC_NAL_BLA_N_LP: > -case HEVC_NAL_IDR_N_LP: > return 1; > break; > default: break; Pushed, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
[FFmpeg-devel] [PATCH] avcodec/hevcdec: fix non-ref frame judgement
After inspecting the source code of x265, mpv and ffmpeg, I've found that ffmpeg mistakenly regards EVC_NAL_BLA_N_LP and HEVC_NAL_IDR_N_LP as non- reference frames, which are acutally reference frames according to the specification in x265, and drops them. This patch should address the problem. I have tested it with mpv. Signed-off-by: Mark Wu --- libavcodec/hevcdec.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/hevcdec.h b/libavcodec/hevcdec.h index f0f588f2b8..dd951aae06 100644 --- a/libavcodec/hevcdec.h +++ b/libavcodec/hevcdec.h @@ -559,8 +559,6 @@ static av_always_inline int ff_hevc_nal_is_nonref(enum HEVCNALUnitType type) case HEVC_NAL_VCL_N10: case HEVC_NAL_VCL_N12: case HEVC_NAL_VCL_N14: -case HEVC_NAL_BLA_N_LP: -case HEVC_NAL_IDR_N_LP: return 1; break; default: break; -- 2.19.1 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
