diff options
| author | knu <knu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-01-01 03:26:20 +0000 |
|---|---|---|
| committer | knu <knu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-01-01 03:26:20 +0000 |
| commit | d367b1b9f1cb7447fede62d987dbf6e8586d8f15 (patch) | |
| tree | da3801d01b665476432def1fa91fb4f7ab5a7e62 /lib/shellwords.rb | |
| parent | 069f7ec90a6ef2ecb938fd2bcc1a44f65fbac485 (diff) | |
| download | ruby-d367b1b9f1cb7447fede62d987dbf6e8586d8f15.tar.gz | |
* lib/shellwords.rb (Shellwords#shellescape): Drop the //n flag
that only causes warnings with no real effect. [Bug #5637]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@34166 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'lib/shellwords.rb')
| -rw-r--r-- | lib/shellwords.rb | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/lib/shellwords.rb b/lib/shellwords.rb index 5d6ba7544e..a83d1f1c86 100644 --- a/lib/shellwords.rb +++ b/lib/shellwords.rb @@ -75,15 +75,21 @@ module Shellwords # # ... # } # + # It is caller's responsibility to encode the string in the right + # encoding for the shell environment where this string is used. + # Multibyte characters are treated as multibyte characters, not + # bytes. + # def shellescape(str) # An empty argument will be skipped, so return empty quotes. return "''" if str.empty? str = str.dup - # Process as a single byte sequence because not all shell - # implementations are multibyte aware. - str.gsub!(/([^A-Za-z0-9_\-.,:\/@\n])/n, "\\\\\\1") + # Treat multibyte characters as is. It is caller's responsibility + # to encode the string in the right encoding for the shell + # environment. + str.gsub!(/([^A-Za-z0-9_\-.,:\/@\n])/, "\\\\\\1") # A LF cannot be escaped with a backslash because a backslash + LF # combo is regarded as line continuation and simply ignored. |