cppcheck fix for openjp2 (#740)
[openjpeg.git] / tests / test_tile_encoder.c
index e1d93b3da19d51c14cff7b2a89941d95e9a01c05..d01a7e5245b4da50dd3cd2b1e9ed1b1d1c5121cd 100644 (file)
 
 /* -------------------------------------------------------------------------- */
 
-/**
-sample error callback expecting a FILE* client object
-*/
-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
-*/
-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
 */
-void error_callback(const char *msg, void *client_data) {
+static void error_callback(const char *msg, void *client_data) {
        (void)client_data;
        fprintf(stdout, "[ERROR] %s", msg);
 }
 /**
 sample warning debug callback expecting no client object
 */
-void warning_callback(const char *msg, void *client_data) {
+static void warning_callback(const char *msg, void *client_data) {
        (void)client_data;
        fprintf(stdout, "[WARNING] %s", msg);
 }
 /**
 sample debug callback expecting no client object
 */
-void info_callback(const char *msg, void *client_data) {
+static void info_callback(const char *msg, void *client_data) {
        (void)client_data;
        fprintf(stdout, "[INFO] %s", msg);
 }
@@ -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;
@@ -240,17 +227,18 @@ int main (int argc, char *argv[])
                ++l_current_param_ptr;
        }
 
-  // should we do j2k or jp2 ?
-  len = strlen( output_file );
+  /* should we do j2k or jp2 ?*/
+  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,73 +247,73 @@ 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);
 
        free(l_data);
 
-       // Print profiling
-       //PROFPRINT();
+       /* Print profiling*/
+       /*PROFPRINT();*/
 
        return 0;
 }