[FFmpeg-devel] [PATCH 1/2] libvpxenc: only force CBR if a bitrate is set

Michael Niedermayer michaelni at gmx.at
Fri Aug 17 21:49:16 CEST 2012


On Fri, Aug 17, 2012 at 11:11:50AM -0700, James Zern wrote:
> On Fri, Aug 17, 2012 at 8:55 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavcodec/libvpxenc.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavcodec/libvpxenc.c b/libavcodec/libvpxenc.c
> > index 0b97907..b438301 100644
> > --- a/libavcodec/libvpxenc.c
> > +++ b/libavcodec/libvpxenc.c
> > @@ -266,7 +266,7 @@ static av_cold int vp8_init(AVCodecContext *avctx)
> >          enccfg.g_pass = VPX_RC_ONE_PASS;
> >
> >      if (avctx->rc_min_rate == avctx->rc_max_rate &&
> > -        avctx->rc_min_rate == avctx->bit_rate)
> > +        avctx->rc_min_rate == avctx->bit_rate && avctx->bit_rate)
> >          enccfg.rc_end_usage = VPX_CBR;
> >      else if (ctx->crf)
> >          enccfg.rc_end_usage = VPX_CQ;
> > --
> > 1.7.9.5
> >
> lgtm

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120817/e1402e77/attachment.asc>


More information about the ffmpeg-devel mailing list