Permalink
Browse files

Prevent using String#scrub on Rubinius

Rubinius' has built-in support for String#scrub but it doesn't have yet
support for ASCII-incompatible chars so for now, we should rely on the
old implementation of #tidy_bytes.
  • Loading branch information...
robin850 committed Jul 30, 2014
1 parent 4efb36e commit 29e895770bc341ae8b8fa035f36c24367eeb5e37
Showing with 2 additions and 1 deletion.
  1. +2 −1 activesupport/lib/active_support/multibyte/unicode.rb
@@ -213,7 +213,8 @@ def compose(codepoints)
end
# Ruby >= 2.1 has String#scrub, which is faster than the workaround used for < 2.1.
- if '<3'.respond_to?(:scrub)
+ # Rubinius' String#scrub, however, doesn't support ASCII-incompatible chars.
+ if '<3'.respond_to?(:scrub) && !defined?(Rubinius)
# Replaces all ISO-8859-1 or CP1252 characters by their UTF-8 equivalent
# resulting in a valid UTF-8 string.
#

0 comments on commit 29e8957

Please sign in to comment.