[MOVED FROM BAD 106/134] vp8enc: fix target bitrate config with libvpx 1.1.0
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Fri, 18 May 2012 11:46:55 +0000 (12:46 +0100)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Sun, 16 Sep 2012 13:28:18 +0000 (15:28 +0200)
libvpx 1.1.0 disallows a bitrate of 0, which was used by
vp8enc as a default value.

Instead, we use the default libvpx bitrate, scaled to our
video size, if no bitrate was specified.

This fixes encoding VP8 video with libvpx 1.1.0.

https://bugzilla.gnome.org/show_bug.cgi?id=676245

ext/vp8/gstvp8enc.c

index 2d007d1..80f82f6 100644 (file)
@@ -716,6 +716,10 @@ gst_vp8_enc_set_format (GstBaseVideoEncoder * base_video_encoder,
     return FALSE;
   }
 
+  /* Scale default bitrate to our size */
+  cfg.rc_target_bitrate = gst_util_uint64_scale (cfg.rc_target_bitrate,
+      info->width * info->height, cfg.g_w * cfg.g_h);
+
   cfg.g_w = info->width;
   cfg.g_h = info->height;
   cfg.g_timebase.num = info->fps_d;
@@ -739,7 +743,6 @@ gst_vp8_enc_set_format (GstBaseVideoEncoder * base_video_encoder,
   } else {
     cfg.rc_min_quantizer = (gint) (63 - encoder->quality * 6.2);
     cfg.rc_max_quantizer = (gint) (63 - encoder->quality * 6.2);
-    cfg.rc_target_bitrate = encoder->bitrate;
   }
   cfg.rc_dropframe_thresh = encoder->drop_frame;
   cfg.rc_resize_allowed = encoder->resize_allowed;