Remove visit_Fixnum and visit_Bignum

Follow up ae406cd.
rails · Sep 30, 2018 · 8bc0a43 · 8bc0a43
1 parent 7ea546b
commit 8bc0a43
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 4 deletions.
diff --git a/activerecord/lib/arel/visitors/depth_first.rb b/activerecord/lib/arel/visitors/depth_first.rb
@@ -135,12 +135,10 @@ def terminal(o)
         alias :visit_Arel_Nodes_True               :terminal
         alias :visit_Arel_Nodes_False              :terminal
         alias :visit_BigDecimal                    :terminal
-        alias :visit_Bignum                        :terminal
         alias :visit_Class                         :terminal
         alias :visit_Date                          :terminal
         alias :visit_DateTime                      :terminal
         alias :visit_FalseClass                    :terminal
-        alias :visit_Fixnum                        :terminal
         alias :visit_Float                         :terminal
         alias :visit_Integer                       :terminal
         alias :visit_NilClass                      :terminal

diff --git a/activerecord/lib/arel/visitors/dot.rb b/activerecord/lib/arel/visitors/dot.rb
@@ -211,7 +211,6 @@ def visit_String(o)
         alias :visit_TrueClass :visit_String
         alias :visit_FalseClass :visit_String
         alias :visit_Integer :visit_String
-        alias :visit_Fixnum :visit_String
         alias :visit_BigDecimal :visit_String
         alias :visit_Float :visit_String
         alias :visit_Symbol :visit_String

diff --git a/activerecord/test/cases/arel/visitors/to_sql_test.rb b/activerecord/test/cases/arel/visitors/to_sql_test.rb
@@ -221,7 +221,7 @@ def dispatch
         sql.must_be_like "foo AS bar"
       end
 
-      it "should visit_Bignum" do
+      it "should visit_Integer" do
         compile 8787878092
       end