unparser 0.4.7 → 0.6.7

data/lib/unparser/emitter/flow_modifier.rb

@@ -4,77 +4,39 @@ module Unparser
   class Emitter
     # Emitter control flow modifiers
     class FlowModifier < self
-
       MAP = {
-        return: K_RETURN,
-        next:   K_NEXT,
-        break:  K_BREAK
+        return: 'return',
+        next:   'next',
+        break:  'break'
       }.freeze
 
+      private_constant(*constants(false))
+
       handle(*MAP.keys)
 
-      def terminated?
-        children.empty?
+      def emit_heredoc_reminders
+        children.each do |node|
+          emitter(node).emit_heredoc_reminders
+        end
       end
 
     private
 
-      # Perform dispatch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def dispatch
         write(MAP.fetch(node.type))
-        case children.length
-        when 0 # rubocop:disable Lint/EmptyWhen
-        when 1
-          emit_single_argument
+
+        if children.one? && n_if?(children.first)
+          ws
+          emitter(children.first).emit_ternary
         else
-          emit_arguments
+          emit_arguments unless children.empty?
         end
       end
 
-      # Emit break or return arguments
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_arguments
         ws
-        head, *tail = children
-        emit_argument(head)
-        tail.each do |node|
-          write(DEFAULT_DELIMITER)
-          emit_argument(node)
-        end
+        delimited(children)
       end
-
-      PARENS = %i[if case begin].to_set.freeze
-
-      # Emit argument
-      #
-      # @param [Parser::AST::Node] node
-      #
-      # @api private
-      #
-      def emit_argument(node)
-        visit_plain(node)
-      end
-
-      # Emit single argument
-      #
-      # @api private
-      #
-      def emit_single_argument
-        ws
-        conditional_parentheses(PARENS.include?(first_child.type)) do
-          visit_plain(first_child)
-        end
-      end
-
     end # Return
   end # Emitter
 end # Unparser

data/lib/unparser/emitter/for.rb

@@ -4,38 +4,24 @@ module Unparser
   class Emitter
     # Emitter for for nodes
     class For < self
-      include Terminated
-
       handle :for
 
       children :condition, :assignment, :body
 
     private
 
-      # Perform dispatch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def dispatch
-        write(K_FOR, WS)
+        write('for ')
         emit_condition
-        emit_body
+        emit_optional_body(body)
         k_end
       end
 
-      # Emit assignment
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_condition
-        visit_plain(condition)
-        write(WS, K_IN, WS)
+        visit(condition)
+        write(' in ')
         visit(assignment)
-        write(WS, K_DO)
+        write(' do')
       end
 
     end # For

data/lib/unparser/emitter/hash.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Unparser
+  class Emitter
+    # Emitter for Hash literals
+    class Hash < self
+      handle :hash
+
+      def emit_heredoc_reminders
+        children.each(&method(:emit_heredoc_reminder_member))
+      end
+
+    private
+
+      def dispatch
+        if children.empty?
+          write('{}')
+        else
+          parentheses('{', '}') do
+            write(' ')
+            emit_hash_body
+            write(' ')
+          end
+        end
+      end
+
+      def emit_heredoc_reminder_member(node)
+        emitter(node.children.last).emit_heredoc_reminders
+      end
+
+      def emit_hash_body
+        delimited(children)
+      end
+    end # Hash
+  end # Emitter
+end # Unparser

data/lib/unparser/emitter/hash_pattern.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+module Unparser
+  class Emitter
+    # Emitter for hash patterns
+    class HashPattern < self
+
+      handle :hash_pattern
+
+      def emit_const_pattern
+        parentheses do
+          emit_hash_body
+        end
+      end
+
+    private
+
+      def dispatch
+        parentheses('{', '}') do
+          emit_hash_body
+        end
+      end
+
+      def emit_hash_body
+        delimited(children, &method(:emit_member))
+      end
+
+      def emit_member(node)
+        case node.type
+        when :pair
+          emit_pair(node)
+        when :match_var
+          emit_match_var(node)
+        when :match_rest
+          writer_with(MatchRest, node).emit_hash_pattern
+        else
+          visit(node)
+        end
+      end
+
+      def emit_match_var(node)
+        write_symbol_body(node.children.first)
+        write(':')
+      end
+
+      def emit_pair(node)
+        key, value = node.children
+
+        if n_sym?(key)
+          write_symbol_body(key.children.first)
+        else
+          visit(s(:dstr, *key))
+        end
+
+        write(':')
+
+        ws
+
+        visit(value)
+      end
+
+      def write_symbol_body(symbol)
+        write(symbol.inspect[1..])
+      end
+    end # Pin
+  end # Emitter
+end # Unparser

data/lib/unparser/emitter/hookexe.rb

@@ -6,8 +6,8 @@ module Unparser
     class Hookexe < self
 
       MAP = {
-        preexe:  K_PREEXE,
-        postexe: K_POSTEXE
+        preexe:  'BEGIN',
+        postexe: 'END'
       }.freeze
 
       handle(*MAP.keys)
@@ -16,16 +16,10 @@ module Unparser
 
     private
 
-      # Perfrom dispatch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def dispatch
-        write(MAP.fetch(node.type), WS)
-        parentheses(*BRACKETS_CURLY) do
-          emit_body
+        write(MAP.fetch(node.type), ' ')
+        parentheses('{', '}') do
+          emit_body(body)
         end
       end
 

data/lib/unparser/emitter/if.rb

@@ -3,25 +3,21 @@
 module Unparser
   class Emitter
     # Emitter if nodes
-    #
-    # ignore :reek:RepeatedConditional
     class If < self
       handle :if
 
       children :condition, :if_branch, :else_branch
 
-      def terminated?
-        !postcondition?
+      def emit_ternary
+        visit(condition)
+        write(' ? ')
+        visit(if_branch)
+        write(' : ')
+        visit(else_branch)
       end
 
     private
 
-      # Perform dispatch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def dispatch
         if postcondition?
           emit_postcondition
@@ -30,105 +26,53 @@ module Unparser
         end
       end
 
-      # Test for postcondition
-      #
-      # @return [Boolean]
-      #
-      # @api private
-      #
       def postcondition?
         return false unless if_branch.nil? ^ else_branch.nil?
 
         body = if_branch || else_branch
 
-        local_variable_scope.first_assignment_in_body_and_used_in_condition?(body, condition)
+        local_variable_scope.first_assignment_in?(body, condition)
       end
 
-      # Emit in postcondition style
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_postcondition
-        visit_plain(if_branch || else_branch)
-        write(WS, keyword, WS)
+        visit(if_branch || else_branch)
+        write(' ', keyword, ' ')
         emit_condition
       end
 
-      # Emit in normal style
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_normal
-        write(keyword, WS)
+        write(keyword, ' ')
         emit_condition
         emit_if_branch
         emit_else_branch
         k_end
       end
 
-      # Test if AST can be emitted as unless
-      #
-      # @return [Boolean]
-      #
-      # @api private
-      #
       def unless?
         !if_branch && else_branch
       end
 
-      # Return keyword
-      #
-      # @return [String]
-      #
-      # @api private
-      #
       def keyword
-        unless? ? K_UNLESS : K_IF
+        unless? ? 'unless' : 'if'
       end
 
-      # Emit condition
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_condition
-        if condition.type.equal?(:match_current_line)
-          visit_plain(condition)
-        else
-          visit(condition)
-        end
+        visit(condition)
       end
 
-      # Emit if branch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_if_branch
         if if_branch
-          visit_indented(if_branch)
+          emit_body(if_branch)
         end
 
         nl if !if_branch && !else_branch
       end
 
-      # Emit else branch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_else_branch
         return unless else_branch
 
-        write(K_ELSE) unless unless?
-        visit_indented(else_branch)
+        write('else') unless unless?
+        emit_body(else_branch)
       end
 
     end # If

data/lib/unparser/emitter/in_match.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Unparser
+  class Emitter
+    # Emitter for in pattern nodes
+    class InMatch < self
+
+      handle :in_match
+
+      children :target, :pattern
+
+    private
+
+      def dispatch
+        visit(target)
+        write(' in ')
+        visit(pattern)
+      end
+    end # InMatch
+  end # Emitter
+end # Unparser

data/lib/unparser/emitter/in_pattern.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Unparser
+  class Emitter
+    # Emitter for in pattern nodes
+    class InPattern < self
+
+      handle :in_pattern
+
+      children :target, :unless_guard, :branch, :else_branch
+
+    private
+
+      def dispatch
+        write('in')
+
+        ws
+
+        visit(target)
+
+        if unless_guard
+          ws
+          visit(unless_guard)
+        end
+
+        if branch
+          ws
+          write('then')
+          emit_body(branch)
+        else
+          nl
+        end
+      end
+    end # InPattern
+  end # Emitter
+end # Unparser

data/lib/unparser/emitter/index.rb

@@ -3,61 +3,29 @@
 module Unparser
   class Emitter
     # Emitter for send to index references
-    #
-    # ignore :reek:RepeatedConditional
     class Index < self
 
-      # Perform dispatch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
+    private
+
       def dispatch
         emit_receiver
         emit_operation
       end
 
-    private
-
-      # Emit receiver
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def emit_receiver
         visit(first_child)
       end
 
-      # Test for mlhs
-      #
-      # @return [Boolean]
-      #
-      # @api private
-      #
-      def mlhs?
-        parent_type.equal?(:mlhs)
-      end
-
       class Reference < self
-        include Terminated
-
         define_group(:indices, 1..-1)
 
         handle :index
 
       private
 
-        # Emit arguments
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
         def emit_operation
-          parentheses(*BRACKETS_SQUARE) do
-            delimited_plain(indices)
+          parentheses('[', ']') do
+            delimited(indices)
           end
         end
       end # Reference
@@ -68,67 +36,32 @@ module Unparser
         handle :indexasgn
 
         VALUE_RANGE     = (1..-2).freeze
-        NO_VALUE_PARENT = IceNine.deep_freeze(%i[and_asgn op_asgn or_asgn].to_set)
-
-        # Test if assign will be emitted terminated
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        #
-        def terminated?
-          !emit_value?
-        end
+        NO_VALUE_PARENT = %i[and_asgn op_asgn or_asgn].to_set.freeze
 
-      private
+        private_constant(*constants(false))
 
-        # Emit arguments
-        #
-        # @return [undefined]
-        #
-        # @api private
-        #
-        def emit_operation
-          parentheses(*BRACKETS_SQUARE) do
-            delimited_plain(indices)
-          end
-
-          if emit_value?
-            write(WS, T_ASN, WS)
-            visit(children.last)
-          end
+        def emit_heredoc_reminders
+          emitter(children.last).emit_heredoc_reminders
         end
 
-        # The indices
-        #
-        # @return [Array<Parser::AST::Node>]
-        #
-        def indices
-          if emit_value?
-            children[VALUE_RANGE]
-          else
-            children.drop(1)
-          end
+        def dispatch
+          emit_receiver
+          emit_operation(children[VALUE_RANGE])
+          write(' = ')
+          visit(children.last)
         end
 
-        # Test if value should be emitted
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        #
-        def emit_value?
-          !mlhs? && !no_value_parent?
+        def emit_mlhs
+          emit_receiver
+          emit_operation(children.drop(1))
         end
 
-        # Test for no value parent
-        #
-        # @return [Boolean]
-        #
-        # @api private
-        #
-        def no_value_parent?
-          NO_VALUE_PARENT.include?(parent_type)
+      private
+
+        def emit_operation(indices)
+          parentheses('[', ']') do
+            delimited(indices)
+          end
         end
       end # Assign
     end # Index

data/lib/unparser/emitter/kwargs.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module Unparser
+  class Emitter
+    class Kwargs < self
+      handle :kwargs
+
+      def dispatch
+        delimited(children)
+      end
+    end # Kwargs
+  end # Emitter
+end # Unparser

data/lib/unparser/emitter/kwbegin.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Unparser
+  class Emitter
+    # Emitter for explicit begins
+    class KWBegin < self
+      handle :kwbegin
+
+    private
+
+      def dispatch
+        write('begin')
+
+        if children.one?
+          emit_body_ensure_rescue(children.first)
+        else
+          indented do
+            emit_multiple_body
+          end
+        end
+
+        k_end
+      end
+
+      def emit_multiple_body
+        emit_join(children, method(:emit_body_member), method(:nl))
+      end
+
+    end # KWBegin
+  end # Emitter
+end # Unparser

data/lib/unparser/emitter/lambda.rb

@@ -4,18 +4,10 @@ module Unparser
   class Emitter
     # Emitter for lambda nodes
     class Lambda < self
-      include Terminated
-
       handle :lambda
 
     private
 
-      # Perform dispatch
-      #
-      # @return [undefined]
-      #
-      # @api private
-      #
       def dispatch
         write('->')
       end