[trunk] Fix issue with ITU-T T.809 (05/2011)

- ZSI is now known as NSI (FF54)
- NSI now stores Ndim (=3)
- CAP extended marker was not set properly
This commit is contained in:
Mathieu Malaterre 2012-11-28 11:46:16 +00:00
parent 650c587875
commit be3a1da14a
3 changed files with 26 additions and 12 deletions

View File

@ -518,7 +518,8 @@ int main(int argc, char **argv) {
fprintf(stdout, "[RESULT] Volume: %d x %d x %d (x %d bpv)\n ",
(volume->comps[0].w >> volume->comps[0].factor[0]),
(volume->comps[0].h >> volume->comps[0].factor[1]),
(volume->comps[0].l >> volume->comps[0].factor[2]),volume->comps[0].prec);
(volume->comps[0].l >> volume->comps[0].factor[2]),
volume->comps[0].prec);
if(original){
psnr = calc_PSNR(original,volume);

View File

@ -58,15 +58,15 @@ Read the SIZ marker (2D volume and tile size)
*/
static void j3d_read_siz(opj_j3d_t *j3d);
/**
Write the ZSI marker (3rd volume and tile size)
Write the NSI marker (3rd volume and tile size)
@param j3d J3D handle
*/
static void j3d_write_zsi(opj_j3d_t *j3d);
static void j3d_write_nsi(opj_j3d_t *j3d);
/**
Read the ZSI marker (3rd volume and tile size)
Read the NSI marker (3rd volume and tile size)
@param j3d J3D handle
*/
static void j3d_read_zsi(opj_j3d_t *j3d);
static void j3d_read_nsi(opj_j3d_t *j3d);
/**
Write the COM marker (comment)
@param j3d J3D handle
@ -418,6 +418,10 @@ static void j3d_write_cap(opj_j3d_t *j3d){
lenp = cio_tell(cio);
cio_skip(cio, 2);
cio_write(cio,J3D_CAP_10, 4);
if( J3D_CAP_10 )
{
cio_write(cio, 0x0, 2);
}
len = cio_tell(cio) - lenp;
cio_seek(cio, lenp);
cio_write(cio, len, 2); /* Lsiz */
@ -430,18 +434,24 @@ static void j3d_read_cap(opj_j3d_t *j3d){
/*cio_read(cio, 2); CAP */
len = cio_read(cio, 2);
Cap = cio_read(cio, 4);
if(Cap) {
cio_read(cio, 2);
}
static void j3d_write_zsi(opj_j3d_t *j3d) {
assert( len == 2 + 4 + 2 );
}
static void j3d_write_nsi(opj_j3d_t *j3d) {
int i;
int lenp, len;
int ndim = 3;
opj_cio_t *cio = j3d->cio;
opj_volume_t *volume = j3d->volume;
opj_cp_t *cp = j3d->cp;
cio_write(cio, J3D_MS_ZSI, 2); /* ZSI */
cio_write(cio, J3D_MS_NSI, 2); /* NSI */
lenp = cio_tell(cio);
cio_skip(cio, 2);
cio_write(cio, ndim, 1); /* Ndim */
cio_write(cio, volume->z1, 4); /* Zsiz */
cio_write(cio, volume->z0, 4); /* Z0siz */
cio_write(cio, cp->tdz, 4); /* ZTsiz */
@ -455,14 +465,17 @@ static void j3d_write_zsi(opj_j3d_t *j3d) {
cio_seek(cio, lenp + len);
}
static void j3d_read_zsi(opj_j3d_t *j3d) {
static void j3d_read_nsi(opj_j3d_t *j3d) {
int ndim;
int len, i;
opj_cio_t *cio = j3d->cio;
opj_volume_t *volume = j3d->volume;
opj_cp_t *cp = j3d->cp;
len = cio_read(cio, 2); /* Lsiz */
len = cio_read(cio, 2); /* Lnsi */
ndim = cio_read(cio, 1); /* Ndim */
assert( ndim == 3 );
volume->z1 = cio_read(cio, 4); /* Zsiz */
volume->z0 = cio_read(cio, 4); /* Z0siz */
cp->tdz = cio_read(cio, 4); /* ZTsiz */
@ -1545,7 +1558,7 @@ opj_dec_mstabent_t j3d_dec_mstab[] = {
{J3D_MS_EOC, J3D_STATE_TPHSOT, j3d_read_eoc},
{J3D_MS_CAP, J3D_STATE_MHSIZ, j3d_read_cap},
{J3D_MS_SIZ, J3D_STATE_MHSIZ, j3d_read_siz},
{J3D_MS_ZSI, J3D_STATE_MHSIZ, j3d_read_zsi},
{J3D_MS_NSI, J3D_STATE_MHSIZ, j3d_read_nsi},
{J3D_MS_COD, J3D_STATE_MH | J3D_STATE_TPH, j3d_read_cod},
{J3D_MS_COC, J3D_STATE_MH | J3D_STATE_TPH, j3d_read_coc},
{J3D_MS_RGN, J3D_STATE_MH | J3D_STATE_TPH, j3d_read_rgn},
@ -2243,7 +2256,7 @@ bool j3d_encode(opj_j3d_t *j3d, opj_cio_t *cio, opj_volume_t *volume, char *inde
j3d_write_siz(j3d);
if (j3d->cinfo->codec_format == CODEC_J3D) {
j3d_write_cap(j3d);
j3d_write_zsi(j3d);
j3d_write_nsi(j3d);
}
j3d_write_cod(j3d);
j3d_write_qcd(j3d);

View File

@ -65,7 +65,7 @@ The functions in J3D.C have for goal to read/write the several parts of the code
#define J3D_MS_EOC 0xffd9 /**< EOC marker value */
#define J3D_MS_CAP 0xff50 /**< CAP marker value */
#define J3D_MS_SIZ 0xff51 /**< SIZ marker value */
#define J3D_MS_ZSI 0xff54 /**< ZSI marker value */
#define J3D_MS_NSI 0xff54 /**< NSI marker value */
#define J3D_MS_COD 0xff52 /**< COD marker value */
#define J3D_MS_COC 0xff53 /**< COC marker value */
#define J3D_MS_RGN 0xff5e /**< RGN marker value */