From a360cd83fcd0594d0d135ad87b5746fbbf0a83f9 Mon Sep 17 00:00:00 2001 From: SharafMohamed Date: Fri, 2 Aug 2024 10:10:32 -0400 Subject: [PATCH] Add comment explaiing alraedy_added_var --- components/core/src/clp/Grep.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/components/core/src/clp/Grep.cpp b/components/core/src/clp/Grep.cpp index 0751ae3f7..a75fa6630 100644 --- a/components/core/src/clp/Grep.cpp +++ b/components/core/src/clp/Grep.cpp @@ -1179,6 +1179,10 @@ vector Grep::get_possible_substr_types( for (int id : variable_types) { auto& schema_type = lexer.m_id_symbol[id]; if (schema_type != "int" && schema_type != "float") { + // LogSurgeon differentiates between all variable types. For example, LogSurgeon + // might report thet types has#, userID, and int. However, CLP only supports + // dict, int, and float variables. So there is no benefit in duplicating the + // dict variable option for both has# and userID in the example. if (already_added_var) { continue; }