]> granicus.if.org Git - libx264/commitdiff
cosmetics
authorLoren Merritt <pengvado@videolan.org>
Thu, 10 Feb 2005 22:11:39 +0000 (22:11 +0000)
committerLoren Merritt <pengvado@videolan.org>
Thu, 10 Feb 2005 22:11:39 +0000 (22:11 +0000)
git-svn-id: svn://svn.videolan.org/x264/trunk@120 df754926-b1dd-0310-bc7b-ec298dee348c

encoder/analyse.c
encoder/encoder.c
encoder/me.c

index 2e5609cde332a6e0db528f3d95761a5e789a077e..7f4cd7c0988713f6ddb62209801b41db38d2e11a 100644 (file)
@@ -1177,11 +1177,9 @@ void x264_macroblock_analyse( x264_t *h )
 
     h->mb.qp[h->mb.i_mb_xy] = x264_ratecontrol_qp(h);
 
-    /* FIXME check if it's 12 */
-    if( h->mb.qp[h->mb.i_mb_xy] - h->mb.i_last_qp < -12 )
-        h->mb.qp[h->mb.i_mb_xy] = h->mb.i_last_qp - 12;
-    else if( h->mb.qp[h->mb.i_mb_xy] - h->mb.i_last_qp > 12 )
-        h->mb.qp[h->mb.i_mb_xy] = h->mb.i_last_qp + 12;
+    /* prevent QP from varying too fast. FIXME what's a sane limit? */
+    h->mb.qp[h->mb.i_mb_xy] = x264_clip3( h->mb.qp[h->mb.i_mb_xy],
+                                          h->mb.i_last_qp - 12, h->mb.i_last_qp + 12 );
 
     /* init analysis */
     x264_mb_analyse_init( h, &analysis, h->mb.qp[h->mb.i_mb_xy] );
index 8e4a2c73da32b622849917947485d74df49a8bad..3f48518572054e32d145622cf9932d930bba6669 100644 (file)
@@ -63,7 +63,7 @@ static int64_t i_mtime_filter = 0;
  ******************************* x264 libs **********************************
  *
  ****************************************************************************/
-static float x264_sqe( uint8_t *pix1, int i_pix_stride, uint8_t *pix2, int i_pix2_stride, int i_width, int i_height )
+static int64_t x264_sqe( uint8_t *pix1, int i_pix_stride, uint8_t *pix2, int i_pix2_stride, int i_width, int i_height )
 {
     int64_t i_sqe = 0;
 
index 1b9a448980a56b2cb803bbd786162b4463750136..660b5de2c1cea54e6d8b40183b1a3682b99e1a67 100644 (file)
@@ -33,7 +33,7 @@
  * subpel_iters[i_subpel_refine] = { refine_hpel, refine_qpel, me_hpel, me_qpel }
  * where me_* are the number of EPZS iterations run on all candidate block types,
  * and refine_* are run only on the winner. */
-const static int subpel_iterations[][4] = 
+static const int subpel_iterations[][4] = 
    {{1,0,0,0},
     {1,1,0,0},
     {1,2,0,0},