use if-initialize statement to create scoped variables

apache · Jun 24, 2024 · a609470 · a609470
1 parent 93deee5
commit a609470
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/commands/cmd_stream.cc b/src/commands/cmd_stream.cc
@@ -367,11 +367,11 @@ class CommandAutoClaim : public Commander {
     key_name_ = GET_OR_RET(parser.TakeStr());
     group_name_ = GET_OR_RET(parser.TakeStr());
     consumer_name_ = GET_OR_RET(parser.TakeStr());
-    auto parse_int_status = parser.TakeInt<uint64_t>();
-    if (!parse_int_status.IsOK()) {
+    if (auto parse_status = parser.TakeInt<uint64_t>(); !parse_status.IsOK()) {
       return {Status::RedisParseErr, "Invalid min-idle-time argument for XAUTOCLAIM"};
+    } else {
+      options_.min_idle_time_ms = parse_status.GetValue();
     }
-    options_.min_idle_time_ms = parse_int_status.GetValue();
 
     auto start_str = GET_OR_RET(parser.TakeStr());
     if (!start_str.empty() && start_str.front() == '(') {
@@ -381,9 +381,9 @@ class CommandAutoClaim : public Commander {
     if (!options_.exclude_start && start_str == "-") {
       options_.start_id = StreamEntryID::Minimum();
     } else {
-      auto parse_range_status = ParseRangeStart(start_str, &options_.start_id);
-      if (!parse_range_status.IsOK()) {
-        return parse_range_status;
+      auto parse_status = ParseRangeStart(start_str, &options_.start_id);
+      if (!parse_status.IsOK()) {
+        return parse_status;
       }
     }