vl: Fix trivial sign compare warnings

v2: add whitepace fixes

Signed-off-by: Jan Vesely <jan.vesely@rutgers.edu>
Acked-by: Jose Fonseca <jfonseca@vmware.com>
[Emil Velikov: squash a few more whitespace issues]
Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
This commit is contained in:
Jan Vesely
2016-06-09 23:01:46 -04:00
committed by Emil Velikov
parent 112e988329
commit 1fb4179f92
7 changed files with 15 additions and 18 deletions

View File

@@ -447,7 +447,8 @@ vl_deint_filter_render(struct vl_deint_filter *filter,
struct pipe_sampler_view *sampler_views[4];
struct pipe_surface **dst_surfaces;
const unsigned *plane_order;
int i, j;
int i;
unsigned j;
assert(filter && prevprev && prev && cur && next && field <= 1);

View File

@@ -321,13 +321,11 @@ static void *
create_stage1_frag_shader(struct vl_idct *idct)
{
struct ureg_program *shader;
struct ureg_src l_addr[2], r_addr[2];
struct ureg_dst l[4][2], r[2];
struct ureg_dst *fragment;
int i, j;
unsigned i;
int j;
shader = ureg_create(PIPE_SHADER_FRAGMENT);
if (!shader)

View File

@@ -85,7 +85,7 @@ create_frag_shader(struct vl_matrix_filter *filter, unsigned num_offsets,
struct ureg_dst t_sum;
struct ureg_dst o_fragment;
bool first;
int i;
unsigned i;
shader = ureg_create(PIPE_SHADER_FRAGMENT);
if (!shader) {

View File

@@ -84,7 +84,7 @@ create_frag_shader(struct vl_median_filter *filter,
struct ureg_dst *t_array = MALLOC(sizeof(struct ureg_dst) * num_offsets);
struct ureg_dst o_fragment;
const unsigned median = num_offsets >> 1;
int i, j;
unsigned i, j;
assert(num_offsets & 1); /* we need an odd number of offsets */
if (!(num_offsets & 1)) { /* yeah, we REALLY need an odd number of offsets!!! */
@@ -158,7 +158,8 @@ static void
generate_offsets(enum vl_median_filter_shape shape, unsigned size,
struct vertex2f **offsets, unsigned *num_offsets)
{
int i = 0, half_size;
unsigned i = 0;
int half_size;
struct vertex2f v;
assert(offsets && num_offsets);

View File

@@ -583,12 +583,12 @@ init_dct_coeff_table(struct dct_coeff *dst, const struct dct_coeff_compressed *s
break;
}
for(i=0; i<(1 << (17 - coeff.length)); ++i)
for(i = 0; i < (1u << (17 - coeff.length)); ++i)
dst[src->bitcode << 1 | i] = coeff;
if (has_sign) {
coeff.level = -coeff.level;
for(; i<(1 << (18 - coeff.length)); ++i)
for(; i < (1u << (18 - coeff.length)); ++i)
dst[src->bitcode << 1 | i] = coeff;
}
}

View File

@@ -79,7 +79,7 @@ vl_vlc_init_table(struct vl_vlc_entry *dst, unsigned dst_size, const struct vl_v
}
for(; src_size > 0; --src_size, ++src) {
for(i=0; i<(1 << (bits - src->entry.length)); ++i)
for(i = 0; i < (1u << (bits - src->entry.length)); ++i)
dst[src->bitcode >> (16 - bits) | i] = src->entry;
}
}
@@ -293,7 +293,7 @@ vl_vlc_search_byte(struct vl_vlc *vlc, unsigned num_bits, uint8_t value)
{
/* make sure we are on a byte boundary */
assert((vl_vlc_valid_bits(vlc) % 8) == 0);
assert(num_bits == ~0 || (num_bits % 8) == 0);
assert(num_bits == ~0u || (num_bits % 8) == 0);
/* deplete the bit buffer */
while (vl_vlc_valid_bits(vlc) > 0) {
@@ -305,7 +305,7 @@ vl_vlc_search_byte(struct vl_vlc *vlc, unsigned num_bits, uint8_t value)
vl_vlc_eatbits(vlc, 8);
if (num_bits != ~0) {
if (num_bits != ~0u) {
num_bits -= 8;
if (num_bits == 0)
return FALSE;
@@ -332,7 +332,7 @@ vl_vlc_search_byte(struct vl_vlc *vlc, unsigned num_bits, uint8_t value)
}
++vlc->data;
if (num_bits != ~0) {
if (num_bits != ~0u) {
num_bits -= 8;
if (num_bits == 0) {
vl_vlc_align_data_ptr(vlc);

View File

@@ -99,15 +99,12 @@ static void *
create_vert_shader(struct vl_zscan *zscan)
{
struct ureg_program *shader;
struct ureg_src scale;
struct ureg_src vrect, vpos, block_num;
struct ureg_dst tmp;
struct ureg_dst o_vpos;
struct ureg_dst *o_vtex;
signed i;
unsigned i;
shader = ureg_create(PIPE_SHADER_VERTEX);
if (!shader)