Strexp#names is only used in a test, so rm

actionpack/lib/action_dispatch/journey/router/strexp.rb

@@ -21,10 +21,6 @@ def initialize(ast, path, requirements, separators, anchor = true)
           @separators   = separators
           @anchor       = anchor
         end
-
-        def names
-          @path.scan(/:\w+/).map { |s| s.tr(':', '') }
-        end
       end
     end
   end

actionpack/test/journey/router/strexp_test.rb

-require 'abstract_unit'
-
-module ActionDispatch
-  module Journey
-    class Router
-      class TestStrexp < ActiveSupport::TestCase
-        def test_many_names
-          exp = Strexp.build(
-            "/:controller(/:action(/:id(.:format)))",
-            {:controller=>/.+?/},
-            ["/", ".", "?"],
-            true)
-
-          assert_equal ["controller", "action", "id", "format"], exp.names
-        end
-
-        def test_names
-          {
-            "/bar(.:format)"    => %w{ format },
-            ":format"           => %w{ format },
-            ":format-"          => %w{ format },
-            ":format0"          => %w{ format0 },
-            ":format1,:format2" => %w{ format1 format2 },
-          }.each do |string, expected|
-            exp = Strexp.build(string, {}, ["/", ".", "?"])
-            assert_equal expected, exp.names
-          end
-        end
-      end
-    end
-  end
-end