reek 1.2.7.2 → 1.2.7.3

data/History.txt

@@ -1,4 +1,12 @@
-== 1.2.8 (in development)
+== 1.2.7.3 (2010-03-29)
+
+=== Minor Changes
+* UtilityFunction no longer reported when local method called in param initializer (#60)
+* Spaces removed from smell class names in report output
+* Masked smells are no longer reported
+** the -a command-line option has been removed
+** some methods on Examiner are now deprecated
+* DataClump no longer needs infinite memory for large classes (#57 again)
 
 == 1.2.7.2 (2010-03-05)
 

data/config/defaults.reek

@@ -1,7 +1,7 @@
 --- 
 UncommunicativeParameterName: 
-  accept: 
-  - Inline::C
+  accept: []
+
   exclude: []
 
   enabled: true
@@ -78,8 +78,8 @@ Attribute:
 
   enabled: false
 UncommunicativeVariableName: 
-  accept: 
-  - Inline::C
+  accept: []
+
   exclude: []
 
   enabled: true

data/features/masking_smells.feature

@@ -10,12 +10,12 @@ Feature: Masking smells using config files
     And it reports:
       """
       spec/samples/empty_config_file/dirty.rb -- 6 warnings:
-        Dirty has the variable name '@s' (Uncommunicative Name)
+        Dirty has the variable name '@s' (UncommunicativeName)
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        Dirty#a has the name 'a' (Uncommunicative Name)
-        Dirty#a has the variable name 'x' (Uncommunicative Name)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
+        Dirty#a has the name 'a' (UncommunicativeName)
+        Dirty#a has the variable name 'x' (UncommunicativeName)
 
       """
 
@@ -25,13 +25,13 @@ Feature: Masking smells using config files
     And it reports:
       """
       spec/samples/corrupt_config_file/dirty.rb -- 7 warnings:
-        Dirty has no descriptive comment (Irresponsible Module)
-        Dirty has the variable name '@s' (Uncommunicative Name)
+        Dirty has no descriptive comment (IrresponsibleModule)
+        Dirty has the variable name '@s' (UncommunicativeName)
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        Dirty#a has the name 'a' (Uncommunicative Name)
-        Dirty#a has the variable name 'x' (Uncommunicative Name)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
+        Dirty#a has the name 'a' (UncommunicativeName)
+        Dirty#a has the variable name 'x' (UncommunicativeName)
 
       """
     And it reports the error 'Error: Invalid configuration file "corrupt.reek" -- "This is not a config file" is not a code smell'
@@ -44,7 +44,7 @@ Feature: Masking smells using config files
       spec/samples/masked/dirty.rb -- 3 warnings:
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
 
       """
     And it reports the error "Error: No such file - no_such_file.rb"
@@ -57,34 +57,7 @@ Feature: Masking smells using config files
       spec/samples/masked/dirty.rb -- 3 warnings:
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-
-      """
-
-  Scenario: switch off one smell but show all in the report
-    When I run reek --show-all spec/samples/masked/dirty.rb
-    Then the exit status indicates smells
-    And it reports:
-      """
-      spec/samples/masked/dirty.rb -- 6 warnings:
-        (masked) Dirty has the variable name '@s' (Uncommunicative Name)
-        Dirty#a calls @s.title twice (Duplication)
-        Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        (masked) Dirty#a has the name 'a' (Uncommunicative Name)
-        (masked) Dirty#a has the variable name 'x' (Uncommunicative Name)
-
-      """
-
-  Scenario: switch off one smell and hide them in the report
-    When I run reek --no-show-all spec/samples/masked/dirty.rb
-    Then the exit status indicates smells
-    And it reports:
-      """
-      spec/samples/masked/dirty.rb -- 3 warnings:
-        Dirty#a calls @s.title twice (Duplication)
-        Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
 
       """
 
@@ -94,11 +67,11 @@ Feature: Masking smells using config files
     And it reports:
       """
       spec/samples/not_quite_masked/dirty.rb -- 5 warnings:
-        Dirty has the variable name '@s' (Uncommunicative Name)
+        Dirty has the variable name '@s' (UncommunicativeName)
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        Dirty#a has the name 'a' (Uncommunicative Name)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
+        Dirty#a has the name 'a' (UncommunicativeName)
 
       """
 
@@ -113,19 +86,3 @@ Feature: Masking smells using config files
         Dirty#a calls puts(@s.title) twice (Duplication)
 
       """
-
-  @overrides
-  Scenario: all show up masked even when overridden
-    When I run reek --show-all spec/samples/overrides
-    Then the exit status indicates smells
-    And it reports:
-      """
-      spec/samples/overrides/masked/dirty.rb -- 6 warnings:
-        (masked) Dirty has the variable name '@s' (Uncommunicative Name)
-        Dirty#a calls @s.title twice (Duplication)
-        Dirty#a calls puts(@s.title) twice (Duplication)
-        (masked) Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        (masked) Dirty#a has the name 'a' (Uncommunicative Name)
-        (masked) Dirty#a has the variable name 'x' (Uncommunicative Name)
-
-      """

data/features/options.feature

@@ -25,7 +25,7 @@ Feature: Reek can be controlled using command-line options
       Examples:
 
       reek lib/*.rb
-      reek -q -a lib
+      reek -q lib
       cat my_class.rb | reek
 
       See http://wiki.github.com/kevinrutherford/reek for detailed help.
@@ -36,7 +36,6 @@ Feature: Reek can be controlled using command-line options
 
 
       Report formatting:
-          -a, --[no-]show-all              Show all smells, including those masked by config settings
           -q, --[no-]quiet                 Suppress headings for smell-free source files
           -y, --yaml                       Report smells in YAML format
 

data/features/rake_task.feature

@@ -16,7 +16,7 @@ Feature: Reek can be driven through its Task
       spec/samples/masked/dirty.rb -- 3 warnings:
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
       """
 
   Scenario: name changes the task name
@@ -32,7 +32,7 @@ Feature: Reek can be driven through its Task
       spec/samples/masked/dirty.rb -- 3 warnings:
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
       """
 
   Scenario: verbose prints the reek command
@@ -58,10 +58,10 @@ Feature: Reek can be driven through its Task
     And it reports:
       """
       spec/samples/empty_config_file/dirty.rb -- 6 warnings:
-        Dirty has the variable name '@s' (Uncommunicative Name)
+        Dirty has the variable name '@s' (UncommunicativeName)
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        Dirty#a has the name 'a' (Uncommunicative Name)
-        Dirty#a has the variable name 'x' (Uncommunicative Name)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
+        Dirty#a has the name 'a' (UncommunicativeName)
+        Dirty#a has the variable name 'x' (UncommunicativeName)
       """

data/features/reports.feature

@@ -10,19 +10,19 @@ Feature: Correctly formatted reports
     And it reports:
       """
       spec/samples/two_smelly_files/dirty_one.rb -- 6 warnings:
-        Dirty has the variable name '@s' (Uncommunicative Name)
+        Dirty has the variable name '@s' (UncommunicativeName)
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        Dirty#a has the name 'a' (Uncommunicative Name)
-        Dirty#a has the variable name 'x' (Uncommunicative Name)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
+        Dirty#a has the name 'a' (UncommunicativeName)
+        Dirty#a has the variable name 'x' (UncommunicativeName)
       spec/samples/two_smelly_files/dirty_two.rb -- 6 warnings:
-        Dirty has the variable name '@s' (Uncommunicative Name)
+        Dirty has the variable name '@s' (UncommunicativeName)
         Dirty#a calls @s.title twice (Duplication)
         Dirty#a calls puts(@s.title) twice (Duplication)
-        Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        Dirty#a has the name 'a' (Uncommunicative Name)
-        Dirty#a has the variable name 'x' (Uncommunicative Name)
+        Dirty#a contains iterators nested 2 deep (NestedIterators)
+        Dirty#a has the name 'a' (UncommunicativeName)
+        Dirty#a has the variable name 'x' (UncommunicativeName)
 
       """
 
@@ -56,33 +56,3 @@ Feature: Correctly formatted reports
       | option  |
       | -q      |
       | --quiet |
-
-
-  Scenario Outline: -a turns on details in presence of -q
-    When I run reek <options> spec/samples/clean_due_to_masking/*.rb
-    Then the exit status indicates smells
-    And it reports:
-      """
-      spec/samples/clean_due_to_masking/dirty_one.rb -- 7 warnings:
-        (masked) Dirty has no descriptive comment (Irresponsible Module)
-        (masked) Dirty has the variable name '@s' (Uncommunicative Name)
-        (masked) Dirty#a calls @s.title twice (Duplication)
-        (masked) Dirty#a calls puts(@s.title) twice (Duplication)
-        (masked) Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        (masked) Dirty#a has the name 'a' (Uncommunicative Name)
-        (masked) Dirty#a has the variable name 'x' (Uncommunicative Name)
-      spec/samples/clean_due_to_masking/dirty_two.rb -- 7 warnings:
-        (masked) Dirty has no descriptive comment (Irresponsible Module)
-        (masked) Dirty has the variable name '@s' (Uncommunicative Name)
-        (masked) Dirty#a calls @s.title twice (Duplication)
-        (masked) Dirty#a calls puts(@s.title) twice (Duplication)
-        (masked) Dirty#a contains iterators nested 2 deep (Nested Iterators)
-        (masked) Dirty#a has the name 'a' (Uncommunicative Name)
-        (masked) Dirty#a has the variable name 'x' (Uncommunicative Name)
-
-      """
-
-    Examples:
-      | options  |
-      | -q -a    |
-      | -a -q    |

data/features/samples.feature

@@ -11,17 +11,17 @@ Feature: Basic smell detection
     And it reports:
     """
     spec/samples/inline.rb -- 41 warnings:
-      File has no descriptive comment (Irresponsible Module)
-      Inline declares the class variable @@directory (Class Variable)
-      Inline declares the class variable @@rootdir (Class Variable)
+      File has no descriptive comment (IrresponsibleModule)
+      Inline declares the class variable @@directory (ClassVariable)
+      Inline declares the class variable @@rootdir (ClassVariable)
       Inline#self.rootdir calls env.nil? twice (Duplication)
-      Inline#self.rootdir has approx 8 statements (Long Method)
-      Inline::C declares the class variable @@type_map (Class Variable)
-      Inline::C has at least 13 instance variables (Large Class)
-      Inline::C takes parameters [options, src] to 5 methods (Data Clump)
-      Inline::C tests $DEBUG at least 7 times (Simulated Polymorphism)
-      Inline::C tests $TESTING at least 4 times (Simulated Polymorphism)
-      Inline::C tests @@type_map.has_key?(type) at least 3 times (Simulated Polymorphism)
+      Inline#self.rootdir has approx 8 statements (LongMethod)
+      Inline::C declares the class variable @@type_map (ClassVariable)
+      Inline::C has at least 13 instance variables (LargeClass)
+      Inline::C takes parameters [options, src] to 5 methods (DataClump)
+      Inline::C tests $DEBUG at least 7 times (SimulatedPolymorphism)
+      Inline::C tests $TESTING at least 4 times (SimulatedPolymorphism)
+      Inline::C tests @@type_map.has_key?(type) at least 3 times (SimulatedPolymorphism)
       Inline::C#build calls ($? == 0) twice (Duplication)
       Inline::C#build calls Inline.directory 5 times (Duplication)
       Inline::C#build calls io.puts 6 times (Duplication)
@@ -29,29 +29,29 @@ Feature: Basic smell detection
       Inline::C#build calls io.puts("#ifdef __cplusplus") twice (Duplication)
       Inline::C#build calls module_name twice (Duplication)
       Inline::C#build calls warn("Output:\n#{result}") twice (Duplication)
-      Inline::C#build contains iterators nested 2 deep (Nested Iterators)
-      Inline::C#build has approx 60 statements (Long Method)
-      Inline::C#build has the variable name 'n' (Uncommunicative Name)
-      Inline::C#build has the variable name 't' (Uncommunicative Name)
-      Inline::C#c has the name 'c' (Uncommunicative Name)
+      Inline::C#build contains iterators nested 2 deep (NestedIterators)
+      Inline::C#build has approx 60 statements (LongMethod)
+      Inline::C#build has the variable name 'n' (UncommunicativeName)
+      Inline::C#build has the variable name 't' (UncommunicativeName)
+      Inline::C#c has the name 'c' (UncommunicativeName)
       Inline::C#crap_for_windoze calls Config::CONFIG["libdir"] twice (Duplication)
       Inline::C#generate calls result.sub!(/\A\n/, "") twice (Duplication)
       Inline::C#generate calls signature["args"] twice (Duplication)
       Inline::C#generate calls signature["args"].map twice (Duplication)
-      Inline::C#generate has approx 32 statements (Long Method)
+      Inline::C#generate has approx 32 statements (LongMethod)
       Inline::C#initialize calls stack.empty? twice (Duplication)
       Inline::C#load calls so_name twice (Duplication)
-      Inline::C#module_name has the variable name 'm' (Uncommunicative Name)
-      Inline::C#module_name has the variable name 'x' (Uncommunicative Name)
-      Inline::C#parse_signature has approx 15 statements (Long Method)
-      Inline::C#parse_signature has boolean parameter 'raw' (Control Couple)
-      Inline::C#parse_signature has the variable name 'x' (Uncommunicative Name)
-      Inline::C#parse_signature is controlled by argument raw (Control Couple)
-      Inline::C#strip_comments doesn't depend on instance state (Low Cohesion)
-      Inline::C#strip_comments refers to src more than self (Low Cohesion)
+      Inline::C#module_name has the variable name 'm' (UncommunicativeName)
+      Inline::C#module_name has the variable name 'x' (UncommunicativeName)
+      Inline::C#parse_signature has approx 15 statements (LongMethod)
+      Inline::C#parse_signature has boolean parameter 'raw' (ControlCouple)
+      Inline::C#parse_signature has the variable name 'x' (UncommunicativeName)
+      Inline::C#parse_signature is controlled by argument raw (ControlCouple)
+      Inline::C#strip_comments doesn't depend on instance state (LowCohesion)
+      Inline::C#strip_comments refers to src more than self (LowCohesion)
       Module#inline calls Inline.const_get(lang) twice (Duplication)
-      Module#inline has approx 11 statements (Long Method)
-      Module#inline is controlled by argument options (Control Couple)
+      Module#inline has approx 11 statements (LongMethod)
+      Module#inline is controlled by argument options (ControlCouple)
 
     """
 
@@ -61,23 +61,23 @@ Feature: Basic smell detection
     And it reports:
     """
     spec/samples/optparse.rb -- 109 warnings:
-      OptionParser has at least 42 methods (Large Class)
-      OptionParser has the variable name 'f' (Uncommunicative Name)
-      OptionParser has the variable name 'k' (Uncommunicative Name)
-      OptionParser has the variable name 'o' (Uncommunicative Name)
-      OptionParser has the variable name 's' (Uncommunicative Name)
-      OptionParser has the variable name 'v' (Uncommunicative Name)
-      OptionParser tests ((argv.size == 1) and Array.===(argv[0])) at least 3 times (Simulated Polymorphism)
-      OptionParser tests a at least 7 times (Simulated Polymorphism)
-      OptionParser tests default_pattern at least 7 times (Simulated Polymorphism)
-      OptionParser tests not_style at least 3 times (Simulated Polymorphism)
-      OptionParser tests s at least 7 times (Simulated Polymorphism)
-      OptionParser#complete contains iterators nested 2 deep (Nested Iterators)
-      OptionParser#complete has 4 parameters (Long Parameter List)
-      OptionParser#complete has boolean parameter 'icase' (Control Couple)
+      OptionParser has at least 42 methods (LargeClass)
+      OptionParser has the variable name 'f' (UncommunicativeName)
+      OptionParser has the variable name 'k' (UncommunicativeName)
+      OptionParser has the variable name 'o' (UncommunicativeName)
+      OptionParser has the variable name 's' (UncommunicativeName)
+      OptionParser has the variable name 'v' (UncommunicativeName)
+      OptionParser tests ((argv.size == 1) and Array.===(argv[0])) at least 3 times (SimulatedPolymorphism)
+      OptionParser tests a at least 7 times (SimulatedPolymorphism)
+      OptionParser tests default_pattern at least 7 times (SimulatedPolymorphism)
+      OptionParser tests not_style at least 3 times (SimulatedPolymorphism)
+      OptionParser tests s at least 7 times (SimulatedPolymorphism)
+      OptionParser#complete contains iterators nested 2 deep (NestedIterators)
+      OptionParser#complete has 4 parameters (LongParameterList)
+      OptionParser#complete has boolean parameter 'icase' (ControlCouple)
       OptionParser#getopts calls result[opt] = false twice (Duplication)
-      OptionParser#getopts has approx 17 statements (Long Method)
-      OptionParser#load has the variable name 's' (Uncommunicative Name)
+      OptionParser#getopts has approx 17 statements (LongMethod)
+      OptionParser#load has the variable name 's' (UncommunicativeName)
       OptionParser#make_switch calls (long << o = q.downcase) twice (Duplication)
       OptionParser#make_switch calls (sdesc << "-#{q}") twice (Duplication)
       OptionParser#make_switch calls default_style.guess(arg = a) 4 times (Duplication)
@@ -89,20 +89,20 @@ Feature: Basic smell detection
       OptionParser#make_switch calls q.downcase 3 times (Duplication)
       OptionParser#make_switch calls search(:atype, FalseClass) twice (Duplication)
       OptionParser#make_switch calls search(:atype, o) 6 times (Duplication)
-      OptionParser#make_switch contains iterators nested 2 deep (Nested Iterators)
-      OptionParser#make_switch contains iterators nested 3 deep (Nested Iterators)
-      OptionParser#make_switch has approx 68 statements (Long Method)
-      OptionParser#make_switch has the variable name 'a' (Uncommunicative Name)
-      OptionParser#make_switch has the variable name 'c' (Uncommunicative Name)
-      OptionParser#make_switch has the variable name 'n' (Uncommunicative Name)
-      OptionParser#make_switch has the variable name 'o' (Uncommunicative Name)
-      OptionParser#make_switch has the variable name 'q' (Uncommunicative Name)
-      OptionParser#make_switch has the variable name 's' (Uncommunicative Name)
-      OptionParser#make_switch has the variable name 'v' (Uncommunicative Name)
+      OptionParser#make_switch contains iterators nested 2 deep (NestedIterators)
+      OptionParser#make_switch contains iterators nested 3 deep (NestedIterators)
+      OptionParser#make_switch has approx 68 statements (LongMethod)
+      OptionParser#make_switch has the variable name 'a' (UncommunicativeName)
+      OptionParser#make_switch has the variable name 'c' (UncommunicativeName)
+      OptionParser#make_switch has the variable name 'n' (UncommunicativeName)
+      OptionParser#make_switch has the variable name 'o' (UncommunicativeName)
+      OptionParser#make_switch has the variable name 'q' (UncommunicativeName)
+      OptionParser#make_switch has the variable name 's' (UncommunicativeName)
+      OptionParser#make_switch has the variable name 'v' (UncommunicativeName)
       OptionParser#order calls argv[0] twice (Duplication)
-      OptionParser#order refers to argv more than self (Low Cohesion)
+      OptionParser#order refers to argv more than self (LowCohesion)
       OptionParser#parse calls argv[0] twice (Duplication)
-      OptionParser#parse refers to argv more than self (Low Cohesion)
+      OptionParser#parse refers to argv more than self (LowCohesion)
       OptionParser#parse_in_order calls $!.set_option(arg, true) twice (Duplication)
       OptionParser#parse_in_order calls cb.call(val) twice (Duplication)
       OptionParser#parse_in_order calls raise($!.set_option(arg, true)) twice (Duplication)
@@ -110,48 +110,48 @@ Feature: Basic smell detection
       OptionParser#parse_in_order calls setter.call(sw.switch_name, val) twice (Duplication)
       OptionParser#parse_in_order calls sw.block twice (Duplication)
       OptionParser#parse_in_order calls sw.switch_name twice (Duplication)
-      OptionParser#parse_in_order contains iterators nested 2 deep (Nested Iterators)
-      OptionParser#parse_in_order contains iterators nested 3 deep (Nested Iterators)
-      OptionParser#parse_in_order has approx 28 statements (Long Method)
-      OptionParser#parse_in_order is controlled by argument setter (Control Couple)
+      OptionParser#parse_in_order contains iterators nested 2 deep (NestedIterators)
+      OptionParser#parse_in_order contains iterators nested 3 deep (NestedIterators)
+      OptionParser#parse_in_order has approx 28 statements (LongMethod)
+      OptionParser#parse_in_order is controlled by argument setter (ControlCouple)
       OptionParser#permute calls argv[0] twice (Duplication)
-      OptionParser#permute refers to argv more than self (Low Cohesion)
-      OptionParser#search has the variable name 'k' (Uncommunicative Name)
-      OptionParser#summarize has 4 parameters (Long Parameter List)
-      OptionParser#summarize has the variable name 'l' (Uncommunicative Name)
-      OptionParser#ver has the variable name 'v' (Uncommunicative Name)
-      OptionParser::CompletingHash#match contains iterators nested 2 deep (Nested Iterators)
+      OptionParser#permute refers to argv more than self (LowCohesion)
+      OptionParser#search has the variable name 'k' (UncommunicativeName)
+      OptionParser#summarize has 4 parameters (LongParameterList)
+      OptionParser#summarize has the variable name 'l' (UncommunicativeName)
+      OptionParser#ver has the variable name 'v' (UncommunicativeName)
+      OptionParser::CompletingHash#match contains iterators nested 2 deep (NestedIterators)
       OptionParser::Completion#complete calls candidates.size twice (Duplication)
       OptionParser::Completion#complete calls k.id2name twice (Duplication)
-      OptionParser::Completion#complete has approx 22 statements (Long Method)
-      OptionParser::Completion#complete has boolean parameter 'icase' (Control Couple)
-      OptionParser::Completion#complete has the variable name 'k' (Uncommunicative Name)
-      OptionParser::Completion#complete has the variable name 'v' (Uncommunicative Name)
-      OptionParser::Completion#complete refers to candidates more than self (Low Cohesion)
-      OptionParser::List#accept has the parameter name 't' (Uncommunicative Name)
-      OptionParser::List#accept is controlled by argument pat (Control Couple)
-      OptionParser::List#accept refers to pat more than self (Low Cohesion)
-      OptionParser::List#add_banner refers to opt more than self (Low Cohesion)
-      OptionParser::List#complete has 4 parameters (Long Parameter List)
-      OptionParser::List#complete has boolean parameter 'icase' (Control Couple)
-      OptionParser::List#reject has the parameter name 't' (Uncommunicative Name)
-      OptionParser::List#summarize refers to opt more than self (Low Cohesion)
-      OptionParser::List#update has 5 parameters (Long Parameter List)
-      OptionParser::List#update has approx 6 statements (Long Method)
-      OptionParser::List#update has the variable name 'o' (Uncommunicative Name)
-      OptionParser::List#update is controlled by argument lopts (Control Couple)
-      OptionParser::List#update is controlled by argument sopts (Control Couple)
-      OptionParser::ParseError#set_option is controlled by argument eq (Control Couple)
-      OptionParser::Switch#add_banner has the variable name 's' (Uncommunicative Name)
+      OptionParser::Completion#complete has approx 22 statements (LongMethod)
+      OptionParser::Completion#complete has boolean parameter 'icase' (ControlCouple)
+      OptionParser::Completion#complete has the variable name 'k' (UncommunicativeName)
+      OptionParser::Completion#complete has the variable name 'v' (UncommunicativeName)
+      OptionParser::Completion#complete refers to candidates more than self (LowCohesion)
+      OptionParser::List#accept has the parameter name 't' (UncommunicativeName)
+      OptionParser::List#accept is controlled by argument pat (ControlCouple)
+      OptionParser::List#accept refers to pat more than self (LowCohesion)
+      OptionParser::List#add_banner refers to opt more than self (LowCohesion)
+      OptionParser::List#complete has 4 parameters (LongParameterList)
+      OptionParser::List#complete has boolean parameter 'icase' (ControlCouple)
+      OptionParser::List#reject has the parameter name 't' (UncommunicativeName)
+      OptionParser::List#summarize refers to opt more than self (LowCohesion)
+      OptionParser::List#update has 5 parameters (LongParameterList)
+      OptionParser::List#update has approx 6 statements (LongMethod)
+      OptionParser::List#update has the variable name 'o' (UncommunicativeName)
+      OptionParser::List#update is controlled by argument lopts (ControlCouple)
+      OptionParser::List#update is controlled by argument sopts (ControlCouple)
+      OptionParser::ParseError#set_option is controlled by argument eq (ControlCouple)
+      OptionParser::Switch#add_banner has the variable name 's' (UncommunicativeName)
       OptionParser::Switch#conv_arg calls conv twice (Duplication)
-      OptionParser::Switch#initialize has 7 parameters (Long Parameter List)
+      OptionParser::Switch#initialize has 7 parameters (LongParameterList)
       OptionParser::Switch#parse_arg calls pattern twice (Duplication)
       OptionParser::Switch#parse_arg calls s.length twice (Duplication)
-      OptionParser::Switch#parse_arg has approx 11 statements (Long Method)
-      OptionParser::Switch#parse_arg has the variable name 'm' (Uncommunicative Name)
-      OptionParser::Switch#parse_arg has the variable name 's' (Uncommunicative Name)
-      OptionParser::Switch#self.guess has the variable name 't' (Uncommunicative Name)
-      OptionParser::Switch#self.incompatible_argument_styles has the parameter name 't' (Uncommunicative Name)
+      OptionParser::Switch#parse_arg has approx 11 statements (LongMethod)
+      OptionParser::Switch#parse_arg has the variable name 'm' (UncommunicativeName)
+      OptionParser::Switch#parse_arg has the variable name 's' (UncommunicativeName)
+      OptionParser::Switch#self.guess has the variable name 't' (UncommunicativeName)
+      OptionParser::Switch#self.incompatible_argument_styles has the parameter name 't' (UncommunicativeName)
       OptionParser::Switch#summarize calls (indent + l) twice (Duplication)
       OptionParser::Switch#summarize calls arg 4 times (Duplication)
       OptionParser::Switch#summarize calls left.collect twice (Duplication)
@@ -160,16 +160,16 @@ Feature: Basic smell detection
       OptionParser::Switch#summarize calls left.shift twice (Duplication)
       OptionParser::Switch#summarize calls left[-1] 3 times (Duplication)
       OptionParser::Switch#summarize calls s.length 3 times (Duplication)
-      OptionParser::Switch#summarize contains iterators nested 2 deep (Nested Iterators)
-      OptionParser::Switch#summarize has 5 parameters (Long Parameter List)
-      OptionParser::Switch#summarize has approx 25 statements (Long Method)
-      OptionParser::Switch#summarize has the variable name 'l' (Uncommunicative Name)
-      OptionParser::Switch#summarize has the variable name 'r' (Uncommunicative Name)
-      OptionParser::Switch#summarize has the variable name 's' (Uncommunicative Name)
-      OptionParser::Switch::NoArgument#parse is controlled by argument arg (Control Couple)
-      OptionParser::Switch::OptionalArgument#parse is controlled by argument arg (Control Couple)
-      OptionParser::Switch::PlacedArgument#parse has approx 6 statements (Long Method)
-      OptionParser::Switch::RequiredArgument#parse is controlled by argument arg (Control Couple)
+      OptionParser::Switch#summarize contains iterators nested 2 deep (NestedIterators)
+      OptionParser::Switch#summarize has 5 parameters (LongParameterList)
+      OptionParser::Switch#summarize has approx 25 statements (LongMethod)
+      OptionParser::Switch#summarize has the variable name 'l' (UncommunicativeName)
+      OptionParser::Switch#summarize has the variable name 'r' (UncommunicativeName)
+      OptionParser::Switch#summarize has the variable name 's' (UncommunicativeName)
+      OptionParser::Switch::NoArgument#parse is controlled by argument arg (ControlCouple)
+      OptionParser::Switch::OptionalArgument#parse is controlled by argument arg (ControlCouple)
+      OptionParser::Switch::PlacedArgument#parse has approx 6 statements (LongMethod)
+      OptionParser::Switch::RequiredArgument#parse is controlled by argument arg (ControlCouple)
 
     """
 
@@ -179,84 +179,84 @@ Feature: Basic smell detection
     And it reports:
     """
     spec/samples/redcloth.rb -- 97 warnings:
-      RedCloth has at least 44 methods (Large Class)
-      RedCloth has the variable name 'a' (Uncommunicative Name)
-      RedCloth has the variable name 'b' (Uncommunicative Name)
-      RedCloth takes parameters [atts, cite, content, tag] to 3 methods (Data Clump)
-      RedCloth tests atts at least 6 times (Simulated Polymorphism)
-      RedCloth tests codepre.zero? at least 3 times (Simulated Polymorphism)
-      RedCloth tests href at least 3 times (Simulated Polymorphism)
-      RedCloth tests title at least 4 times (Simulated Polymorphism)
-      RedCloth#block_markdown_atx refers to text more than self (Low Cohesion)
-      RedCloth#block_markdown_bq has approx 6 statements (Long Method)
-      RedCloth#block_markdown_rule refers to text more than self (Low Cohesion)
-      RedCloth#block_markdown_setext refers to text more than self (Low Cohesion)
+      RedCloth has at least 44 methods (LargeClass)
+      RedCloth has the variable name 'a' (UncommunicativeName)
+      RedCloth has the variable name 'b' (UncommunicativeName)
+      RedCloth takes parameters [atts, cite, content, tag] to 3 methods (DataClump)
+      RedCloth tests atts at least 6 times (SimulatedPolymorphism)
+      RedCloth tests codepre.zero? at least 3 times (SimulatedPolymorphism)
+      RedCloth tests href at least 3 times (SimulatedPolymorphism)
+      RedCloth tests title at least 4 times (SimulatedPolymorphism)
+      RedCloth#block_markdown_atx refers to text more than self (LowCohesion)
+      RedCloth#block_markdown_bq has approx 6 statements (LongMethod)
+      RedCloth#block_markdown_rule refers to text more than self (LowCohesion)
+      RedCloth#block_markdown_setext refers to text more than self (LowCohesion)
       RedCloth#block_textile_lists calls (line_id - 1) twice (Duplication)
       RedCloth#block_textile_lists calls depth.last 5 times (Duplication)
       RedCloth#block_textile_lists calls depth.last.length twice (Duplication)
       RedCloth#block_textile_lists calls depth[i] twice (Duplication)
       RedCloth#block_textile_lists calls lines[(line_id - 1)] twice (Duplication)
       RedCloth#block_textile_lists calls tl.length 3 times (Duplication)
-      RedCloth#block_textile_lists contains iterators nested 3 deep (Nested Iterators)
-      RedCloth#block_textile_lists has approx 20 statements (Long Method)
-      RedCloth#block_textile_lists has the variable name 'i' (Uncommunicative Name)
-      RedCloth#block_textile_lists has the variable name 'v' (Uncommunicative Name)
-      RedCloth#block_textile_lists refers to depth more than self (Low Cohesion)
-      RedCloth#block_textile_table contains iterators nested 2 deep (Nested Iterators)
-      RedCloth#block_textile_table contains iterators nested 3 deep (Nested Iterators)
-      RedCloth#block_textile_table has approx 18 statements (Long Method)
-      RedCloth#block_textile_table has the variable name 'x' (Uncommunicative Name)
-      RedCloth#blocks contains iterators nested 2 deep (Nested Iterators)
-      RedCloth#blocks has approx 18 statements (Long Method)
-      RedCloth#blocks has boolean parameter 'deep_code' (Control Couple)
-      RedCloth#blocks is controlled by argument deep_code (Control Couple)
-      RedCloth#check_refs is controlled by argument text (Control Couple)
+      RedCloth#block_textile_lists contains iterators nested 3 deep (NestedIterators)
+      RedCloth#block_textile_lists has approx 20 statements (LongMethod)
+      RedCloth#block_textile_lists has the variable name 'i' (UncommunicativeName)
+      RedCloth#block_textile_lists has the variable name 'v' (UncommunicativeName)
+      RedCloth#block_textile_lists refers to depth more than self (LowCohesion)
+      RedCloth#block_textile_table contains iterators nested 2 deep (NestedIterators)
+      RedCloth#block_textile_table contains iterators nested 3 deep (NestedIterators)
+      RedCloth#block_textile_table has approx 18 statements (LongMethod)
+      RedCloth#block_textile_table has the variable name 'x' (UncommunicativeName)
+      RedCloth#blocks contains iterators nested 2 deep (NestedIterators)
+      RedCloth#blocks has approx 18 statements (LongMethod)
+      RedCloth#blocks has boolean parameter 'deep_code' (ControlCouple)
+      RedCloth#blocks is controlled by argument deep_code (ControlCouple)
+      RedCloth#check_refs is controlled by argument text (ControlCouple)
       RedCloth#clean_html calls tags[tag] twice (Duplication)
-      RedCloth#clean_html contains iterators nested 3 deep (Nested Iterators)
-      RedCloth#clean_html doesn't depend on instance state (Low Cohesion)
-      RedCloth#clean_html has approx 14 statements (Long Method)
-      RedCloth#clean_html has the variable name 'q' (Uncommunicative Name)
-      RedCloth#clean_html has the variable name 'q2' (Uncommunicative Name)
-      RedCloth#clean_html refers to raw more than self (Low Cohesion)
-      RedCloth#clean_html refers to tags more than self (Low Cohesion)
-      RedCloth#clean_white_space has approx 7 statements (Long Method)
-      RedCloth#clean_white_space refers to text more than self (Low Cohesion)
-      RedCloth#flush_left doesn't depend on instance state (Low Cohesion)
-      RedCloth#flush_left refers to indt more than self (Low Cohesion)
-      RedCloth#flush_left refers to text more than self (Low Cohesion)
-      RedCloth#footnote_ref refers to text more than self (Low Cohesion)
-      RedCloth#glyphs_textile has approx 10 statements (Long Method)
-      RedCloth#htmlesc doesn't depend on instance state (Low Cohesion)
-      RedCloth#htmlesc refers to str more than self (Low Cohesion)
-      RedCloth#incoming_entities refers to text more than self (Low Cohesion)
-      RedCloth#initialize has the variable name 'r' (Uncommunicative Name)
-      RedCloth#inline contains iterators nested 2 deep (Nested Iterators)
-      RedCloth#inline_markdown_link has approx 6 statements (Long Method)
-      RedCloth#inline_markdown_link has the variable name 'm' (Uncommunicative Name)
-      RedCloth#inline_markdown_reflink has approx 8 statements (Long Method)
-      RedCloth#inline_markdown_reflink has the variable name 'm' (Uncommunicative Name)
-      RedCloth#inline_textile_code has the variable name 'm' (Uncommunicative Name)
-      RedCloth#inline_textile_image has approx 17 statements (Long Method)
-      RedCloth#inline_textile_image has the variable name 'href_a1' (Uncommunicative Name)
-      RedCloth#inline_textile_image has the variable name 'href_a2' (Uncommunicative Name)
-      RedCloth#inline_textile_image has the variable name 'm' (Uncommunicative Name)
-      RedCloth#inline_textile_link has approx 9 statements (Long Method)
-      RedCloth#inline_textile_link has the variable name 'm' (Uncommunicative Name)
-      RedCloth#inline_textile_span contains iterators nested 2 deep (Nested Iterators)
-      RedCloth#inline_textile_span has approx 8 statements (Long Method)
-      RedCloth#inline_textile_span has the variable name 'm' (Uncommunicative Name)
-      RedCloth#lT has the name 'lT' (Uncommunicative Name)
-      RedCloth#no_textile doesn't depend on instance state (Low Cohesion)
-      RedCloth#no_textile refers to text more than self (Low Cohesion)
+      RedCloth#clean_html contains iterators nested 3 deep (NestedIterators)
+      RedCloth#clean_html doesn't depend on instance state (LowCohesion)
+      RedCloth#clean_html has approx 14 statements (LongMethod)
+      RedCloth#clean_html has the variable name 'q' (UncommunicativeName)
+      RedCloth#clean_html has the variable name 'q2' (UncommunicativeName)
+      RedCloth#clean_html refers to raw more than self (LowCohesion)
+      RedCloth#clean_html refers to tags more than self (LowCohesion)
+      RedCloth#clean_white_space has approx 7 statements (LongMethod)
+      RedCloth#clean_white_space refers to text more than self (LowCohesion)
+      RedCloth#flush_left doesn't depend on instance state (LowCohesion)
+      RedCloth#flush_left refers to indt more than self (LowCohesion)
+      RedCloth#flush_left refers to text more than self (LowCohesion)
+      RedCloth#footnote_ref refers to text more than self (LowCohesion)
+      RedCloth#glyphs_textile has approx 10 statements (LongMethod)
+      RedCloth#htmlesc doesn't depend on instance state (LowCohesion)
+      RedCloth#htmlesc refers to str more than self (LowCohesion)
+      RedCloth#incoming_entities refers to text more than self (LowCohesion)
+      RedCloth#initialize has the variable name 'r' (UncommunicativeName)
+      RedCloth#inline contains iterators nested 2 deep (NestedIterators)
+      RedCloth#inline_markdown_link has approx 6 statements (LongMethod)
+      RedCloth#inline_markdown_link has the variable name 'm' (UncommunicativeName)
+      RedCloth#inline_markdown_reflink has approx 8 statements (LongMethod)
+      RedCloth#inline_markdown_reflink has the variable name 'm' (UncommunicativeName)
+      RedCloth#inline_textile_code has the variable name 'm' (UncommunicativeName)
+      RedCloth#inline_textile_image has approx 17 statements (LongMethod)
+      RedCloth#inline_textile_image has the variable name 'href_a1' (UncommunicativeName)
+      RedCloth#inline_textile_image has the variable name 'href_a2' (UncommunicativeName)
+      RedCloth#inline_textile_image has the variable name 'm' (UncommunicativeName)
+      RedCloth#inline_textile_link has approx 9 statements (LongMethod)
+      RedCloth#inline_textile_link has the variable name 'm' (UncommunicativeName)
+      RedCloth#inline_textile_span contains iterators nested 2 deep (NestedIterators)
+      RedCloth#inline_textile_span has approx 8 statements (LongMethod)
+      RedCloth#inline_textile_span has the variable name 'm' (UncommunicativeName)
+      RedCloth#lT has the name 'lT' (UncommunicativeName)
+      RedCloth#no_textile doesn't depend on instance state (LowCohesion)
+      RedCloth#no_textile refers to text more than self (LowCohesion)
       RedCloth#pba calls $1.length twice (Duplication)
-      RedCloth#pba has approx 21 statements (Long Method)
-      RedCloth#pba is controlled by argument text_in (Control Couple)
-      RedCloth#pba refers to style more than self (Low Cohesion)
-      RedCloth#pba refers to text more than self (Low Cohesion)
-      RedCloth#refs_markdown has the variable name 'm' (Uncommunicative Name)
-      RedCloth#refs_textile has the variable name 'm' (Uncommunicative Name)
-      RedCloth#retrieve has the variable name 'i' (Uncommunicative Name)
-      RedCloth#retrieve has the variable name 'r' (Uncommunicative Name)
+      RedCloth#pba has approx 21 statements (LongMethod)
+      RedCloth#pba is controlled by argument text_in (ControlCouple)
+      RedCloth#pba refers to style more than self (LowCohesion)
+      RedCloth#pba refers to text more than self (LowCohesion)
+      RedCloth#refs_markdown has the variable name 'm' (UncommunicativeName)
+      RedCloth#refs_textile has the variable name 'm' (UncommunicativeName)
+      RedCloth#retrieve has the variable name 'i' (UncommunicativeName)
+      RedCloth#retrieve has the variable name 'r' (UncommunicativeName)
       RedCloth#rip_offtags calls ((codepre - used_offtags.length) > 0) twice (Duplication)
       RedCloth#rip_offtags calls (@pre_list.last << line) twice (Duplication)
       RedCloth#rip_offtags calls (codepre - used_offtags.length) twice (Duplication)
@@ -265,16 +265,16 @@ Feature: Basic smell detection
       RedCloth#rip_offtags calls htmlesc(line, :NoQuotes) twice (Duplication)
       RedCloth#rip_offtags calls used_offtags.length twice (Duplication)
       RedCloth#rip_offtags calls used_offtags["notextile"] 3 times (Duplication)
-      RedCloth#rip_offtags has approx 18 statements (Long Method)
-      RedCloth#textile_bq has 4 parameters (Long Parameter List)
-      RedCloth#textile_bq is controlled by argument atts (Control Couple)
-      RedCloth#textile_bq is controlled by argument cite (Control Couple)
-      RedCloth#textile_fn_ has 5 parameters (Long Parameter List)
-      RedCloth#textile_fn_ is controlled by argument atts (Control Couple)
-      RedCloth#textile_p has 4 parameters (Long Parameter List)
-      RedCloth#textile_p is controlled by argument atts (Control Couple)
-      RedCloth#textile_popup_help has the parameter name 'windowH' (Uncommunicative Name)
-      RedCloth#textile_popup_help has the parameter name 'windowW' (Uncommunicative Name)
-      RedCloth#to_html has approx 24 statements (Long Method)
+      RedCloth#rip_offtags has approx 18 statements (LongMethod)
+      RedCloth#textile_bq has 4 parameters (LongParameterList)
+      RedCloth#textile_bq is controlled by argument atts (ControlCouple)
+      RedCloth#textile_bq is controlled by argument cite (ControlCouple)
+      RedCloth#textile_fn_ has 5 parameters (LongParameterList)
+      RedCloth#textile_fn_ is controlled by argument atts (ControlCouple)
+      RedCloth#textile_p has 4 parameters (LongParameterList)
+      RedCloth#textile_p is controlled by argument atts (ControlCouple)
+      RedCloth#textile_popup_help has the parameter name 'windowH' (UncommunicativeName)
+      RedCloth#textile_popup_help has the parameter name 'windowW' (UncommunicativeName)
+      RedCloth#to_html has approx 24 statements (LongMethod)
 
     """