From dd5dc25e6536263a939c39692dfd32967852589d Mon Sep 17 00:00:00 2001
From: Igor Nikonov <igor@clickhouse.com>
Date: Fri, 26 Apr 2024 11:26:05 +0000
Subject: [PATCH] Remove unnecessary check

---
 src/Functions/FunctionsCodingUUID.cpp | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/Functions/FunctionsCodingUUID.cpp b/src/Functions/FunctionsCodingUUID.cpp
index 240bb3a32fc..f14b7720cfb 100644
--- a/src/Functions/FunctionsCodingUUID.cpp
+++ b/src/Functions/FunctionsCodingUUID.cpp
@@ -128,14 +128,11 @@ UUIDSerializer::Variant parseVariant(const DB::ColumnsWithTypeAndName & argument
     if (arguments.size() < 2)
         return UUIDSerializer::Variant::Default;
 
-    Int64 value = static_cast<UInt64>(UUIDSerializer::Variant::Default);
-    if (!arguments[1].column->empty())
-        value = arguments[1].column->getInt(0);
+    const auto representation = static_cast<magic_enum::underlying_type_t<UUIDSerializer::Variant>>(arguments[1].column->getInt(0));
+    const auto as_enum = magic_enum::enum_cast<UUIDSerializer::Variant>(representation);
 
-    const auto as_enum
-        = magic_enum::enum_cast<UUIDSerializer::Variant>(static_cast<int>(value));
     if (!as_enum)
-        throw DB::Exception(DB::ErrorCodes::ARGUMENT_OUT_OF_BOUND, "Expected UUID variant, got {}", value);
+        throw DB::Exception(DB::ErrorCodes::ARGUMENT_OUT_OF_BOUND, "Expected UUID variant, got {}", representation);
 
     return *as_enum;
 }