From: Ralph Giles <giles@mozilla.com>
Date: Thu, 5 Jan 2012 16:39:38 +0000 (-0600)
Subject: Reduce the default kf_max_dist to 128.
X-Git-Tag: v1.0.0~22^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2a0d7b1;p=libvpx

Reduce the default kf_max_dist to 128.

The default maximum keyframe interval is 9999, or over five minutes
at normal video rates. When the encoder produces streams with such
a long interval seeking (with correct output) is more expensive,
and live streaming is impossible.

Of course the encoding application should set this parameter
based on its knowledge of the intended use of the stream, but
reducing the default gives better results for applications
which do not.

Change-Id: I900b15d74ce72ecc3ade4d43f758c5cf97a2098a
---

diff --git a/vp8/vp8_cx_iface.c b/vp8/vp8_cx_iface.c
index 6181ee8ee..abaf85b40 100644
--- a/vp8/vp8_cx_iface.c
+++ b/vp8/vp8_cx_iface.c
@@ -1237,7 +1237,7 @@ static vpx_codec_enc_cfg_map_t vp8e_usage_cfg_map[] =
         /* keyframing settings (kf) */
         VPX_KF_AUTO,        /* g_kfmode*/
         0,                  /* kf_min_dist */
-        9999,               /* kf_max_dist */
+        128,                /* kf_max_dist */
 
 #if VPX_ENCODER_ABI_VERSION == (1 + VPX_CODEC_ABI_VERSION)
         1,                  /* g_delete_first_pass_file */