[qcms] Keep the output of the TRC between 0 and 1

third_party/qcms/src/transform_util.h

 /* vim: set ts=8 sw=8 noexpandtab: */
 //  qcms
 //  Copyright (C) 2009 Mozilla Foundation
 //  Copyright (C) 1998-2007 Marti Maria
 //
 // 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 
@@ skipping 13 matching lines @@
 #ifndef _QCMS_TRANSFORM_UTIL_H
 #define _QCMS_TRANSFORM_UTIL_H
 
 #include <stdlib.h>
 
 #define CLU(table,x,y,z) table[(x*len + y*x_len + z*xy_len)*3]
 
 //XXX: could use a bettername
 typedef uint16_t uint16_fract_t;
 
-float lut_interp_linear(double value, uint16_t *table, size_t length);
-float lut_interp_linear_float(float value, float *table, size_t length);
+float lut_interp_linear(double input_value, uint16_t *table, size_t length);

[Matt Giuca, 2015/07/01 01:46:59]

The upstream patch doesn't change these names in the header file.

[Noel Gordon, 2015/07/01 03:48:54]

Well, "consistency" maybe.

[Matt Giuca, 2015/07/01 07:10:50]

It would be consistent with the .c file to use the "input_value" name, but it is
more important to remain compatible with the upstream code.

+float lut_interp_linear_float(float input_value, float *table, size_t length);
 uint16_t lut_interp_linear16(uint16_t input_value, uint16_t *table, size_t length);
 
 
 static inline float lerp(float a, float b, float t)
 {
         return a*(1.f-t) + b*t;
 }
 
 unsigned char clamp_u8(float v);
 float clamp_float(float a);
 
 float u8Fixed8Number_to_float(uint16_t x);
 
 
 float *build_input_gamma_table(struct curveType *TRC);
 struct matrix build_colorant_matrix(qcms_profile *p);
 void build_output_lut(struct curveType *trc,
                       uint16_t **output_gamma_lut, size_t *output_gamma_lut_length);
 
 struct matrix matrix_invert(struct matrix mat);
 qcms_bool compute_precache(struct curveType *trc, uint8_t *output);
 
 #endif