Review comments: Explained 33 sample size, rename xyz to pcs in some places, fix spelling

1 files changed, 5 insertions, 0 deletions

diff --git a/transform.c b/transform.c
index cded207..72e5535 100644
--- a/transform.c
+++ b/transform.c
@@ -1115,6 +1115,11 @@ qcms_transform* qcms_transform_create(
 	}
 
 	if (in->A2B0 || out->B2A0 || in->mAB || out->mAB) {
+		// Precache the transformation to a CLUT 33x33x33 in size.
+		// 33 is used by many profiles and works well in pratice. 
+		// This evenly divides 256 into blocks of 8x8x8.
+		// TODO For transforming small data sets of about 200x200 or less
+		// precaching should be avoided.
 		qcms_transform *result = qcms_transform_precacheLUT_float(transform, in, out, 33, in_type);
 		if (!result) {
             		assert(0 && "precacheLUT failed");