code_analyzer 0.4.6 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 84254a2c93bfef8fd24e54f1830729c22cb24cb9
-  data.tar.gz: d098c0883a5b139a4147118045f2c6ce3a76e034
+SHA256:
+  metadata.gz: 5a68561649e1e345495d4a494530dc7abb9ca256dc5a999d1aa71b1c567b0e66
+  data.tar.gz: d21414e33aa72bbb23571985b9eaa00e177057f7781cbb3f1398476119f38c69
 SHA512:
-  metadata.gz: baddc030af7af45aba9b808432ea3374dd10231d0b1f4c1d0624499bb27319762405e2f318a42cf0a4102ac81833ccd1b535e0813e4ddfb9fa50a996bbf7ea18
-  data.tar.gz: 0da69c9d2fc22b0db89a84b5b0f3435b06e1cfcb81cad98de5ead1bd34e89cbcdc8615b695d82b11155d1cfc608db4c1604ab4dfb52ab8dcdec397ea85d22136
+  metadata.gz: 99369336e083dd38d76e7436ee9d824565108760339d3da59b057649341917da4649e68c78f01a29ba45c09c95a2ca71c1b2392d9c2a252826cb81abb2580131
+  data.tar.gz: cbba861e5caa35dcd4a0e887159ae0a14bb166a21f2b0bb8b5e33f188c346f9b71b6b11bbee253e9ee2b18b4fc7be039801d438168d43d00b4f007b0541698d8

data/.travis.yml

@@ -1,5 +1,5 @@
 language: ruby
 cache: bundler
-sudo: false
 rvm:
-  - 2.3.1
+  - 2.6.6
+  - 2.7.1

data/code_analyzer.gemspec

@@ -1,4 +1,5 @@
-# -*- encoding: utf-8 -*-
+# frozen_string_literal: true
+
 require File.expand_path('../lib/code_analyzer/version', __FILE__)
 
 Gem::Specification.new do |gem|

data/lib/code_analyzer.rb

@@ -1,12 +1,13 @@
-# encoding: utf-8
-require "ripper"
-require "code_analyzer/version"
-require "code_analyzer/nil"
-require "code_analyzer/sexp"
+# frozen_string_literal: true
+
+require 'ripper'
+require 'code_analyzer/version'
+require 'code_analyzer/nil'
+require 'code_analyzer/sexp'
 
 module CodeAnalyzer
-   autoload :AnalyzerException, "code_analyzer/analyzer_exception"
-   autoload :Checker, "code_analyzer/checker"
-   autoload :CheckingVisitor, "code_analyzer/checking_visitor"
-   autoload :Warning, "code_analyzer/warning"
+  autoload :AnalyzerException, 'code_analyzer/analyzer_exception'
+  autoload :Checker, 'code_analyzer/checker'
+  autoload :CheckingVisitor, 'code_analyzer/checking_visitor'
+  autoload :Warning, 'code_analyzer/warning'
 end

data/lib/code_analyzer/analyzer_exception.rb

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer
   class AnalyzerException < Exception; end
 end

data/lib/code_analyzer/checker.rb

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer
   # A checker class that takes charge of checking the sexp.
   class Checker
@@ -28,9 +29,7 @@ module CodeAnalyzer
     # @param [Sexp] node
     def node_start(node)
       @node = node
-      self.class.get_callbacks("start_#{node.sexp_type}".to_sym).each do |block|
-        self.instance_exec(node, &block)
-      end
+      self.class.get_callbacks("start_#{node.sexp_type}".to_sym).each { |block| self.instance_exec(node, &block) }
     end
 
     # delegate to end_### according to the sexp_type, like
@@ -41,9 +40,7 @@ module CodeAnalyzer
     # @param [Sexp] node
     def node_end(node)
       @node = node
-      self.class.get_callbacks("end_#{node.sexp_type}".to_sym).each do |block|
-        self.instance_exec(node, &block)
-      end
+      self.class.get_callbacks("end_#{node.sexp_type}".to_sym).each { |block| self.instance_exec(node, &block) }
     end
 
     # add an warning.
@@ -60,7 +57,7 @@ module CodeAnalyzer
       @warnings ||= []
     end
 
-    class <<self
+    class << self
       def interesting_nodes(*nodes)
         @interesting_nodes ||= []
         @interesting_nodes += nodes

data/lib/code_analyzer/checking_visitor.rb

@@ -1,8 +1,9 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer
   module CheckingVisitor
-    autoload :Base, "code_analyzer/checking_visitor/base"
-    autoload :Plain, "code_analyzer/checking_visitor/plain"
-    autoload :Default, "code_analyzer/checking_visitor/default"
+    autoload :Base, 'code_analyzer/checking_visitor/base'
+    autoload :Plain, 'code_analyzer/checking_visitor/plain'
+    autoload :Default, 'code_analyzer/checking_visitor/default'
   end
 end

data/lib/code_analyzer/checking_visitor/base.rb

@@ -1,8 +1,9 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer::CheckingVisitor
   # Base class for checking visitor.
   class Base
-    def initialize(options={})
+    def initialize(options = {})
       @checkers = options[:checkers]
     end
 

data/lib/code_analyzer/checking_visitor/default.rb

@@ -1,8 +1,9 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer::CheckingVisitor
   # This is the default checking visitor to check ruby sexp nodes.
   class Default < Base
-    def initialize(options={})
+    def initialize(options = {})
       super
       @checks = {}
       @checkers.each do |checker|
@@ -28,9 +29,7 @@ module CodeAnalyzer::CheckingVisitor
     def after_check
       @checkers.each do |checker|
         after_check_callbacks = checker.class.get_callbacks(:after_check)
-        after_check_callbacks.each do |block|
-          checker.instance_exec &block
-        end
+        after_check_callbacks.each { |block| checker.instance_exec &block }
       end
     end
 
@@ -51,16 +50,14 @@ module CodeAnalyzer::CheckingVisitor
     # 3. it triggers the interesting checkers' end callbacks.
     def check_node(node)
       checkers = @checks[node.sexp_type]
-      if checkers
-        checkers.each { |checker| checker.node_start(node) if checker.parse_file?(node.file) }
-      end
-      node.children.each { |child_node|
+
+      checkers.each { |checker| checker.node_start(node) if checker.parse_file?(node.file) } if checkers
+      node.children.each do |child_node|
         child_node.file = node.file
         check_node(child_node)
-      }
-      if checkers
-        checkers.each { |checker| checker.node_end(node) if checker.parse_file?(node.file) }
       end
+
+      checkers.each { |checker| checker.node_end(node) if checker.parse_file?(node.file) } if checkers
     end
   end
 end

data/lib/code_analyzer/checking_visitor/plain.rb

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer::CheckingVisitor
   # This is the checking visitor to check ruby plain code.
   class Plain < Base
@@ -7,9 +8,7 @@ module CodeAnalyzer::CheckingVisitor
     # @param [String] filename is the filename of ruby code.
     # @param [String] content is the content of ruby file.
     def check(filename, content)
-      @checkers.each do |checker|
-        checker.check(filename, content) if checker.parse_file?(filename)
-      end
+      @checkers.each { |checker| checker.check(filename, content) if checker.parse_file?(filename) }
     end
   end
 end

data/lib/code_analyzer/nil.rb

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 module CodeAnalyzer
   # Fake nil.
   class Nil

data/lib/code_analyzer/sexp.rb

@@ -1,4 +1,5 @@
-# encoding: utf-8
+# frozen_string_literal: true
+
 require 'sexp'
 
 class Sexp
@@ -13,22 +14,23 @@ class Sexp
   #
   #     s(:@ident, "test", s(2, 12)
   #       => 2
+
   def line_number
     case sexp_type
-    when :def, :defs, :command, :command_call, :call, :fcall, :method_add_arg, :method_add_block,
-         :var_ref, :vcall, :const_ref, :const_path_ref, :class, :module,
-         :if, :unless, :elsif, :ifop, :if_mod, :unless_mod, :binary,
-         :alias, :symbol_literal, :symbol, :aref, :hash, :assoc_new, :string_literal,
-         :massign
+    when :def, :defs, :command, :command_call, :call, :fcall, :method_add_arg, :method_add_block, :var_ref, :vcall,
+         :const_ref, :top_const_ref, :const_path_ref, :class, :module, :if, :unless, :elsif, :ifop, :if_mod, :unless_mod, :binary, :alias,
+         :symbol_literal, :symbol, :aref, :hash, :assoc_new, :string_literal, :massign, :var_field, :assign, :paren, :dot2, :dot3
       self[1].line_number
     when :assoclist_from_args, :bare_assoc_hash
       self[1][0].line_number
-    when :string_add, :opassign
+    when :string_add, :opassign, :unary, :stmts_add
       self[2].line_number
     when :array
       array_values.first.line_number
     when :mlhs_add
       self.last.line_number
+    when :params
+      self[1][0].line_number if self[1].is_a? Array
     else
       self.last.first if self.last.is_a? Array
     end
@@ -38,7 +40,7 @@ class Sexp
   #
   # @return [Array] child nodes.
   def children
-    find_all { | sexp | Sexp === sexp }
+    find_all { |sexp| Sexp === sexp }
   end
 
   # recursively find all child nodes, and yeild each child node.
@@ -68,10 +70,17 @@ class Sexp
     message = options[:message]
     to_s = options[:to_s]
     self.recursive_children do |child|
-      if (!sexp_type || (sexp_type.is_a?(Array) ? sexp_type.include?(child.sexp_type) : sexp_type == child.sexp_type)) &&
-         (!receiver || (receiver.is_a?(Array) ? receiver.include?(child.receiver.to_s) : receiver == child.receiver.to_s)) &&
-         (!message || (message.is_a?(Array) ? message.include?(child.message.to_s) : message == child.message.to_s)) &&
-         (!to_s || (to_s.is_a?(Array) ? to_s.include?(child.to_s) : to_s == child.to_s))
+      if (
+           !sexp_type || (sexp_type.is_a?(Array) ? sexp_type.include?(child.sexp_type) : sexp_type == child.sexp_type)
+         ) &&
+           (
+             !receiver ||
+               (receiver.is_a?(Array) ? receiver.include?(child.receiver.to_s) : receiver == child.receiver.to_s)
+           ) &&
+           (
+             !message || (message.is_a?(Array) ? message.include?(child.message.to_s) : message == child.message.to_s)
+           ) &&
+           (!to_s || (to_s.is_a?(Array) ? to_s.include?(child.to_s) : to_s == child.to_s))
         yield child
       end
     end
@@ -91,7 +100,10 @@ class Sexp
   # the condition value can be Symbol, Array or Sexp.
   def grep_node(options)
     result = CodeAnalyzer::Nil.new
-    grep_nodes(options) { |node| result = node; break; }
+    grep_nodes(options) do |node|
+      result = node
+      break
+    end
     result
   end
 
@@ -138,9 +150,7 @@ class Sexp
   #
   # @return [Sexp] module name node
   def module_name
-    if :module == sexp_type
-      self[1]
-    end
+    self[1] if :module == sexp_type
   end
 
   # Get the class name of the class node.
@@ -154,9 +164,7 @@ class Sexp
   #
   # @return [Sexp] class name node
   def class_name
-    if :class == sexp_type
-      self[1]
-    end
+    self[1] if :class == sexp_type
   end
 
   # Get the base class of the class node.
@@ -170,9 +178,7 @@ class Sexp
   #
   # @return [Sexp] base class of class node
   def base_class
-    if :class == sexp_type
-      self[2]
-    end
+    self[2] if :class == sexp_type
   end
 
   # Get the left value of the assign node.
@@ -185,9 +191,7 @@ class Sexp
   #
   # @return [Symbol] left value of lasgn or iasgn node
   def left_value
-    if :assign == sexp_type
-      self[1]
-    end
+    self[1] if :assign == sexp_type
   end
 
   # Get the right value of assign node.
@@ -200,9 +204,7 @@ class Sexp
   #
   # @return [Sexp] right value of assign node
   def right_value
-    if :assign == sexp_type
-      self[2]
-    end
+    self[2] if :assign == sexp_type
   end
 
   # Get the message node.
@@ -277,9 +279,7 @@ class Sexp
   #
   # @return [Sexp] argument node
   def argument
-    if :binary == sexp_type
-      self[3]
-    end
+    self[3] if :binary == sexp_type
   end
 
   # Get all arguments.
@@ -305,7 +305,7 @@ class Sexp
       else
         node = self[1]
         while true
-          if [:args_add, :args_add_star].include? node.sexp_type
+          if %i[args_add args_add_star].include? node.sexp_type
             nodes.unshift node[2]
             node = node[1]
           elsif :args_new == node.sexp_type
@@ -330,9 +330,7 @@ class Sexp
   #
   # @return [Sexp] conditional statement of if node
   def conditional_statement
-    if [:if, :unless, :elsif, :ifop, :if_mod, :unless_mod].include? sexp_type
-      self[1]
-    end
+    self[1] if %i[if unless elsif ifop if_mod unless_mod].include? sexp_type
   end
 
   # Get all condition nodes.
@@ -366,13 +364,13 @@ class Sexp
   # @return [Array] all condition nodes
   def all_conditions
     nodes = []
-    if :binary == sexp_type && %w(&& || and or).include?(self[2].to_s)
-      if :binary == self[1].sexp_type && %w(&& || and or).include?(self[1][2].to_s)
+    if :binary == sexp_type && %w[&& || and or].include?(self[2].to_s)
+      if :binary == self[1].sexp_type && %w[&& || and or].include?(self[1][2].to_s)
         nodes += self[1].all_conditions
       else
         nodes << self[1]
       end
-      if :binary == self[3].sexp_type && %w(&& || and or).include?(self[3][2].to_s)
+      if :binary == self[3].sexp_type && %w[&& || and or].include?(self[3][2].to_s)
         nodes += self[3].all_conditions
       else
         nodes << self[3]
@@ -392,6 +390,7 @@ class Sexp
   #         => s(:@ident, "show", s(1, 4)),
   #
   # @return [Sexp] method name node
+
   def method_name
     case sexp_type
     when :def
@@ -399,6 +398,7 @@ class Sexp
     when :defs
       self[3]
     else
+
     end
   end
 
@@ -428,6 +428,7 @@ class Sexp
   #            )
   #
   # @return [Sexp] body node
+
   def body
     case sexp_type
     when :else
@@ -506,13 +507,15 @@ class Sexp
   # @return [Array] all statements
   def statements
     stmts = []
-    node = case sexp_type
-           when :do_block, :brace_block
-             self[2]
-           when :bodystmt
-             self[1]
-           else
-           end
+    node =
+      case sexp_type
+      when :do_block, :brace_block
+        self[2][1]
+      when :bodystmt
+        self[1]
+      else
+
+      end
     if node
       while true
         if :stmts_add == node.sexp_type && s(:void_stmt) != node[2]
@@ -542,6 +545,7 @@ class Sexp
   def exception_classes
     if :rescue == sexp_type
       return [] unless self[1]
+
       if :mrhs_add == self[1].sexp_type
         exceptions = Array.new(self[1][2])
         arg_nodes = self[1][1][1]
@@ -566,9 +570,7 @@ class Sexp
   #     )
   #         => s(:var_field, s(:@ident, "e", s(1, 20)))
   def exception_variable
-    if :rescue == sexp_type
-      self[2]
-    end
+    self[2] if :rescue == sexp_type
   end
 
   # Get hash value node.
@@ -585,20 +587,17 @@ class Sexp
   #
   # @return [Sexp] hash value node
   def hash_value(key)
-    pair_nodes = case sexp_type
-                 when :bare_assoc_hash
-                   self[1]
-                 when :hash
-                   self[1][1]
-                 else
-                 end
-    if pair_nodes
-      pair_nodes.size.times do |i|
-        if key == pair_nodes[i][1].to_s
-          return pair_nodes[i][2]
-        end
+    pair_nodes =
+      case sexp_type
+      when :bare_assoc_hash
+        self[1]
+      when :hash
+        self[1][1]
+      else
+
       end
-    end
+
+    pair_nodes.size.times { |i| return pair_nodes[i][2] if key == pair_nodes[i][1].to_s } if pair_nodes
     CodeAnalyzer::Nil.new
   end
 
@@ -640,18 +639,18 @@ class Sexp
   #
   # @return [Array] hash keys
   def hash_keys
-    pair_nodes = case sexp_type
-                 when :bare_assoc_hash
-                   self[1]
-                 when :hash
-                   self[1][1]
-                 else
-                 end
+    pair_nodes =
+      case sexp_type
+      when :bare_assoc_hash
+        self[1]
+      when :hash
+        self[1][1]
+      else
+
+      end
     if pair_nodes
       keys = []
-      pair_nodes.size.times do |i|
-        keys << pair_nodes[i][1].to_s
-      end
+      pair_nodes.size.times { |i| keys << pair_nodes[i][1].to_s }
       keys
     end
   end
@@ -673,18 +672,18 @@ class Sexp
   #
   # @return [Array] hash values
   def hash_values
-    pair_nodes = case sexp_type
-                 when :bare_assoc_hash
-                   self[1]
-                 when :hash
-                   self[1][1]
-                 else
-                 end
+    pair_nodes =
+      case sexp_type
+      when :bare_assoc_hash
+        self[1]
+      when :hash
+        self[1][1]
+      else
+
+      end
     if pair_nodes
       values = []
-      pair_nodes.size.times do |i|
-        values << pair_nodes[i][2]
-      end
+      pair_nodes.size.times { |i| values << pair_nodes[i][2] }
       values
     end
   end
@@ -709,9 +708,7 @@ class Sexp
           array_size += 1
           first_node = s(:args_new) == first_node[1] ? first_node[2] : first_node[1]
           if :args_add != first_node.sexp_type
-            if :array == first_node.sexp_type
-              array_size += first_node.array_size
-            end
+            array_size += first_node.array_size if :array == first_node.sexp_type
             break
           end
         end
@@ -737,21 +734,21 @@ class Sexp
   def array_values
     case sexp_type
     when :array
-      if nil == self[1]
+      if nil == self[1] || %i[words_new qwords_new symbols_new qsymbols_new].include?(self[1].sexp_type)
         []
-      elsif [:words_add, :qwords_add, :qsymbols_add].include? self[1].sexp_type
+      elsif %i[words_add qwords_add symbols_add qsymbols_add].include? self[1].sexp_type
         self[1].array_values
       else
         arguments.all
       end
-    when :words_add, :qwords_add, :qsymbols_add
+    when :words_add, :qwords_add, :symbols_add, :qsymbols_add
       values = []
       node = self
       while true
-        if [:words_add, :qwords_add, :qsymbols_add].include? node.sexp_type
+        if %i[words_add qwords_add symbols_add qsymbols_add].include? node.sexp_type
           values.unshift node[2]
           node = node[1]
-        elsif [:words_new, :qwords_new, :qsymbols_new].include? node.sexp_type
+        elsif %i[words_new qwords_new symbols_new qsymbols_new].include? node.sexp_type
           break
         end
       end
@@ -808,22 +805,13 @@ class Sexp
   # @return [String] to_s
   def to_s
     case sexp_type
-    when :string_literal, :xstring_literal, :string_content, :const_ref, :symbol_literal, :symbol,
-         :args_add_block, :var_ref, :vcall, :var_field,
-         :@ident, :@tstring_content, :@const, :@ivar, :@kw, :@gvar, :@cvar
+    when :string_literal, :xstring_literal, :string_content, :const_ref, :symbol_literal, :symbol, :args_add_block, :var_ref,
+         :vcall, :var_field, :@ident, :@tstring_content, :@const, :@ivar, :@kw, :@gvar, :@cvar, :@period
       self[1].to_s
     when :string_add
-      if s(:string_content) == self[1]
-        self[2].to_s
-      else
-        self[1].to_s
-      end
+      s(:string_content) == self[1] ? self[2].to_s : self[1].to_s
     when :args_add
-      if s(:args_new) == self[1]
-        self[2].to_s
-      else
-        self[1].to_s
-      end
+      s(:args_new) == self[1] ? self[2].to_s : self[1].to_s
     when :qwords_add
       self[2].to_s
     when :word_add
@@ -839,13 +827,13 @@ class Sexp
     when :top_const_ref
       "::#{self[1]}"
     else
-      ""
+      ''
     end
   end
 
   # check if the self node is a const.
   def const?
-    :@const == self.sexp_type || ([:var_ref, :vcall].include?(self.sexp_type) && :@const == self[1].sexp_type)
+    :@const == self.sexp_type || (%i[var_ref vcall].include?(self.sexp_type) && :@const == self[1].sexp_type)
   end
 
   # true
@@ -865,22 +853,33 @@ class Sexp
     if Sexp === last_node && last_node.size == 2 && last_node.first.is_a?(Integer) && last_node.last.is_a?(Integer)
       node.delete_at(-1)
     end
-    node.sexp_body.each_with_index do |child, index|
-      if Sexp === child
-        node[index+1] = child.remove_line_and_column
-      end
-    end
+    node.sexp_body.each_with_index { |child, index| node[index + 1] = child.remove_line_and_column if Sexp === child }
     node
   end
 
   # if the return value of these methods is nil, then return CodeAnalyzer::Nil.new instead
-  [:sexp_type, :receiver, :message, :arguments, :argument, :class_name, :base_class, :method_name,
-   :body, :block_node, :conditional_statement, :left_value, :right_value].each do |method|
+  %i[
+    sexp_type
+    receiver
+    message
+    arguments
+    argument
+    class_name
+    base_class
+    method_name
+    body
+    block_node
+    conditional_statement
+    left_value
+    right_value
+  ].each do |method|
     class_eval <<-EOS
       alias_method :origin_#{method}, :#{method}
 
       def #{method}
-        ret = origin_#{method}
+        ret = origin_#{
+      method
+    }
         ret.nil? ? CodeAnalyzer::Nil.new : ret
       end
     EOS