Casts can have a side-effect and thus their object can be used

crystal-ameba · Feb 2, 2025 · b0e4bf2 · b0e4bf2
1 parent 59c11ff
commit b0e4bf2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/ameba/ast/visitors/implicit_return_visitor.cr b/src/ameba/ast/visitors/implicit_return_visitor.cr
@@ -25,7 +25,7 @@ module Ameba::AST
           if exp.is_a?(Crystal::ControlExpression)
             incr_stack { exp.accept(self) }
             break
-          elsif idx == last_idx && old_stack > 0
+          elsif idx == last_idx && old_stack.positive?
             incr_stack { exp.accept(self) }
           else
             exp.accept(self)
@@ -162,7 +162,7 @@ module Ameba::AST
     def visit(node : Crystal::Cast | Crystal::NilableCast) : Bool
       @rule.test(@source, node, @stack.positive?)
 
-      node.obj.accept(self)
+      incr_stack { node.obj.accept(self) }
 
       false
     end