reek 1.2.2 → 1.2.3

data/lib/reek/module_context.rb

@@ -1,32 +1,30 @@
 require 'reek/code_context'
+require 'reek/code_parser'
+require 'reek/sniffer'
 
 module Reek
   class ModuleContext < CodeContext
 
-    def ModuleContext.create(outer, exp)
-      res = Name.resolve(exp[1], outer)
-      ModuleContext.new(res[0], res[1])
-    end
+    class << self
+      def create(outer, exp)
+        res = Name.resolve(exp[1], outer)
+        new(res[0], res[1], exp)
+      end
 
-    def ModuleContext.from_s(src)
-      source = src.to_reek_source
-      sniffer = Sniffer.new(source)
-      CodeParser.new(sniffer).process_module(source.syntax_tree)
+      def from_s(src)
+        source = src.to_reek_source
+        sniffer = Sniffer.new(source)
+        CodeParser.new(sniffer).do_module_or_class(source.syntax_tree, self)
+      end
     end
 
-    attr_reader :class_variables, :attributes
+    attr_reader :attributes
 
-    def initialize(outer, name)
-      super(outer, nil)
+    def initialize(outer, name, exp)
+      super(outer, exp)
       @name = name
       @attributes = Set.new
-      @class_variables = Set.new
-    end
-
-    def check_for_attribute_declaration(exp)
-      if [:attr, :attr_reader, :attr_writer, :attr_accessor].include? exp[2]
-        exp[3][1..-1].each {|arg| record_attribute(arg[1])}
-      end
+      @parsed_methods = []
     end
 
     def myself
@@ -38,16 +36,26 @@ module Reek
       @myself.const_or_nil(modname.to_s)
     end
 
-    def outer_name
-      "#{@outer.outer_name}#{@name}::"
+    def parameterized_methods(min_clump_size)
+      @parsed_methods.select {|meth| meth.parameters.length >= min_clump_size }
     end
 
     def record_attribute(attr)
       @attributes << Name.new(attr)
     end
 
-    def record_class_variable(cvar)
-      @class_variables << Name.new(cvar)
+    def record_method(meth)
+      @parsed_methods << meth
+    end
+
+    def check_for_attribute_declaration(exp)
+      if [:attr, :attr_reader, :attr_writer, :attr_accessor].include? exp[2]
+        exp[3][1..-1].each {|arg| record_attribute(arg[1])}
+      end
+    end
+
+    def outer_name
+      "#{@outer.outer_name}#{@name}::"
     end
 
     def variable_names

data/lib/reek/name.rb

@@ -1,3 +1,5 @@
+require 'ruby_parser'
+
 module Reek
   class Name
     include Comparable

data/lib/reek/object_refs.rb

@@ -1,6 +1,3 @@
-require 'rubygems'
-require 'sexp_processor'
-
 module Reek
 
   class ObjectRefs  # :nodoc:

data/lib/reek/sexp_formatter.rb

@@ -1,5 +1,3 @@
-require 'rubygems'
-require 'ruby_parser'
 require 'ruby2ruby'
 
 module Reek

data/lib/reek/smells/class_variable.rb

@@ -18,14 +18,27 @@ module Reek
       end
 
       #
-      # Checks whether the given class declares any class variables.
+      # Checks whether the given class or module declares any class variables.
       # Remembers any smells found.
       #
-      def examine_context(klass)
-        klass.class_variables.each do |cvar|
-          found(klass, "declares the class variable #{cvar}")
+      def examine_context(mod)
+        class_variables_in(mod).each do |cvar_name|
+          found(mod, "declares the class variable #{cvar_name}")
         end
       end
+
+      #
+      # Collects the names of the class variables declared and/or used
+      # in the given module.
+      #
+      def class_variables_in(mod)
+        result = Set.new
+        collector = proc { |cvar_node| result << cvar_node.name }
+        [:cvar, :cvasgn, :cvdecl].each do |stmt_type|
+          mod.each(stmt_type, [:class, :module], &collector)
+        end
+        result
+      end
     end
   end
 end

data/lib/reek/smells/control_couple.rb

@@ -34,19 +34,12 @@ module Reek
     # because it includes at least two different code paths.
     #
     class ControlCouple < SmellDetector
+      include ExcludeInitialize
 
       def self.contexts      # :nodoc:
         [:if]
       end
 
-      def self.default_config
-        super.adopt(EXCLUDE_KEY => ['initialize'])
-      end
-
-      def initialize(config = ControlCouple.default_config)
-        super
-      end
-
       #
       # Checks whether the given conditional statement relies on a control couple.
       # Remembers any smells found.
@@ -56,8 +49,8 @@ module Reek
         # SMELL: Duplication
         # This smell is reported once for each conditional that tests the
         # same parameter. Which means that the same smell can recur within
-        # a single sniffer. Which in turn means that the sniffer can't count
-        # its smells without knowing which are duplicates.
+        # a single detector. Which in turn means that SmellDetector must
+        # use a Set to hold smells found.
         found(cond, "is controlled by argument #{SexpFormatter.format(cond.if_expr)}")
       end
     end

data/lib/reek/smells/data_clump.rb

@@ -20,7 +20,7 @@ module Reek
     class DataClump < SmellDetector
 
       def self.contexts      # :nodoc:
-        [:class]
+        [:class, :module]
       end
 
       # The name of the config field that sets the maximum allowed
@@ -44,14 +44,14 @@ module Reek
       end
 
       #
-      # Checks the given ClassContext for multiple identical conditional tests.
+      # Checks the given class or module for multiple identical parameter sets.
       # Remembers any smells found.
       #
-      def examine_context(klass)
-        max_copies = value(MAX_COPIES_KEY, klass, DEFAULT_MAX_COPIES)
-        min_clump_size = value(MIN_CLUMP_SIZE_KEY, klass, DEFAULT_MIN_CLUMP_SIZE)
-        MethodGroup.new(klass, min_clump_size, max_copies).clumps.each do |clump, occurs|
-          found(klass, "takes parameters #{DataClump.print_clump(clump)} to #{occurs} methods")
+      def examine_context(ctx)
+        max_copies = value(MAX_COPIES_KEY, ctx, DEFAULT_MAX_COPIES)
+        min_clump_size = value(MIN_CLUMP_SIZE_KEY, ctx, DEFAULT_MIN_CLUMP_SIZE)
+        MethodGroup.new(ctx, min_clump_size, max_copies).clumps.each do |clump, occurs|
+          found(ctx, "takes parameters #{DataClump.print_clump(clump)} to #{occurs} methods")
         end
       end
 
@@ -68,15 +68,15 @@ module Reek
       methods.map {|meth| meth.parameters.names.sort}.intersection
     end
 
-    def initialize(klass, min_clump_size, max_copies)
-      @klass = klass
+    def initialize(ctx, min_clump_size, max_copies)
+      @ctx = ctx
       @min_clump_size = min_clump_size
       @max_copies = max_copies
     end
 
     def clumps
       results = Hash.new(0)
-      @klass.parameterized_methods(@min_clump_size).bounded_power_set(@max_copies).each do |methods|
+      @ctx.parameterized_methods(@min_clump_size).bounded_power_set(@max_copies).each do |methods|
         clump = MethodGroup.intersection_of_parameters_of(methods)
         if clump.length >= @min_clump_size
           results[clump] = [methods.length, results[clump]].max

data/lib/reek/smells/feature_envy.rb

@@ -33,14 +33,7 @@ module Reek
     # often than it refers to (ie. send messages to) some other object.
     #
     class FeatureEnvy < SmellDetector
-
-      def self.default_config
-        super.adopt(EXCLUDE_KEY => ['initialize'])
-      end
-
-      def initialize(config = FeatureEnvy.default_config)
-        super
-      end
+      include ExcludeInitialize
 
       #
       # Checks whether the given +context+ includes any code fragment that

data/lib/reek/smells/large_class.rb

@@ -45,9 +45,9 @@ module Reek
       end
 
       def check_num_methods(klass)  # :nodoc:
-        count = klass.num_methods
-        return if count <= value(MAX_ALLOWED_METHODS_KEY, klass, DEFAULT_MAX_METHODS)
-        found(klass, "has at least #{count} methods")
+        actual = klass.each(:defn, [:class, :module]).length
+        return if actual <= value(MAX_ALLOWED_METHODS_KEY, klass, DEFAULT_MAX_METHODS)
+        found(klass, "has at least #{actual} methods")
       end
 
       def check_num_ivars(klass)  # :nodoc:

data/lib/reek/smells/simulated_polymorphism.rb

@@ -47,12 +47,26 @@ module Reek
       # Remembers any smells found.
       #
       def examine_context(klass)
-        counts = Hash.new(0)
-        klass.conditionals.each {|cond| counts[cond] += 1}
-        counts.each do |key, val|
+        conditional_counts(klass).each do |key, val|
           found(klass, "tests #{SexpFormatter.format(key)} at least #{val} times") if val > value(MAX_IDENTICAL_IFS_KEY, klass, DEFAULT_MAX_IFS)
         end
       end
+
+      #
+      # Returns a Hash listing all of the conditional expressions in
+      # the given syntax tree together with the number of times each
+      # occurs. Ignores nested classes and modules.
+      #
+      def conditional_counts(klass)
+        result = Hash.new(0)
+        collector = proc { |node|
+          condition = node.condition
+          result[condition] += 1 unless condition == s(:call, nil, :block_given?, s(:arglist))
+        }
+        klass.each(:if, [:class, :module], &collector)
+        klass.each(:case, [:class, :module], &collector)
+        result
+      end
     end
   end
 end

data/lib/reek/smells/smell_detector.rb

@@ -10,6 +10,15 @@ end
 module Reek
   module Smells
 
+    module ExcludeInitialize
+      def self.default_config
+        super.adopt(EXCLUDE_KEY => ['initialize'])
+      end
+      def initialize(config = self.class.default_config)
+        super
+      end
+    end
+
     class SmellDetector
 
       # The name of the config field that lists the names of code contexts
@@ -70,11 +79,11 @@ module Reek
       end
 
       def configure_with(config)
-        @config.hash.adopt!(config)
+        @config.adopt!(config)
       end
 
       def copy
-        self.class.new(@config.hash.deep_copy)
+        self.class.new(@config.deep_copy)
       end
 
       def supersede_with(config)

data/lib/reek/smells/utility_function.rb

@@ -30,7 +30,7 @@ module Reek
         super.adopt(HELPER_CALLS_LIMIT_KEY => DEFAULT_HELPER_CALLS_LIMIT)
       end
 
-      def initialize(config = Duplication.default_config)
+      def initialize(config = UtilityFunction.default_config)
         super(config)
       end
 

data/lib/reek/sniffer.rb

@@ -125,10 +125,6 @@ module Reek
       stack.has_smell?(patterns)
     end
 
-    def smells_only_of?(klass, patterns)
-      num_smells == 1 and has_smell?(klass, patterns)
-    end
-
     def sniff
       self
     end
@@ -175,10 +171,6 @@ private
       total
     end
 
-    def smells_only_of?(klass, patterns)
-      num_smells == 1 and has_smell?(klass, patterns)
-    end
-
     def sniff
       self
     end

data/lib/reek/stop_context.rb

@@ -5,11 +5,11 @@ module Reek
       @refs = ObjectRefs.new
       @myself = Object
     end
+
     def method_missing(method, *args)
       nil
     end
 
-
     def count_statements(num)
       0
     end

data/lib/reek/tree_dresser.rb

@@ -0,0 +1,74 @@
+module Reek
+
+  #
+  # Extensions to +Sexp+ to allow +CodeParser+ to navigate the abstract
+  # syntax tree more easily.
+  #
+  module SexpNode
+    def children
+      find_all { |item| Sexp === item }
+    end
+
+    def is_language_node?
+      first.class == Symbol
+    end
+
+    def has_type?(type)
+      is_language_node? and first == type
+    end
+
+    #
+    # Carries out a depth-first traversal of this syntax tree, yielding
+    # every Sexp of type +target_type+. The traversal ignores any node
+    # whose type is listed in the Array +ignoring+.
+    #
+    def look_for(target_type, ignoring, &blk)
+      each do |elem|
+        if Sexp === elem then
+          elem.look_for(target_type, ignoring, &blk) unless ignoring.include?(elem.first)
+        end
+      end
+      blk.call(self) if first == target_type
+    end
+  end
+
+  module CvarNode
+    def name
+      self[1]
+    end
+  end
+
+  module IfNode
+    def condition
+      self[1]
+    end
+  end
+
+  module CaseNode
+    def condition
+      self[1]
+    end
+  end
+
+  class TreeDresser
+    # SMELL: Duplication
+    # Put these into a new module and build the mapping automagically
+    # based on the node type
+    EXTENSIONS = {
+      :cvar => CvarNode,
+      :cvasgn => CvarNode,
+      :cvdecl => CvarNode,
+      :if => IfNode,
+      :case => CaseNode,
+    }
+
+    def dress(sexp)
+      sexp.extend(SexpNode)
+      if EXTENSIONS.has_key?(sexp[0])
+        sexp.extend(EXTENSIONS[sexp[0]])
+      end
+      sexp[0..-1].each { |sub| dress(sub) if Array === sub }
+      sexp
+    end
+  end
+end

data/reek.gemspec

@@ -2,17 +2,17 @@
 
 Gem::Specification.new do |s|
   s.name = %q{reek}
-  s.version = "1.2.2"
+  s.version = "1.2.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Kevin Rutherford"]
-  s.date = %q{2009-10-06}
+  s.date = %q{2009-11-02}
   s.default_executable = %q{reek}
   s.description = %q{Code smell detector for Ruby}
   s.email = ["kevin@rutherford-software.com"]
   s.executables = ["reek"]
   s.extra_rdoc_files = ["History.txt", "License.txt", "README.rdoc"]
-  s.files = ["History.txt", "License.txt", "README.rdoc", "Rakefile", "bin/reek", "config/defaults.reek", "features/masking_smells.feature", "features/options.feature", "features/reports.feature", "features/samples.feature", "features/stdin.feature", "features/step_definitions/reek_steps.rb", "features/support/env.rb", "lib/reek.rb", "lib/reek/adapters/application.rb", "lib/reek/adapters/command_line.rb", "lib/reek/adapters/config_file.rb", "lib/reek/adapters/core_extras.rb", "lib/reek/adapters/rake_task.rb", "lib/reek/adapters/report.rb", "lib/reek/adapters/source.rb", "lib/reek/adapters/spec.rb", "lib/reek/block_context.rb", "lib/reek/class_context.rb", "lib/reek/code_context.rb", "lib/reek/code_parser.rb", "lib/reek/configuration.rb", "lib/reek/detector_stack.rb", "lib/reek/exceptions.reek", "lib/reek/if_context.rb", "lib/reek/method_context.rb", "lib/reek/module_context.rb", "lib/reek/name.rb", "lib/reek/object_refs.rb", "lib/reek/sexp_formatter.rb", "lib/reek/singleton_method_context.rb", "lib/reek/smell_warning.rb", "lib/reek/smells/attribute.rb", "lib/reek/smells/class_variable.rb", "lib/reek/smells/control_couple.rb", "lib/reek/smells/data_clump.rb", "lib/reek/smells/duplication.rb", "lib/reek/smells/feature_envy.rb", "lib/reek/smells/large_class.rb", "lib/reek/smells/long_method.rb", "lib/reek/smells/long_parameter_list.rb", "lib/reek/smells/long_yield_list.rb", "lib/reek/smells/nested_iterators.rb", "lib/reek/smells/simulated_polymorphism.rb", "lib/reek/smells/smell_detector.rb", "lib/reek/smells/uncommunicative_name.rb", "lib/reek/smells/utility_function.rb", "lib/reek/sniffer.rb", "lib/reek/stop_context.rb", "lib/reek/yield_call_context.rb", "reek.gemspec", "spec/reek/adapters/report_spec.rb", "spec/reek/adapters/should_reek_of_spec.rb", "spec/reek/adapters/should_reek_only_of_spec.rb", "spec/reek/adapters/should_reek_spec.rb", "spec/reek/block_context_spec.rb", "spec/reek/class_context_spec.rb", "spec/reek/code_context_spec.rb", "spec/reek/code_parser_spec.rb", "spec/reek/config_spec.rb", "spec/reek/configuration_spec.rb", "spec/reek/if_context_spec.rb", "spec/reek/method_context_spec.rb", "spec/reek/module_context_spec.rb", "spec/reek/name_spec.rb", "spec/reek/object_refs_spec.rb", "spec/reek/object_source_spec.rb", "spec/reek/singleton_method_context_spec.rb", "spec/reek/smell_warning_spec.rb", "spec/reek/smells/attribute_spec.rb", "spec/reek/smells/behaves_like_variable_detector.rb", "spec/reek/smells/class_variable_spec.rb", "spec/reek/smells/control_couple_spec.rb", "spec/reek/smells/data_clump_spec.rb", "spec/reek/smells/duplication_spec.rb", "spec/reek/smells/feature_envy_spec.rb", "spec/reek/smells/large_class_spec.rb", "spec/reek/smells/long_method_spec.rb", "spec/reek/smells/long_parameter_list_spec.rb", "spec/reek/smells/nested_iterators_spec.rb", "spec/reek/smells/simulated_polymorphism_spec.rb", "spec/reek/smells/smell_detector_spec.rb", "spec/reek/smells/uncommunicative_name_spec.rb", "spec/reek/smells/utility_function_spec.rb", "spec/reek/sniffer_spec.rb", "spec/reek/stop_context_spec.rb", "spec/samples/all_but_one_masked/clean_one.rb", "spec/samples/all_but_one_masked/dirty.rb", "spec/samples/all_but_one_masked/masked.reek", "spec/samples/clean_due_to_masking/clean_one.rb", "spec/samples/clean_due_to_masking/clean_three.rb", "spec/samples/clean_due_to_masking/clean_two.rb", "spec/samples/clean_due_to_masking/dirty_one.rb", "spec/samples/clean_due_to_masking/dirty_two.rb", "spec/samples/clean_due_to_masking/masked.reek", "spec/samples/corrupt_config_file/corrupt.reek", "spec/samples/corrupt_config_file/dirty.rb", "spec/samples/empty_config_file/dirty.rb", "spec/samples/empty_config_file/empty.reek", "spec/samples/exceptions.reek", "spec/samples/inline.rb", "spec/samples/masked/dirty.rb", "spec/samples/masked/masked.reek", "spec/samples/mixed_results/clean_one.rb", "spec/samples/mixed_results/clean_three.rb", "spec/samples/mixed_results/clean_two.rb", "spec/samples/mixed_results/dirty_one.rb", "spec/samples/mixed_results/dirty_two.rb", "spec/samples/not_quite_masked/dirty.rb", "spec/samples/not_quite_masked/masked.reek", "spec/samples/optparse.rb", "spec/samples/overrides/masked/dirty.rb", "spec/samples/overrides/masked/lower.reek", "spec/samples/overrides/upper.reek", "spec/samples/redcloth.rb", "spec/samples/three_clean_files/clean_one.rb", "spec/samples/three_clean_files/clean_three.rb", "spec/samples/three_clean_files/clean_two.rb", "spec/samples/two_smelly_files/dirty_one.rb", "spec/samples/two_smelly_files/dirty_two.rb", "spec/spec.opts", "spec/spec_helper.rb", "tasks/reek.rake", "tasks/test.rake"]
+  s.files = ["History.txt", "License.txt", "README.rdoc", "Rakefile", "bin/reek", "config/defaults.reek", "features/masking_smells.feature", "features/options.feature", "features/profile.feature", "features/rake_task.feature", "features/reports.feature", "features/samples.feature", "features/stdin.feature", "features/step_definitions/reek_steps.rb", "features/support/env.rb", "lib/reek.rb", "lib/reek/adapters/application.rb", "lib/reek/adapters/command_line.rb", "lib/reek/adapters/config_file.rb", "lib/reek/adapters/core_extras.rb", "lib/reek/adapters/rake_task.rb", "lib/reek/adapters/report.rb", "lib/reek/adapters/source.rb", "lib/reek/adapters/spec.rb", "lib/reek/block_context.rb", "lib/reek/class_context.rb", "lib/reek/code_context.rb", "lib/reek/code_parser.rb", "lib/reek/configuration.rb", "lib/reek/detector_stack.rb", "lib/reek/if_context.rb", "lib/reek/method_context.rb", "lib/reek/module_context.rb", "lib/reek/name.rb", "lib/reek/object_refs.rb", "lib/reek/sexp_formatter.rb", "lib/reek/singleton_method_context.rb", "lib/reek/smell_warning.rb", "lib/reek/smells/attribute.rb", "lib/reek/smells/class_variable.rb", "lib/reek/smells/control_couple.rb", "lib/reek/smells/data_clump.rb", "lib/reek/smells/duplication.rb", "lib/reek/smells/feature_envy.rb", "lib/reek/smells/large_class.rb", "lib/reek/smells/long_method.rb", "lib/reek/smells/long_parameter_list.rb", "lib/reek/smells/long_yield_list.rb", "lib/reek/smells/nested_iterators.rb", "lib/reek/smells/simulated_polymorphism.rb", "lib/reek/smells/smell_detector.rb", "lib/reek/smells/uncommunicative_name.rb", "lib/reek/smells/utility_function.rb", "lib/reek/sniffer.rb", "lib/reek/stop_context.rb", "lib/reek/tree_dresser.rb", "lib/reek/yield_call_context.rb", "reek.gemspec", "spec/reek/adapters/report_spec.rb", "spec/reek/adapters/should_reek_of_spec.rb", "spec/reek/adapters/should_reek_only_of_spec.rb", "spec/reek/adapters/should_reek_spec.rb", "spec/reek/block_context_spec.rb", "spec/reek/class_context_spec.rb", "spec/reek/code_context_spec.rb", "spec/reek/code_parser_spec.rb", "spec/reek/config_spec.rb", "spec/reek/configuration_spec.rb", "spec/reek/if_context_spec.rb", "spec/reek/method_context_spec.rb", "spec/reek/module_context_spec.rb", "spec/reek/name_spec.rb", "spec/reek/object_refs_spec.rb", "spec/reek/object_source_spec.rb", "spec/reek/singleton_method_context_spec.rb", "spec/reek/smell_warning_spec.rb", "spec/reek/smells/attribute_spec.rb", "spec/reek/smells/behaves_like_variable_detector.rb", "spec/reek/smells/class_variable_spec.rb", "spec/reek/smells/control_couple_spec.rb", "spec/reek/smells/data_clump_spec.rb", "spec/reek/smells/duplication_spec.rb", "spec/reek/smells/feature_envy_spec.rb", "spec/reek/smells/large_class_spec.rb", "spec/reek/smells/long_method_spec.rb", "spec/reek/smells/long_parameter_list_spec.rb", "spec/reek/smells/nested_iterators_spec.rb", "spec/reek/smells/simulated_polymorphism_spec.rb", "spec/reek/smells/smell_detector_spec.rb", "spec/reek/smells/uncommunicative_name_spec.rb", "spec/reek/smells/utility_function_spec.rb", "spec/reek/sniffer_spec.rb", "spec/reek/stop_context_spec.rb", "spec/samples/all_but_one_masked/clean_one.rb", "spec/samples/all_but_one_masked/dirty.rb", "spec/samples/all_but_one_masked/masked.reek", "spec/samples/clean_due_to_masking/clean_one.rb", "spec/samples/clean_due_to_masking/clean_three.rb", "spec/samples/clean_due_to_masking/clean_two.rb", "spec/samples/clean_due_to_masking/dirty_one.rb", "spec/samples/clean_due_to_masking/dirty_two.rb", "spec/samples/clean_due_to_masking/masked.reek", "spec/samples/corrupt_config_file/corrupt.reek", "spec/samples/corrupt_config_file/dirty.rb", "spec/samples/empty_config_file/dirty.rb", "spec/samples/empty_config_file/empty.reek", "spec/samples/exceptions.reek", "spec/samples/inline.rb", "spec/samples/masked/dirty.rb", "spec/samples/masked/masked.reek", "spec/samples/mixed_results/clean_one.rb", "spec/samples/mixed_results/clean_three.rb", "spec/samples/mixed_results/clean_two.rb", "spec/samples/mixed_results/dirty_one.rb", "spec/samples/mixed_results/dirty_two.rb", "spec/samples/not_quite_masked/dirty.rb", "spec/samples/not_quite_masked/masked.reek", "spec/samples/optparse.rb", "spec/samples/overrides/masked/dirty.rb", "spec/samples/overrides/masked/lower.reek", "spec/samples/overrides/upper.reek", "spec/samples/redcloth.rb", "spec/samples/three_clean_files/clean_one.rb", "spec/samples/three_clean_files/clean_three.rb", "spec/samples/three_clean_files/clean_two.rb", "spec/samples/two_smelly_files/dirty_one.rb", "spec/samples/two_smelly_files/dirty_two.rb", "spec/spec.opts", "spec/spec_helper.rb", "tasks/reek.rake", "tasks/test.rake"]
   s.homepage = %q{http://wiki.github.com/kevinrutherford/reek}
   s.post_install_message = %q{
 For more information on reek, see http://wiki.github.com/kevinrutherford/reek