From eaa0cb7924d5ffd0de04492a7198e0d79088aaae Mon Sep 17 00:00:00 2001
From: Xavier Noria <fxn@hashref.com>
Date: Wed, 7 Oct 2015 23:43:58 +0200
Subject: code gardening in transliterate.rb

Saw this while doing a review of a patch:

* Normalize case and punctuation across comments.
* ascii -> ASCII
* Since I was on it, some blank lines that visually
  add some clarity IMO.
---
 activesupport/lib/active_support/inflector/transliterate.rb | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'activesupport/lib')

diff --git a/activesupport/lib/active_support/inflector/transliterate.rb b/activesupport/lib/active_support/inflector/transliterate.rb
index 7472d4386a..103207fb63 100644
--- a/activesupport/lib/active_support/inflector/transliterate.rb
+++ b/activesupport/lib/active_support/inflector/transliterate.rb
@@ -69,10 +69,12 @@ module ActiveSupport
     #   parameterize("Donald E. Knuth") # => "donald-e-knuth"
     #   parameterize("^trés|Jolie-- ")  # => "tres-jolie"
     def parameterize(string, sep = '-')
-      # replace accented chars with their ascii equivalents
+      # Replace accented chars with their ASCII equivalents.
       parameterized_string = transliterate(string)
-      # Turn unwanted chars into the separator
+
+      # Turn unwanted chars into the separator.
       parameterized_string.gsub!(/[^a-z0-9\-_]+/i, sep)
+
       unless sep.nil? || sep.empty?
         if sep == "-".freeze
           re_duplicate_separator        = /-{2,}/
@@ -87,6 +89,7 @@ module ActiveSupport
         # Remove leading/trailing separator.
         parameterized_string.gsub!(re_leading_trailing_separator, ''.freeze)
       end
+
       parameterized_string.downcase!
       parameterized_string
     end
-- 
cgit v1.2.3