[FFmpeg-devel] [PATCH] avcodec/libx264: fix forced_idr logic

Michael Niedermayer michael at niedermayer.cc
Fri Nov 11 16:30:10 EET 2016


On Thu, Oct 13, 2016 at 05:32:54PM +0200, Michael Niedermayer wrote:
> On Wed, Oct 12, 2016 at 09:57:16PM +0200, Timo Rothenpieler wrote:
> > Currently, it forces IDR frames for both true and false.
> > Not entirely sure what the original idea behind the tri-state bool
> > option is.
> > ---
> >  libavcodec/libx264.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> there is X264_TYPE_IDR, X264_TYPE_KEYFRAME and X264_TYPE_I
> 
> so there are 3 possibilities, no idea if that was the original idea

ping

if noone adds suport fo the 3rd constant then the patch should
likely be applied, unless i miss something

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161111/6896a3f1/attachment.sig>


More information about the ffmpeg-devel mailing list