[qcms] Make build_output_lut output 4096 points for parametric curves

third_party/qcms/src/transform_util.c

 //  qcms
 //  Copyright (C) 2009 Mozilla Foundation
 //
 // Permission is hereby granted, free of charge, to any person obtaining
 // a copy of this software and associated documentation files (the "Software"),
 // to deal in the Software without restriction, including without limitation
 // the rights to use, copy, modify, merge, publish, distribute, sublicense,
 // and/or sell copies of the Software, and to permit persons to whom the Software
 // is furnished to do so, subject to the following conditions:
 //
@@ skipping 583 matching lines @@
 }
 
 void build_output_lut(struct curveType *trc,
                 uint16_t **output_gamma_lut, size_t *output_gamma_lut_length)
 {
         if (trc->type == PARAMETRIC_CURVE_TYPE) {
                 float gamma_table[256];
                 uint16_t i;
                 uint16_t *output = malloc(sizeof(uint16_t)*256);
                 uint16_t *inverted;
-                int inverted_size = 256;
+                int inverted_size;
 
                 if (!output) {
                         *output_gamma_lut = NULL;
                         return;
                 }
 
                 compute_curve_gamma_table_type_parametric(gamma_table, trc->parameter, trc->count);
 
                 for(i = 0; i < 256; i++) {
                         output[i] = (uint16_t)(gamma_table[i] * 65535);
                 }
 
-                //XXX: the choice of a minimum of 256 here is not backed by any theory,
-                //     measurement or data, however it is what lcms uses.
-                //     the maximum number we would need is 65535 because that's the
-                //     accuracy used for computing the pre cache table
-                if (inverted_size < 256)
-                        inverted_size = 256;
-
+                // XXX: Comment required.
+                inverted_size = 4096;

[Noel Gordon, 2016/04/28 07:30:27]

Do this at line 604 diff-right. int inverted_size = 4096;

Restore the comment (from diff-left), the comment that was here before.

[radu.velea, 2016/04/28 08:17:25]

Done.

                 inverted = invert_lut(output, 256, inverted_size);
                 if (!inverted)
                         return;
                 *output_gamma_lut = inverted;
                 *output_gamma_lut_length = inverted_size;
                 free(output);
         } else {
                 if (trc->count == 0) {
                         *output_gamma_lut = build_linear_table(4096);
                         *output_gamma_lut_length = 4096;
                 } else if (trc->count == 1) {
                         float gamma = 1./u8Fixed8Number_to_float(trc->data[0]);
                         *output_gamma_lut = build_pow_table(gamma, 4096);
                         *output_gamma_lut_length = 4096;
                 } else {
                         //XXX: the choice of a minimum of 256 here is not backed by any theory, 
                         //     measurement or data, however it is what lcms uses.
                         *output_gamma_lut_length = trc->count;
                         if (*output_gamma_lut_length < 256)
                                 *output_gamma_lut_length = 256;
 
                         *output_gamma_lut = invert_lut(trc->data, trc->count, *output_gamma_lut_length);
                 }
         }
 
 }