(maint) Enable Performance/TimesMap

Constructing an n-element Array where each element is generated by the block is about twice as fast as `n.times.map`[1]. [1] rubocop/rubocop#2583
joshcooper · Oct 8, 2019 · 9900e56 · 9900e56
1 parent c47643a
commit 9900e56
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/.rubocop.yml b/.rubocop.yml
@@ -649,9 +649,6 @@ Style/TrailingUnderscoreVariable:
 Lint/LiteralInInterpolation:
   Enabled: false
 
-Performance/TimesMap:
-  Enabled: false
-
 Layout/InitialIndentation:
   Enabled: false
 

diff --git a/benchmarks/function_loading/benchmarker.rb b/benchmarks/function_loading/benchmarker.rb
@@ -97,7 +97,7 @@ def generate
 
   def dependencies_for(n)
     return [] if n == 0
-    n.times.map do |m|
+    Array.new(n) do |m|
       {'name' => "tester-module#{m}", 'version_requirement' => '1.0.0'}
     end
   end

diff --git a/lib/puppet/pops/parser/heredoc_support.rb b/lib/puppet/pops/parser/heredoc_support.rb
@@ -137,7 +137,7 @@ def heredoc_text(lines, leading, has_margin, remove_break)
       # simply leaves lines that have text in the margin untouched.
       #
       processed_lines = lines.collect {|s| s.gsub(leading_pattern, '') }
-      margin_per_line = processed_lines.length.times.map {|x| lines[x].length - processed_lines[x].length }
+      margin_per_line = Array.new(processed_lines.length) {|x| lines[x].length - processed_lines[x].length }
       lines = processed_lines
     else
       # Array with a 0 per line