lib/yard/handlers/ruby/method_handler.rb in yard-0.9.5 vs lib/yard/handlers/ruby/method_handler.rb in yard-0.9.6

@@ -1,5 +1,6 @@
+# frozen_string_literal: true
 # Handles a method definition
 class YARD::Handlers::Ruby::MethodHandler < YARD::Handlers::Ruby::Base
   handles :def, :defs
 
   process do
@@ -56,15 +57,16 @@
           obj.add_tag(new_tag)
         end
       end
     end
 
-    if info = obj.attr_info
+    info = obj.attr_info
+    if info
       if meth.to_s =~ /=$/ # writer
         info[:write] = obj if info[:read]
-      else
-        info[:read] = obj if info[:write]
+      elsif info[:write]
+        info[:read] = obj
       end
     end
 
     parse_block(blk, :owner => obj) # mainly for yield/exceptions
   end
@@ -73,25 +75,23 @@
     args = statement.parameters
 
     params = []
 
     if args.unnamed_required_params
-      params += args.unnamed_required_params.map { |a| [a.source, nil] }
+      params += args.unnamed_required_params.map {|a| [a.source, nil] }
     end
 
     if args.unnamed_optional_params
       params += args.unnamed_optional_params.map do |a|
         [a[0].source, a[1].source]
       end
     end
 
-    if args.splat_param
-      params << ['*' + args.splat_param.source, nil]
-    end
+    params << ['*' + args.splat_param.source, nil] if args.splat_param
 
     if args.unnamed_end_params
-      params += args.unnamed_end_params.map { |a| [a.source, nil] }
+      params += args.unnamed_end_params.map {|a| [a.source, nil] }
     end
 
     if args.named_params
       params += args.named_params.map do |a|
         [a[0].source, a[1] ? a[1].source : nil]
@@ -100,12 +100,10 @@
 
     if args.double_splat_param
       params << ['**' + args.double_splat_param.source, nil]
     end
 
-    if args.block_param
-      params << ['&' + args.block_param.source, nil]
-    end
+    params << ['&' + args.block_param.source, nil] if args.block_param
 
     params
   end
 
   def method_signature