From bc07139db323770778307179ab9eddf5b877bac2 Mon Sep 17 00:00:00 2001
From: Jean Boussier <jean.boussier@gmail.com>
Date: Wed, 22 Dec 2021 13:45:10 +0100
Subject: [PATCH] Remove feature checking for Class#descendants

Ref: https://bugs.ruby-lang.org/issues/14394#note-38

Based on Matz's last comment, it's not so clear whether `Class#descendants`
will come back in the same form or at all. So let's not assume anything.
---
 .../core_ext/class/subclasses.rb              | 44 +++++++++----------
 .../lib/active_support/descendants_tracker.rb | 13 +-----
 .../lib/active_support/ruby_features.rb       |  1 -
 3 files changed, 23 insertions(+), 35 deletions(-)

diff --git a/activesupport/lib/active_support/core_ext/class/subclasses.rb b/activesupport/lib/active_support/core_ext/class/subclasses.rb
index 11c5968572..1e6799e77e 100644
--- a/activesupport/lib/active_support/core_ext/class/subclasses.rb
+++ b/activesupport/lib/active_support/core_ext/class/subclasses.rb
@@ -3,29 +3,27 @@
 require "active_support/ruby_features"
 
 class Class
-  unless ActiveSupport::RubyFeatures::CLASS_DESCENDANTS
-    if ActiveSupport::RubyFeatures::CLASS_SUBCLASSES
-      def descendants
-        subclasses.concat(subclasses.flat_map(&:descendants))
-      end
-    else
-      # Returns an array with all classes that are < than its receiver.
-      #
-      #   class C; end
-      #   C.descendants # => []
-      #
-      #   class B < C; end
-      #   C.descendants # => [B]
-      #
-      #   class A < B; end
-      #   C.descendants # => [B, A]
-      #
-      #   class D < C; end
-      #   C.descendants # => [B, A, D]
-      def descendants
-        ObjectSpace.each_object(singleton_class).reject do |k|
-          k.singleton_class? || k == self
-        end
+  if ActiveSupport::RubyFeatures::CLASS_SUBCLASSES
+    def descendants
+      subclasses.concat(subclasses.flat_map(&:descendants))
+    end
+  else
+    # Returns an array with all classes that are < than its receiver.
+    #
+    #   class C; end
+    #   C.descendants # => []
+    #
+    #   class B < C; end
+    #   C.descendants # => [B]
+    #
+    #   class A < B; end
+    #   C.descendants # => [B, A]
+    #
+    #   class D < C; end
+    #   C.descendants # => [B, A, D]
+    def descendants
+      ObjectSpace.each_object(singleton_class).reject do |k|
+        k.singleton_class? || k == self
       end
     end
   end
diff --git a/activesupport/lib/active_support/descendants_tracker.rb b/activesupport/lib/active_support/descendants_tracker.rb
index 4fe496cd22..ea3ed1f41e 100644
--- a/activesupport/lib/active_support/descendants_tracker.rb
+++ b/activesupport/lib/active_support/descendants_tracker.rb
@@ -51,7 +51,6 @@ module ActiveSupport
           unless @clear_disabled
             @clear_disabled = true
             remove_method(:subclasses)
-            remove_method(:descendants) if RubyFeatures::CLASS_DESCENDANTS
             @@excluded_descendants = nil
           end
         end
@@ -86,16 +85,8 @@ module ActiveSupport
         subclasses
       end
 
-      if RubyFeatures::CLASS_DESCENDANTS
-        def descendants
-          descendants = super
-          descendants.reject! { |d| @@excluded_descendants[d] }
-          descendants
-        end
-      else
-        def descendants
-          subclasses.concat(subclasses.flat_map(&:descendants))
-        end
+      def descendants
+        subclasses.concat(subclasses.flat_map(&:descendants))
       end
 
       def direct_descendants
diff --git a/activesupport/lib/active_support/ruby_features.rb b/activesupport/lib/active_support/ruby_features.rb
index 30b8b13440..8cdb89c20e 100644
--- a/activesupport/lib/active_support/ruby_features.rb
+++ b/activesupport/lib/active_support/ruby_features.rb
@@ -3,6 +3,5 @@
 module ActiveSupport
   module RubyFeatures # :nodoc:
     CLASS_SUBCLASSES = Class.method_defined?(:subclasses) # RUBY_VERSION >= "3.1"
-    CLASS_DESCENDANTS = Class.method_defined?(:descendants)
   end
 end