diff options
| author | Ryuta Kamizono <kamipo@gmail.com> | 2019-06-15 22:17:20 +0900 |
|---|---|---|
| committer | Ryuta Kamizono <kamipo@gmail.com> | 2019-06-15 23:20:16 +0900 |
| commit | fc38ff6e4417295c870f419f7c164ab5a7dbc4a5 (patch) | |
| tree | b3c24f763b5f59412c07c2360680e0eb3f00f71c /activerecord/lib/arel/visitors.rb | |
| parent | b388061947c087d0635619d7f0b3a2b1cc91a806 (diff) | |
| download | rails-fc38ff6e4417295c870f419f7c164ab5a7dbc4a5.tar.gz rails-fc38ff6e4417295c870f419f7c164ab5a7dbc4a5.tar.bz2 rails-fc38ff6e4417295c870f419f7c164ab5a7dbc4a5.zip | |
Remove unused `DepthFirst` visitor
We only use `ToSQL` visitors in the our codebase, do not use
`DepthFirst` and `Dot` visitors.
The `DepthFirst` visitor (which was introduced at c86c37e5f) is used to
traverse an Arel (partial) ast with depth first.
Is there any worth to keep that undocumented feature with much code and
test cases.
This removes that unused `DepthFirst` code and test cases.
Diffstat (limited to 'activerecord/lib/arel/visitors.rb')
| -rw-r--r-- | activerecord/lib/arel/visitors.rb | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/activerecord/lib/arel/visitors.rb b/activerecord/lib/arel/visitors.rb index e350f52e65..a1097f6750 100644 --- a/activerecord/lib/arel/visitors.rb +++ b/activerecord/lib/arel/visitors.rb @@ -1,7 +1,6 @@ # frozen_string_literal: true require "arel/visitors/visitor" -require "arel/visitors/depth_first" require "arel/visitors/to_sql" require "arel/visitors/sqlite" require "arel/visitors/postgresql" |