cppcheck fix for openjp2 (#740)
[openjpeg.git] / tests / test_tile_encoder.c
index a405d86e9dd81c22db3e081b87cda45943366f0e..d01a7e5245b4da50dd3cd2b1e9ed1b1d1c5121cd 100644 (file)
 
 /* -------------------------------------------------------------------------- */
 
-/**
-sample error callback expecting a FILE* client object
-*/
-static void error_callback_file(const char *msg, void *client_data) {
-       FILE *stream = (FILE*)client_data;
-       fprintf(stream, "[ERROR] %s", msg);
-}
-/**
-sample warning callback expecting a FILE* client object
-*/
-static void warning_callback_file(const char *msg, void *client_data) {
-       FILE *stream = (FILE*)client_data;
-       fprintf(stream, "[WARNING] %s", msg);
-}
 /**
 sample error debug callback expecting no client object
 */
@@ -80,7 +66,6 @@ int main (int argc, char *argv[])
        opj_codec_t * l_codec;
        opj_image_t * l_image;
        opj_image_cmptparm_t l_params [NUM_COMPS_MAX];
-       FILE * l_file;
        opj_stream_t * l_stream;
        OPJ_UINT32 l_nb_tiles;
        OPJ_UINT32 l_data_size;
@@ -116,7 +101,7 @@ int main (int argc, char *argv[])
   /* should be test_tile_encoder 3 2000 2000 1000 1000 8 tte1.j2k */
   if( argc == 9 )
     {
-    num_comps = atoi( argv[1] );
+    num_comps = (OPJ_UINT32)atoi( argv[1] );
     image_width = atoi( argv[2] );
     image_height = atoi( argv[3] );
     tile_width = atoi( argv[4] );
@@ -140,14 +125,16 @@ int main (int argc, char *argv[])
     {
     return 1;
     }
-       l_nb_tiles = (image_width/tile_width) * (image_height/tile_height);
-       l_data_size = tile_width * tile_height * num_comps * (comp_prec/8);
-
-       l_data = (OPJ_BYTE*) malloc(tile_width * tile_height * num_comps * (comp_prec/8) * sizeof(OPJ_BYTE));
+       l_nb_tiles = (OPJ_UINT32)(image_width/tile_width) * (OPJ_UINT32)(image_height/tile_height);
+       l_data_size = (OPJ_UINT32)tile_width * (OPJ_UINT32)tile_height * (OPJ_UINT32)num_comps * (OPJ_UINT32)(comp_prec/8);
 
+       l_data = (OPJ_BYTE*) malloc(l_data_size * sizeof(OPJ_BYTE));
+       if(l_data == NULL){
+               return 1;
+       }
        fprintf(stdout, "Encoding random values -> keep in mind that this is very hard to compress\n");
        for (i=0;i<l_data_size;++i)     {
-               l_data[i] = i; /*rand();*/
+               l_data[i] = (OPJ_BYTE)i; /*rand();*/
        }
 
        opj_set_default_encoder_parameters(&l_param);
@@ -175,7 +162,7 @@ int main (int argc, char *argv[])
        l_param.irreversible = irreversible;
 
        /* do not bother with mct, the rsiz is set when calling opj_set_MCT*/
-       /*l_param.cp_rsiz = STD_RSIZ;*/
+       /*l_param.cp_rsiz = OPJ_STD_RSIZ;*/
 
        /* no cinema */
        /*l_param.cp_cinema = 0;*/
@@ -203,8 +190,8 @@ int main (int argc, char *argv[])
        l_param.numresolution = 6;
 
        /** progression order to use*/
-       /** LRCP, RLCP, RPCL, PCRL, CPRL */
-       l_param.prog_order = LRCP;
+       /** OPJ_LRCP, OPJ_RLCP, OPJ_RPCL, PCRL, CPRL */
+       l_param.prog_order = OPJ_LRCP;
 
        /** no "region" of interest, more precisally component */
        /* l_param.roi_compno = -1; */
@@ -228,11 +215,11 @@ int main (int argc, char *argv[])
                l_current_param_ptr->dx = 1;
                l_current_param_ptr->dy = 1;
 
-               l_current_param_ptr->h = image_height;
-               l_current_param_ptr->w = image_width;
+               l_current_param_ptr->h = (OPJ_UINT32)image_height;
+               l_current_param_ptr->w = (OPJ_UINT32)image_width;
 
                l_current_param_ptr->sgnd = 0;
-               l_current_param_ptr->prec = comp_prec;
+               l_current_param_ptr->prec = (OPJ_UINT32)comp_prec;
 
                l_current_param_ptr->x0 = 0;
                l_current_param_ptr->y0 = 0;
@@ -241,16 +228,17 @@ int main (int argc, char *argv[])
        }
 
   /* should we do j2k or jp2 ?*/
-  len = strlen( output_file );
+  len = (unsigned char)strlen( output_file );
   if( strcmp( output_file + len - 4, ".jp2" ) == 0 )
     {
-    l_codec = opj_create_compress(CODEC_JP2);
+    l_codec = opj_create_compress(OPJ_CODEC_JP2);
     }
   else
     {
-    l_codec = opj_create_compress(CODEC_J2K);
+    l_codec = opj_create_compress(OPJ_CODEC_J2K);
     }
        if (!l_codec) {
+               free(l_data);
                return 1;
        }
 
@@ -259,66 +247,66 @@ int main (int argc, char *argv[])
        opj_set_warning_handler(l_codec, warning_callback,00);
        opj_set_error_handler(l_codec, error_callback,00);
 
-       l_image = opj_image_tile_create(num_comps,l_params,CLRSPC_SRGB);
+       l_image = opj_image_tile_create(num_comps,l_params,OPJ_CLRSPC_SRGB);
        if (! l_image) {
+               free(l_data);
                opj_destroy_codec(l_codec);
                return 1;
        }
 
        l_image->x0 = 0;
        l_image->y0 = 0;
-       l_image->x1 = image_width;
-       l_image->y1 = image_height;
-       l_image->color_space = CLRSPC_SRGB;
+       l_image->x1 = (OPJ_UINT32)image_width;
+       l_image->y1 = (OPJ_UINT32)image_height;
+       l_image->color_space = OPJ_CLRSPC_SRGB;
 
        if (! opj_setup_encoder(l_codec,&l_param,l_image)) {
                fprintf(stderr, "ERROR -> test_tile_encoder: failed to setup the codec!\n");
                opj_destroy_codec(l_codec);
                opj_image_destroy(l_image);
+               free(l_data);
                return 1;
        }
 
-       l_file = fopen(output_file,"wb");
-       if (! l_file) {
-               fprintf(stderr, "ERROR -> test_tile_encoder: failed to create the output file!\n");
+    l_stream = opj_stream_create_default_file_stream(output_file, OPJ_FALSE);
+    if (! l_stream) {
+               fprintf(stderr, "ERROR -> test_tile_encoder: failed to create the stream from the output file %s !\n",output_file );
                opj_destroy_codec(l_codec);
                opj_image_destroy(l_image);
+               free(l_data);
                return 1;
        }
 
-       l_stream = opj_stream_create_default_file_stream(l_file, OPJ_FALSE);
-
        if (! opj_start_compress(l_codec,l_image,l_stream)) {
                fprintf(stderr, "ERROR -> test_tile_encoder: failed to start compress!\n");
-               opj_stream_destroy(l_stream);
-               fclose(l_file);
+        opj_stream_destroy(l_stream);
                opj_destroy_codec(l_codec);
                opj_image_destroy(l_image);
+               free(l_data);
                return 1;
        }
 
        for (i=0;i<l_nb_tiles;++i) {
                if (! opj_write_tile(l_codec,i,l_data,l_data_size,l_stream)) {
                        fprintf(stderr, "ERROR -> test_tile_encoder: failed to write the tile %d!\n",i);
-                       opj_stream_destroy(l_stream);
-                       fclose(l_file);
+            opj_stream_destroy(l_stream);
                        opj_destroy_codec(l_codec);
                        opj_image_destroy(l_image);
+                       free(l_data);
                        return 1;
                }
        }
 
        if (! opj_end_compress(l_codec,l_stream)) {
                fprintf(stderr, "ERROR -> test_tile_encoder: failed to end compress!\n");
-               opj_stream_destroy(l_stream);
-               fclose(l_file);
+        opj_stream_destroy(l_stream);
                opj_destroy_codec(l_codec);
                opj_image_destroy(l_image);
+               free(l_data);
                return 1;
        }
 
-       opj_stream_destroy(l_stream);
-       fclose(l_file);
+    opj_stream_destroy(l_stream);
        opj_destroy_codec(l_codec);
        opj_image_destroy(l_image);