Minor fixes
authorJason Garrett-Glaser <jason@x264.com>
Sun, 3 Apr 2011 23:31:52 +0000 (16:31 -0700)
committerJason Garrett-Glaser <jason@x264.com>
Tue, 12 Apr 2011 08:37:24 +0000 (01:37 -0700)
Fix a comment typo.
Align an array properly.
Make x264_scan8 unsigned: saves a bunch of movsxd instructions on x86_64.

common/common.h
common/set.c
encoder/me.c

index f2021b6..fcf0250 100644 (file)
@@ -137,7 +137,7 @@ typedef union { x264_uint128_t i; uint64_t a[2]; uint32_t b[4]; uint16_t c[8]; u
 #define X264_SCAN8_LUMA_SIZE (5*8)
 #define X264_SCAN8_0 (4+1*8)
 
-static const int x264_scan8[16+2*4+3] =
+static const unsigned x264_scan8[16+2*4+3] =
 {
     /* Luma */
     4+1*8, 5+1*8, 4+2*8, 5+2*8,
index f8f6a11..f967fd8 100644 (file)
@@ -195,7 +195,7 @@ int x264_cqm_init( x264_t *h )
             int dct8x8 = cat == 1;
             int size = dct8x8 ? 64 : 16;
             udctcoef *nr_offset = h->nr_offset_emergency[q][cat];
-            /* Denoise chroma first (due to h264's chroma QP offset, then luma, then DC. */
+            /* Denoise chroma first (due to h264's chroma QP offset), then luma, then DC. */
             int dc_threshold =    (QP_MAX-QP_MAX_SPEC)*2/3;
             int luma_threshold =  (QP_MAX-QP_MAX_SPEC)*2/3;
             int chroma_threshold = 0;
index d5544f3..54de887 100644 (file)
@@ -965,7 +965,7 @@ static void ALWAYS_INLINE x264_me_refine_bidir( x264_t *h, x264_me_t *m0, x264_m
     /* each byte of visited represents 8 possible m1y positions, so a 4D array isn't needed */
     ALIGNED_ARRAY_16( uint8_t, visited,[8],[8][8] );
     /* all permutations of an offset in up to 2 of the dimensions */
-    static const int8_t dia4d[33][4] =
+    ALIGNED_4( static const int8_t dia4d[33][4] ) =
     {
         {0,0,0,0},
         {0,0,0,1}, {0,0,0,-1}, {0,0,1,0}, {0,0,-1,0},