From 4b5e992565b060cc002495f8c58cceb79c75d53a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ra=C3=BAl=20Mar=C3=ADn?= <git@rmr.ninja>
Date: Tue, 30 Jan 2024 19:33:31 +0100
Subject: [PATCH] Fix problems

---
 src/Client/QueryFuzzer.cpp | 17 ++++-------------
 1 file changed, 4 insertions(+), 13 deletions(-)

diff --git a/src/Client/QueryFuzzer.cpp b/src/Client/QueryFuzzer.cpp
index 786d5af0cb38..bb551fcb11eb 100644
--- a/src/Client/QueryFuzzer.cpp
+++ b/src/Client/QueryFuzzer.cpp
@@ -916,22 +916,13 @@ ASTPtr QueryFuzzer::fuzzLiteralUnderExpressionList(ASTPtr child)
     }
 
     if (fuzz_rand() % 11 == 0)
-    {
-        String value = l->value.get<String>();
         child = makeASTFunction("toNullable", child);
-    }
 
     if (fuzz_rand() % 11 == 0)
-    {
-        String value = l->value.get<String>();
         child = makeASTFunction("toLowCardinality", child);
-    }
 
     if (fuzz_rand() % 11 == 0)
-    {
-        String value = l->value.get<String>();
         child = makeASTFunction("materialize", child);
-    }
 
     return child;
 }
@@ -939,15 +930,15 @@ ASTPtr QueryFuzzer::fuzzLiteralUnderExpressionList(ASTPtr child)
 
 void QueryFuzzer::fuzzExpressionList(ASTExpressionList & expr_list)
 {
-    for (size_t i = 0; i < expr_list.children.size(); i++)
+    for (auto & child : expr_list.children)
     {
-        if (auto * literal = typeid_cast<ASTLiteral *>(expr_list.children[i].get()))
+        if (auto * literal = typeid_cast<ASTLiteral *>(child.get()))
         {
             if (fuzz_rand() % 13 == 0)
-                expr_list.children[i] = fuzzLiteralUnderExpressionList(expr_list.children[i]);
+                child = fuzzLiteralUnderExpressionList(child);
         }
         else
-            fuzz(expr_list.children[i]);
+            fuzz(child);
     }
 }