Prefer Enumerable#flat_map over .map.flatten chain

sporkmonger · Feb 3, 2020 · 6db5ebf · 6db5ebf
1 parent cbc62de
commit 6db5ebf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/addressable/template.rb b/lib/addressable/template.rb
@@ -691,12 +691,12 @@ def transform(name, value)
     def ordered_variable_defaults
       @ordered_variable_defaults ||= begin
         expansions, _ = parse_template_pattern(pattern)
-        expansions.map do |capture|
+        expansions.flat_map do |capture|
           _, _, varlist = *capture.match(EXPRESSION)
           varlist.split(',').map do |varspec|
             varspec[VARSPEC, 1]
           end
-        end.flatten
+        end
       end
     end
 

diff --git a/lib/addressable/uri.rb b/lib/addressable/uri.rb
@@ -548,10 +548,10 @@ def self.normalize_component(component, character_class=
         leave_re = if leave_encoded.length > 0
           character_class = "#{character_class}%" unless character_class.include?('%')
 
-          "|%(?!#{leave_encoded.chars.map do |char|
+          "|%(?!#{leave_encoded.chars.flat_map do |char|
             seq = SEQUENCE_ENCODING_TABLE[char]
             [seq.upcase, seq.downcase]
-          end.flatten.join('|')})"
+          end.join('|')})"
         end
 
         character_class = /[^#{character_class}]#{leave_re}/