summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorMickael Savinaud <savmickael@users.noreply.github.com>2012-10-24 09:09:37 +0000
committerMickael Savinaud <savmickael@users.noreply.github.com>2012-10-24 09:09:37 +0000
commit12c4e680580f5a246a7a5b3993cfadbbffbd0e9d (patch)
tree34c19c99a1f1d83d45db336efc3b8ace7d5bdd68 /src
parentb4b451863d47d1830f83edff4d907c26a3c688d7 (diff)
[trunk] finalize bio.c/.h related to the v2 style
Diffstat (limited to 'src')
-rw-r--r--src/lib/openjp2/bio.c6
-rw-r--r--src/lib/openjp2/bio.h2
2 files changed, 4 insertions, 4 deletions
diff --git a/src/lib/openjp2/bio.c b/src/lib/openjp2/bio.c
index c3f92314..f4694069 100644
--- a/src/lib/openjp2/bio.c
+++ b/src/lib/openjp2/bio.c
@@ -78,7 +78,7 @@ opj_bool opj_bio_byteout(opj_bio_t *bio) {
if (bio->bp >= bio->end) {
return OPJ_FALSE;
}
- *bio->bp++ = (unsigned char)(bio->buf >> 8); /* TODO MSD: check this conversion */
+ *bio->bp++ = (OPJ_BYTE)(bio->buf >> 8);
return OPJ_TRUE;
}
@@ -94,7 +94,7 @@ opj_bool opj_bio_bytein(opj_bio_t *bio) {
void opj_bio_putbit(opj_bio_t *bio, OPJ_UINT32 b) {
if (bio->ct == 0) {
- opj_bio_byteout(bio); /* TODO_MSD: check this line */
+ opj_bio_byteout(bio); /* MSD: why not check the return value of this function ? */
}
bio->ct--;
bio->buf |= b << bio->ct;
@@ -102,7 +102,7 @@ void opj_bio_putbit(opj_bio_t *bio, OPJ_UINT32 b) {
OPJ_UINT32 opj_bio_getbit(opj_bio_t *bio) {
if (bio->ct == 0) {
- opj_bio_bytein(bio); /* TODO_MSD: check this line */
+ opj_bio_bytein(bio); /* MSD: why not check the return value of this function ? */
}
bio->ct--;
return (bio->buf >> bio->ct) & 1;
diff --git a/src/lib/openjp2/bio.h b/src/lib/openjp2/bio.h
index 53e0f0db..a69299b9 100644
--- a/src/lib/openjp2/bio.h
+++ b/src/lib/openjp2/bio.h
@@ -57,7 +57,7 @@ typedef struct opj_bio {
/** temporary place where each byte is read or written */
OPJ_UINT32 buf;
/** coder : number of bits free to write. decoder : number of bits read */
- int ct;
+ OPJ_UINT32 ct;
} opj_bio_t;
/** @name Exported functions */