No need to nodoc private methods

Author: amatsuda

actionpack/lib/action_controller/metal/instrumentation.rb

@@ -83,14 +83,14 @@ def halted_callback_hook(filter)
     #   end
     #
     # :api: plugin
-    def cleanup_view_runtime #:nodoc:
+    def cleanup_view_runtime
       yield
     end
 
     # Every time after an action is processed, this method is invoked
     # with the payload, so you can add more information.
     # :api: plugin
-    def append_info_to_payload(payload) #:nodoc:
+    def append_info_to_payload(payload)
       payload[:view_runtime] = view_runtime
     end
 

actionpack/lib/action_controller/metal/params_wrapper.rb

@@ -135,7 +135,7 @@ def name
         #
         # This method also does namespace lookup. Foo::Bar::UsersController will
         # try to find Foo::Bar::User, Foo::User and finally User.
-        def _default_wrap_model #:nodoc:
+        def _default_wrap_model
           return nil if klass.anonymous?
           model_name = klass.name.sub(/Controller$/, "").classify
 

actionpack/lib/action_controller/metal/rendering.rb

@@ -73,14 +73,14 @@ def _set_rendered_content_type(format)
       end
 
       # Normalize arguments by catching blocks and setting them on :update.
-      def _normalize_args(action = nil, options = {}, &blk) #:nodoc:
+      def _normalize_args(action = nil, options = {}, &blk)
         options = super
         options[:update] = blk if block_given?
         options
       end
 
       # Normalize both text and status options.
-      def _normalize_options(options) #:nodoc:
+      def _normalize_options(options)
         _normalize_text(options)
 
         if options[:html]
@@ -103,7 +103,7 @@ def _normalize_text(options)
       end
 
       # Process controller specific options, as status, content-type and location.
-      def _process_options(options) #:nodoc:
+      def _process_options(options)
         status, content_type, location = options.values_at(:status, :content_type, :location)
 
         self.status = status if status

actionpack/lib/action_dispatch/http/mime_type.rb

@@ -297,7 +297,7 @@ def method_missing(method, *args)
         end
       end
 
-      def respond_to_missing?(method, include_private = false) #:nodoc:
+      def respond_to_missing?(method, include_private = false)
         method.to_s.ends_with? "?"
       end
   end

actionpack/lib/action_dispatch/routing/mapper.rb

@@ -996,65 +996,65 @@ def defaults(defaults = {})
         end
 
         private
-          def merge_path_scope(parent, child) #:nodoc:
+          def merge_path_scope(parent, child)
             Mapper.normalize_path("#{parent}/#{child}")
           end
 
-          def merge_shallow_path_scope(parent, child) #:nodoc:
+          def merge_shallow_path_scope(parent, child)
             Mapper.normalize_path("#{parent}/#{child}")
           end
 
-          def merge_as_scope(parent, child) #:nodoc:
+          def merge_as_scope(parent, child)
             parent ? "#{parent}_#{child}" : child
           end
 
-          def merge_shallow_prefix_scope(parent, child) #:nodoc:
+          def merge_shallow_prefix_scope(parent, child)
             parent ? "#{parent}_#{child}" : child
           end
 
-          def merge_module_scope(parent, child) #:nodoc:
+          def merge_module_scope(parent, child)
             parent ? "#{parent}/#{child}" : child
           end
 
-          def merge_controller_scope(parent, child) #:nodoc:
+          def merge_controller_scope(parent, child)
             child
           end
 
-          def merge_action_scope(parent, child) #:nodoc:
+          def merge_action_scope(parent, child)
             child
           end
 
-          def merge_via_scope(parent, child) #:nodoc:
+          def merge_via_scope(parent, child)
             child
           end
 
-          def merge_format_scope(parent, child) #:nodoc:
+          def merge_format_scope(parent, child)
             child
           end
 
-          def merge_path_names_scope(parent, child) #:nodoc:
+          def merge_path_names_scope(parent, child)
             merge_options_scope(parent, child)
           end
 
-          def merge_constraints_scope(parent, child) #:nodoc:
+          def merge_constraints_scope(parent, child)
             merge_options_scope(parent, child)
           end
 
-          def merge_defaults_scope(parent, child) #:nodoc:
+          def merge_defaults_scope(parent, child)
             merge_options_scope(parent, child)
           end
 
-          def merge_blocks_scope(parent, child) #:nodoc:
+          def merge_blocks_scope(parent, child)
             merged = parent ? parent.dup : []
             merged << child if child
             merged
           end
 
-          def merge_options_scope(parent, child) #:nodoc:
+          def merge_options_scope(parent, child)
             (parent || {}).merge(child)
           end
 
-          def merge_shallow_scope(parent, child) #:nodoc:
+          def merge_shallow_scope(parent, child)
             child ? true : false
           end
 
@@ -1868,7 +1868,7 @@ def using_match_shorthand?(path)
             path =~ %r{^/?[-\w]+/[-\w/]+$}
           end
 
-          def decomposed_match(path, controller, options, _path, to, via, formatted, anchor, options_constraints) # :nodoc:
+          def decomposed_match(path, controller, options, _path, to, via, formatted, anchor, options_constraints)
             if on = options.delete(:on)
               send(on) { decomposed_match(path, controller, options, _path, to, via, formatted, anchor, options_constraints) }
             else
@@ -1883,7 +1883,7 @@ def decomposed_match(path, controller, options, _path, to, via, formatted, ancho
             end
           end
 
-          def add_route(action, controller, options, _path, to, via, formatted, anchor, options_constraints) # :nodoc:
+          def add_route(action, controller, options, _path, to, via, formatted, anchor, options_constraints)
             path = path_for_action(action, _path)
             raise ArgumentError, "path is required" if path.blank?