unparser 0.2.8 → 0.3.0

data/lib/unparser/emitter/begin.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 
@@ -36,7 +38,7 @@ module Unparser
           children.empty?
         end
 
-        TERMINATING_PARENT = %i(root interpolated dyn_str_body).to_set.freeze
+        TERMINATING_PARENT = %i[root interpolated dyn_str_body].to_set.freeze
 
       private
 

data/lib/unparser/emitter/binary.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Base class for binary emitters

data/lib/unparser/emitter/block.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 
@@ -34,6 +36,7 @@ module Unparser
       #
       def emit_block_arguments
         return if arguments.children.empty?
+
         ws
         visit_parentheses(arguments, T_PIPE, T_PIPE)
       end

data/lib/unparser/emitter/case.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for case nodes
@@ -34,6 +36,7 @@ module Unparser
       def emit_else
         else_branch = children.last
         return unless else_branch
+
         write(K_ELSE)
         visit_indented(else_branch)
       end
@@ -57,6 +60,7 @@ module Unparser
       #
       def emit_condition
         return unless condition
+
         write(WS)
         visit(condition)
       end

data/lib/unparser/emitter/cbase.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for toplevel constant reference nodes

data/lib/unparser/emitter/class.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for class nodes
@@ -32,6 +34,7 @@ module Unparser
       #
       def emit_superclass
         return unless superclass
+
         write(WS, T_LT, WS)
         visit(superclass)
       end

data/lib/unparser/emitter/def.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for def node
@@ -47,6 +49,7 @@ module Unparser
       #
       def emit_arguments
         return if arguments.children.empty?
+
         visit_parentheses(arguments)
       end
 

data/lib/unparser/emitter/defined.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for defined? nodes

data/lib/unparser/emitter/empty.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 

data/lib/unparser/emitter/ensure.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 

data/lib/unparser/emitter/flipflop.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for flip flops

data/lib/unparser/emitter/flow_modifier.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter control flow modifiers
@@ -50,7 +52,7 @@ module Unparser
         end
       end
 
-      PARENS = [:if, :case, :begin].to_set.freeze
+      PARENS = %i[if case begin].to_set.freeze
 
       # Emit argument
       #

data/lib/unparser/emitter/for.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for for nodes

data/lib/unparser/emitter/hookexe.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Base class for pre and postexe emitters

data/lib/unparser/emitter/if.rb

@@ -1,6 +1,10 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter if nodes
+    #
+    # ignore :reek:RepeatedConditional
     class If < self
       handle :if
 
@@ -122,6 +126,7 @@ module Unparser
       #
       def emit_else_branch
         return unless else_branch
+
         write(K_ELSE) unless unless?
         visit_indented(else_branch)
       end

data/lib/unparser/emitter/literal.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Namespace class for literal emiters

data/lib/unparser/emitter/literal/array.rb

@@ -1,12 +1,13 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal
 
       # Array literal emitter
       class Array < self
-        OPEN      = '['.freeze
-        CLOSE     = ']'.freeze
-        DELIMITER = ', '.freeze
+        OPEN  = '['.freeze
+        CLOSE = ']'.freeze
 
         handle :array
 
@@ -20,7 +21,7 @@ module Unparser
         #
         def dispatch
           parentheses(OPEN, CLOSE) do
-            delimited(children, DELIMITER)
+            delimited(children)
           end
         end
 

data/lib/unparser/emitter/literal/dynamic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal

data/lib/unparser/emitter/literal/dynamic_body.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal

data/lib/unparser/emitter/literal/execute_string.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal

data/lib/unparser/emitter/literal/hash.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal
@@ -70,7 +72,6 @@ module Unparser
       # Emitter for hash bodies
       class HashBody < self
 
-        DELIMITER = ', '.freeze
         BAREWORD = /\A[A-Za-z_][A-Za-z_0-9]*[?!]?\z/.freeze
 
         handle :hash_body
@@ -84,7 +85,7 @@ module Unparser
         # @api private
         #
         def dispatch
-          delimited(effective_body, DELIMITER)
+          delimited(effective_body)
         end
 
         # Return effective body

data/lib/unparser/emitter/literal/primitive.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal
@@ -131,7 +133,7 @@ module Unparser
           # @api private
           #
           def dispatch
-            conditional_parentheses(parent.is_a?(Emitter::Send) && value < 0) do
+            conditional_parentheses(parent.is_a?(Send) && value.negative?) do
               write(value.inspect)
             end
           end

data/lib/unparser/emitter/literal/range.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal

data/lib/unparser/emitter/literal/regexp.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal

data/lib/unparser/emitter/literal/singleton.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Literal

data/lib/unparser/emitter/match.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 

data/lib/unparser/emitter/meta.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Namespace class for meta emitters

data/lib/unparser/emitter/module.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for module nodes

data/lib/unparser/emitter/op_assign.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 

data/lib/unparser/emitter/redo.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for redo nodes

data/lib/unparser/emitter/repetition.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
 
@@ -65,7 +67,8 @@ module Unparser
       # @api private
       #
       def postcontrol?
-        return false unless body
+        return nil unless body # greez from falsyness
+
         local_variable_scope.first_assignment_in_body_and_used_in_condition?(body, condition)
       end
 

data/lib/unparser/emitter/resbody.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for rescue body nodes
@@ -20,30 +22,6 @@ module Unparser
           write(K_RESCUE, WS)
           visit_plain(body)
         end
-
-        # Emit exception
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def emit_exception
-          return unless exception
-          ws
-          delimited(exception.children)
-        end
-
-        # Emit assignment
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def emit_assignment
-          return unless assignment
-          write(WS, T_ASR, WS)
-          visit(assignment)
-        end
       end
 
       # Emitter for resbody in keyworkd-embedded form
@@ -74,6 +52,7 @@ module Unparser
         #
         def emit_exception
           return unless exception
+
           ws
           delimited(exception.children)
         end
@@ -86,6 +65,7 @@ module Unparser
         #
         def emit_assignment
           return unless assignment
+
           write(WS, T_ASR, WS)
           visit(assignment)
         end

data/lib/unparser/emitter/rescue.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for rescue nodes
@@ -10,9 +12,9 @@ module Unparser
 
       define_group :rescue_bodies, 1..-2
 
-      EMBEDDED_TYPES = [:def, :defs, :kwbegin, :ensure].to_set.freeze
+      EMBEDDED_TYPES = %i[def defs kwbegin ensure].to_set.freeze
 
-      NOINDENT_STANDALONE_RESCUE = [:root, :begin, :pair_rocket, :pair_colon, :lvasgn, :ivasgn].to_set.freeze
+      NOINDENT_STANDALONE_RESCUE = %i[root begin pair_rocket pair_colon lvasgn ivasgn].to_set.freeze
 
     private
 
@@ -86,6 +88,7 @@ module Unparser
       #
       def emit_else
         return unless else_branch
+
         write(K_ELSE)
         visit_indented(else_branch)
       end

data/lib/unparser/emitter/retry.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for retry nodes

data/lib/unparser/emitter/root.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Root emitter a special case

data/lib/unparser/emitter/send.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     # Emitter for send
+    # ignore :reek:TooManyMethods
     class Send < self
 
       handle :send

data/lib/unparser/emitter/send/arguments.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Send

data/lib/unparser/emitter/send/attribute_assignment.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Unparser
   class Emitter
     class Send