summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authorEven Rouault <even.rouault@spatialys.com>2017-05-23 13:54:28 +0200
committerEven Rouault <even.rouault@spatialys.com>2017-05-23 13:54:28 +0200
commit8728cfbc799a61dd0600ee04aa64d829937d8710 (patch)
tree28bb0def3a7bdc0a7e2f01d7ec82175d8927e1e8 /src/lib
parent83d7a6d4a44833e78321bfd367dc0f822a8d0945 (diff)
t1.c: fix compiler warnings
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/openjp2/t1.c23
1 files changed, 13 insertions, 10 deletions
diff --git a/src/lib/openjp2/t1.c b/src/lib/openjp2/t1.c
index 0cc67723..af520202 100644
--- a/src/lib/openjp2/t1.c
+++ b/src/lib/openjp2/t1.c
@@ -358,7 +358,7 @@ static INLINE void opj_t1_updateflags(opj_flag_t *flagsp, OPJ_UINT32 s,
flagsp[1] |= (opj_flag_t)(flag_N << 3U);
sp[-1] |= T1_SIG_NE;
- sp[0] |= (opj_flag_t)flag_N;
+ sp[0] = (opj_flag_t)(sp[0] | flag_N);
sp[1] |= T1_SIG_NW;
}
@@ -472,7 +472,8 @@ static INLINE void opj_t1_dec_sigpass_step_raw(
#ifdef CONSISTENCY_CHECK
*flagsp |= T1_VISIT;
#endif
- *colflagsp |= (opj_colflag_t)(T1_COLFLAG_VISIT_ROW_0 << (T1_COLFLAG_RBS * row));
+ *colflagsp = (opj_colflag_t)(*colflagsp | (T1_COLFLAG_VISIT_ROW_0 <<
+ (T1_COLFLAG_RBS * row)));
}
}
@@ -508,7 +509,8 @@ static INLINE void opj_t1_dec_sigpass_step_mqc(
#ifdef CONSISTENCY_CHECK
*flagsp |= T1_VISIT;
#endif
- *colflagsp |= (opj_colflag_t)(T1_COLFLAG_VISIT_ROW_0 << (T1_COLFLAG_RBS * row));
+ *colflagsp = (opj_colflag_t)(*colflagsp | (T1_COLFLAG_VISIT_ROW_0 <<
+ (T1_COLFLAG_RBS * row)));
}
} /* VSC and BYPASS by Antonin */
@@ -540,7 +542,8 @@ static INLINE void opj_t1_dec_sigpass_step_mqc_vsc(
#ifdef CONSISTENCY_CHECK
*flagsp |= T1_VISIT;
#endif
- *colflagsp |= (opj_colflag_t)(T1_COLFLAG_VISIT_ROW_0 << (T1_COLFLAG_RBS * row));
+ *colflagsp = (opj_colflag_t)(*colflagsp | (T1_COLFLAG_VISIT_ROW_0 <<
+ (T1_COLFLAG_RBS * row)));
}
} /* VSC and BYPASS by Antonin */
@@ -772,8 +775,8 @@ static INLINE void opj_t1_dec_refpass_step_raw(
v = (OPJ_INT32)opj_raw_decode(raw);
t = v ? poshalf : neghalf;
*datap += *datap < 0 ? -t : t;
- *colflagsp |= (opj_colflag_t)(T1_COLFLAG_REFINE_ROW_0 <<
- (T1_COLFLAG_RBS * row));
+ *colflagsp = (opj_colflag_t)(*colflagsp | (T1_COLFLAG_REFINE_ROW_0 <<
+ (T1_COLFLAG_RBS * row)));
}
} /* VSC and BYPASS by Antonin */
@@ -812,8 +815,8 @@ static INLINE void opj_t1_dec_refpass_step_mqc(
v = opj_mqc_decode(mqc);
t = v ? poshalf : neghalf;
*datap += *datap < 0 ? -t : t;
- *colflagsp |= (opj_colflag_t)(T1_COLFLAG_REFINE_ROW_0 <<
- (T1_COLFLAG_RBS * row));
+ *colflagsp = (opj_colflag_t)(*colflagsp | (T1_COLFLAG_REFINE_ROW_0 <<
+ (T1_COLFLAG_RBS * row)));
}
} /* VSC and BYPASS by Antonin */
@@ -844,8 +847,8 @@ static INLINE void opj_t1_dec_refpass_step_mqc_vsc(
v = (OPJ_UINT32)opj_mqc_decode(mqc);
t = v ? poshalf : neghalf;
*datap += *datap < 0 ? -t : t;
- *colflagsp |= (opj_colflag_t)(T1_COLFLAG_REFINE_ROW_0 <<
- (T1_COLFLAG_RBS * row));
+ *colflagsp = (opj_colflag_t)(*colflagsp | (T1_COLFLAG_REFINE_ROW_0 <<
+ (T1_COLFLAG_RBS * row)));
}
} /* VSC and BYPASS by Antonin */