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

third_party/qcms/src/transform_util.h

Index: third_party/qcms/src/transform_util.h
diff --git a/third_party/qcms/src/transform_util.h b/third_party/qcms/src/transform_util.h
index 44ff63ec8f0a87e3c914d1a94e32e9875b58db77..f83477aed66b51f7627086c4a3ebe47955b8cd62 100644
--- a/third_party/qcms/src/transform_util.h
+++ b/third_party/qcms/src/transform_util.h
@@ -31,8 +31,8 @@
 //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);