From 285134e43dbb811dfc5bd93c70f0733dc1cb0b37 Mon Sep 17 00:00:00 2001 From: Rowan Trollope Date: Fri, 31 Jan 2025 17:43:23 -0600 Subject: [PATCH] Comment said: "XXX: check explicitly that ELE was passed, not just size" however this code is in the else block after strcasecmp already was checked for ! ELE, therefore we have already validated that this is an ELE element type. REMOVED comment for clarity. --- vset.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/vset.c b/vset.c index a8ca9f80a..12ad47284 100644 --- a/vset.c +++ b/vset.c @@ -603,8 +603,7 @@ int VSIM_RedisCommand(RedisModuleCtx *ctx, RedisModuleString **argv, int argc) { vector_args = consumed_args; /* Apply projection if the set uses it, with the exception - * of ELE type, that will already have the right dimension. - * XXX: check explicitly that ELE was passed, not just size. */ + * of ELE type, that will already have the right dimension. */ if (vset->proj_matrix && dim != vset->hnsw->vector_dim) { float *projected = applyProjection(vec, vset->proj_matrix, vset->proj_input_size, dim);