ryan/mozjpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix compiler warnings under Visual C++

Browse Source 
A few of these are long-standing, but most were exposed when switching
from INT32 to JLONG.
This commit is contained in:
DRC
2015-10-15 02:25:00 -05:00
parent d65e768b2e
commit aa769febf2
6 changed files with 28 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
jcdctmgr.c
View File

@@ -185,7 +185,7 @@ compute_reciprocal (UINT16 divisor, DCTELEM * dtbl)
dtbl[DCTSIZE2 * 0] = (DCTELEM) 1; /* reciprocal */ dtbl[DCTSIZE2 * 0] = (DCTELEM) 1; /* reciprocal */
dtbl[DCTSIZE2 * 1] = (DCTELEM) 0; /* correction */ dtbl[DCTSIZE2 * 1] = (DCTELEM) 0; /* correction */
dtbl[DCTSIZE2 * 2] = (DCTELEM) 1; /* scale */ dtbl[DCTSIZE2 * 2] = (DCTELEM) 1; /* scale */
dtbl[DCTSIZE2 * 3] = (DCTELEM) (-sizeof(DCTELEM) * 8); /* shift */ dtbl[DCTSIZE2 * 3] = -(DCTELEM) (sizeof(DCTELEM) * 8); /* shift */
return 0; return 0;
} }
@@ -427,12 +427,12 @@ quantize (JCOEFPTR coef_block, DCTELEM * divisors, DCTELEM * workspace)
temp = -temp; temp = -temp;
product = (UDCTELEM2)(temp + corr) * recip; product = (UDCTELEM2)(temp + corr) * recip;
product >>= shift + sizeof(DCTELEM)*8; product >>= shift + sizeof(DCTELEM)*8;
temp = product; temp = (DCTELEM)product;
temp = -temp; temp = -temp;
} else { } else {
product = (UDCTELEM2)(temp + corr) * recip; product = (UDCTELEM2)(temp + corr) * recip;
product >>= shift + sizeof(DCTELEM)*8; product >>= shift + sizeof(DCTELEM)*8;
temp = product; temp = (DCTELEM)product;
} }
output_ptr[i] = (JCOEF) temp; output_ptr[i] = (JCOEF) temp;
} }

4
jcphuff.c
View File

@@ -35,7 +35,7 @@ typedef struct {
*/ */
JOCTET * next_output_byte; /* => next byte to write in buffer */ JOCTET * next_output_byte; /* => next byte to write in buffer */
size_t free_in_buffer; /* # of byte spaces remaining in buffer */ size_t free_in_buffer; /* # of byte spaces remaining in buffer */
JLONG put_buffer; /* current bit-accumulation buffer */ size_t put_buffer; /* current bit-accumulation buffer */
int put_bits; /* # of bits now in it */ int put_bits; /* # of bits now in it */
j_compress_ptr cinfo; /* link to cinfo (needed for dump_buffer) */ j_compress_ptr cinfo; /* link to cinfo (needed for dump_buffer) */
@@ -241,7 +241,7 @@ emit_bits (phuff_entropy_ptr entropy, unsigned int code, int size)
if (entropy->gather_statistics) if (entropy->gather_statistics)
return; /* do nothing if we're only getting stats */ return; /* do nothing if we're only getting stats */
put_buffer &= (((JLONG) 1)<<size) - 1; /* mask off any extra bits in code */ put_buffer &= (((size_t) 1)<<size) - 1; /* mask off any extra bits in code */
put_bits += size; /* new number of bits in buffer */ put_bits += size; /* new number of bits in buffer */

24
jdcol565.c
View File

@@ -53,7 +53,7 @@ ycc_rgb565_convert_internal (j_decompress_ptr cinfo,
SCALEBITS))]; SCALEBITS))];
b = range_limit[y + Cbbtab[cb]]; b = range_limit[y + Cbbtab[cb]];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
outptr += 2; outptr += 2;
num_cols--; num_cols--;
} }
@@ -88,7 +88,7 @@ ycc_rgb565_convert_internal (j_decompress_ptr cinfo,
SCALEBITS))]; SCALEBITS))];
b = range_limit[y + Cbbtab[cb]]; b = range_limit[y + Cbbtab[cb]];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
} }
@@ -134,7 +134,7 @@ ycc_rgb565D_convert_internal (j_decompress_ptr cinfo,
SCALEBITS)), d0)]; SCALEBITS)), d0)];
b = range_limit[DITHER_565_B(y + Cbbtab[cb], d0)]; b = range_limit[DITHER_565_B(y + Cbbtab[cb], d0)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
outptr += 2; outptr += 2;
num_cols--; num_cols--;
} }
@@ -174,7 +174,7 @@ ycc_rgb565D_convert_internal (j_decompress_ptr cinfo,
SCALEBITS)), d0)]; SCALEBITS)), d0)];
b = range_limit[DITHER_565_B(y + Cbbtab[cb], d0)]; b = range_limit[DITHER_565_B(y + Cbbtab[cb], d0)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
} }
@@ -206,7 +206,7 @@ rgb_rgb565_convert_internal (j_decompress_ptr cinfo,
g = GETJSAMPLE(*inptr1++); g = GETJSAMPLE(*inptr1++);
b = GETJSAMPLE(*inptr2++); b = GETJSAMPLE(*inptr2++);
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
outptr += 2; outptr += 2;
num_cols--; num_cols--;
} }
@@ -229,7 +229,7 @@ rgb_rgb565_convert_internal (j_decompress_ptr cinfo,
g = GETJSAMPLE(*inptr1); g = GETJSAMPLE(*inptr1);
b = GETJSAMPLE(*inptr2); b = GETJSAMPLE(*inptr2);
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
} }
@@ -263,7 +263,7 @@ rgb_rgb565D_convert_internal (j_decompress_ptr cinfo,
g = range_limit[DITHER_565_G(GETJSAMPLE(*inptr1++), d0)]; g = range_limit[DITHER_565_G(GETJSAMPLE(*inptr1++), d0)];
b = range_limit[DITHER_565_B(GETJSAMPLE(*inptr2++), d0)]; b = range_limit[DITHER_565_B(GETJSAMPLE(*inptr2++), d0)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
outptr += 2; outptr += 2;
num_cols--; num_cols--;
} }
@@ -288,7 +288,7 @@ rgb_rgb565D_convert_internal (j_decompress_ptr cinfo,
g = range_limit[DITHER_565_G(GETJSAMPLE(*inptr1), d0)]; g = range_limit[DITHER_565_G(GETJSAMPLE(*inptr1), d0)];
b = range_limit[DITHER_565_B(GETJSAMPLE(*inptr2), d0)]; b = range_limit[DITHER_565_B(GETJSAMPLE(*inptr2), d0)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
} }
@@ -313,7 +313,7 @@ gray_rgb565_convert_internal (j_decompress_ptr cinfo,
if (PACK_NEED_ALIGNMENT(outptr)) { if (PACK_NEED_ALIGNMENT(outptr)) {
g = *inptr++; g = *inptr++;
rgb = PACK_SHORT_565(g, g, g); rgb = PACK_SHORT_565(g, g, g);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
outptr += 2; outptr += 2;
num_cols--; num_cols--;
} }
@@ -328,7 +328,7 @@ gray_rgb565_convert_internal (j_decompress_ptr cinfo,
if (num_cols & 1) { if (num_cols & 1) {
g = *inptr; g = *inptr;
rgb = PACK_SHORT_565(g, g, g); rgb = PACK_SHORT_565(g, g, g);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
} }
@@ -356,7 +356,7 @@ gray_rgb565D_convert_internal (j_decompress_ptr cinfo,
g = *inptr++; g = *inptr++;
g = range_limit[DITHER_565_R(g, d0)]; g = range_limit[DITHER_565_R(g, d0)];
rgb = PACK_SHORT_565(g, g, g); rgb = PACK_SHORT_565(g, g, g);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
outptr += 2; outptr += 2;
num_cols--; num_cols--;
} }
@@ -378,7 +378,7 @@ gray_rgb565D_convert_internal (j_decompress_ptr cinfo,
g = *inptr; g = *inptr;
g = range_limit[DITHER_565_R(g, d0)]; g = range_limit[DITHER_565_R(g, d0)];
rgb = PACK_SHORT_565(g, g, g); rgb = PACK_SHORT_565(g, g, g);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
} }

8
jdmerge.c
View File

@@ -436,12 +436,12 @@ h2v2_merged_upsample (j_decompress_ptr cinfo,
#define PACK_NEED_ALIGNMENT(ptr) (((size_t)(ptr)) & 3) #define PACK_NEED_ALIGNMENT(ptr) (((size_t)(ptr)) & 3)
#define WRITE_TWO_PIXELS_LE(addr, pixels) { \ #define WRITE_TWO_PIXELS_LE(addr, pixels) { \
((INT16*)(addr))[0] = (pixels); \ ((INT16*)(addr))[0] = (INT16)(pixels); \
((INT16*)(addr))[1] = (pixels) >> 16; \ ((INT16*)(addr))[1] = (INT16)((pixels) >> 16); \
} }
#define WRITE_TWO_PIXELS_BE(addr, pixels) { \ #define WRITE_TWO_PIXELS_BE(addr, pixels) { \
((INT16*)(addr))[1] = (pixels); \ ((INT16*)(addr))[1] = (INT16)(pixels); \
((INT16*)(addr))[0] = (pixels) >> 16; \ ((INT16*)(addr))[0] = (INT16)((pixels) >> 16); \
} }
#define DITHER_565_R(r, dither) ((r) + ((dither) & 0xFF)) #define DITHER_565_R(r, dither) ((r) + ((dither) & 0xFF))

12
jdmrg565.c
View File

@@ -79,7 +79,7 @@ h2v1_merged_upsample_565_internal (j_decompress_ptr cinfo,
g = range_limit[y + cgreen]; g = range_limit[y + cgreen];
b = range_limit[y + cblue]; b = range_limit[y + cblue];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
@@ -153,7 +153,7 @@ h2v1_merged_upsample_565D_internal (j_decompress_ptr cinfo,
g = range_limit[DITHER_565_G(y + cgreen, d0)]; g = range_limit[DITHER_565_G(y + cgreen, d0)];
b = range_limit[DITHER_565_B(y + cblue, d0)]; b = range_limit[DITHER_565_B(y + cblue, d0)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr = rgb; *(INT16*)outptr = (INT16)rgb;
} }
} }
@@ -242,14 +242,14 @@ h2v2_merged_upsample_565_internal (j_decompress_ptr cinfo,
g = range_limit[y + cgreen]; g = range_limit[y + cgreen];
b = range_limit[y + cblue]; b = range_limit[y + cblue];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr0 = rgb; *(INT16*)outptr0 = (INT16)rgb;
y = GETJSAMPLE(*inptr01); y = GETJSAMPLE(*inptr01);
r = range_limit[y + cred]; r = range_limit[y + cred];
g = range_limit[y + cgreen]; g = range_limit[y + cgreen];
b = range_limit[y + cblue]; b = range_limit[y + cblue];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr1 = rgb; *(INT16*)outptr1 = (INT16)rgb;
} }
} }
@@ -344,13 +344,13 @@ h2v2_merged_upsample_565D_internal (j_decompress_ptr cinfo,
g = range_limit[DITHER_565_G(y + cgreen, d0)]; g = range_limit[DITHER_565_G(y + cgreen, d0)];
b = range_limit[DITHER_565_B(y + cblue, d0)]; b = range_limit[DITHER_565_B(y + cblue, d0)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr0 = rgb; *(INT16*)outptr0 = (INT16)rgb;
y = GETJSAMPLE(*inptr01); y = GETJSAMPLE(*inptr01);
r = range_limit[DITHER_565_R(y + cred, d1)]; r = range_limit[DITHER_565_R(y + cred, d1)];
g = range_limit[DITHER_565_G(y + cgreen, d1)]; g = range_limit[DITHER_565_G(y + cgreen, d1)];
b = range_limit[DITHER_565_B(y + cblue, d1)]; b = range_limit[DITHER_565_B(y + cblue, d1)];
rgb = PACK_SHORT_565(r, g, b); rgb = PACK_SHORT_565(r, g, b);
*(INT16*)outptr1 = rgb; *(INT16*)outptr1 = (INT16)rgb;
} }
} }

3
rdppm.c
View File

@@ -99,8 +99,7 @@ read_pbm_integer (j_compress_ptr cinfo, FILE * infile, int maxval)
/* Note that on a 16-bit-int machine, only values up to 64k can be read. */ /* Note that on a 16-bit-int machine, only values up to 64k can be read. */
/* This should not be a problem in practice. */ /* This should not be a problem in practice. */
{ {
register int ch; register int ch, val;
register unsigned int val;
/* Skip any leading whitespace */ /* Skip any leading whitespace */
do { do {