diff options
author | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-01-16 06:30:14 +0000 |
---|---|---|
committer | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-01-16 06:30:14 +0000 |
commit | 3143c6d3819e0bef9c5c4170f9f2e4f5d2adbbb0 (patch) | |
tree | 055676340401d9c51c7f9bf7247892f62d571d17 /io.c | |
parent | 81704536a4f0a0512862c3af8e948aea2ed30e77 (diff) |
merges r30349 from trunk into ruby_1_9_2.
--
* io.c (rb_io_extract_encoding_option): accept Encoding object as
encoding: optional argument. [ruby-dev:42884]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_2@30555 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'io.c')
-rw-r--r-- | io.c | 16 |
1 files changed, 12 insertions, 4 deletions
@@ -4325,9 +4325,12 @@ rb_io_extract_encoding_option(VALUE opt, rb_encoding **enc_p, rb_encoding **enc2 if (v != Qundef) intenc = v; } if ((extenc != Qundef || intenc != Qundef) && !NIL_P(encoding)) { - rb_warn("Ignoring encoding parameter '%s': %s_encoding is used", - StringValueCStr(encoding), - extenc == Qundef ? "internal" : "external"); + if (!NIL_P(ruby_verbose)) { + int idx = rb_to_encoding_index(encoding); + rb_warn("Ignoring encoding parameter '%s': %s_encoding is used", + idx < 0 ? StringValueCStr(encoding) : rb_enc_name(rb_enc_from_index(idx)), + extenc == Qundef ? "internal" : "external"); + } encoding = Qnil; } if (extenc != Qundef && !NIL_P(extenc)) { @@ -4358,7 +4361,12 @@ rb_io_extract_encoding_option(VALUE opt, rb_encoding **enc_p, rb_encoding **enc2 } if (!NIL_P(encoding)) { extracted = 1; - parse_mode_enc(StringValueCStr(encoding), enc_p, enc2_p, fmode_p); + if (!NIL_P(tmp = rb_check_string_type(encoding))) { + parse_mode_enc(StringValueCStr(tmp), enc_p, enc2_p, fmode_p); + } + else { + rb_io_ext_int_to_encs(rb_to_encoding(encoding), NULL, enc_p, enc2_p); + } } else if (extenc != Qundef || intenc != Qundef) { extracted = 1; |