Export to GitHub

openjpeg - issue #461

bio->ct is unnecessarily set to zero in opj_bio_flush method


Posted on Dec 23, 2014 by Happy Wombat

In the opj_bio_flush method, bio->ct = 0 is not necessary, since the opj_bio_byteout call sets ct to either 7 or 8.

See snippets below:

OPJ_BOOL opj_bio_flush(opj_bio_t *bio) { bio->ct = 0; if (! opj_bio_byteout(bio)) { return OPJ_FALSE; } .....

////////////////////////////////////////////

OPJ_BOOL opj_bio_byteout(opj_bio_t *bio) { bio->buf = (bio->buf << 8) & 0xffff; bio->ct = bio->buf == 0xff00 ? 7 : 8; .....

Comment #1

Posted on Dec 23, 2014 by Quick Lion

This issue was closed by revision r2987.

Status: Fixed

Labels:
Type-Defect Priority-Medium