Skip to content

Commit

Permalink
Add comment explaiing alraedy_added_var
Browse files Browse the repository at this point in the history
  • Loading branch information
SharafMohamed committed Aug 2, 2024
1 parent db8e544 commit a360cd8
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions components/core/src/clp/Grep.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1179,6 +1179,10 @@ vector<QueryLogtype> 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;
}
Expand Down

0 comments on commit a360cd8

Please sign in to comment.