Bug #8445: IO.open and IO#set_enconding does not support :fallback option

• Status: Assigned
• Assignee: akr (Akira Tanaka)
• Target version: -
• ruby -v: trunk(ruby 2.1.0dev)
• Tags: core, patch

[pjmtdw (Haruhiro Yoshimoto)]

RubyDoc says that IO.open and IO#set_encoding supports optional argument defined in String#encode.
http://ruby-doc.org/core-2.0/IO.html#method-c-new-label-Options
In fact, :invalid, :undef and :replace works as expected.

However, :fallback option does not work neither for IO.open and IO#set_encoding.
Following is the example code which does not work.
f(x) is never called even if hoge.txt contains non convertible character.

File.open("./hoge.txt","r:Shift_JIS:utf-8", :fallback => lambda{|x|f(x)}){|f|
  ...
}
File.open("./hoge.txt"){|f|
  f.set_encoding("Shift_JIS","utf-8",:fallback => lambda{|x|f(x)})
  ...
}

I Think this is because fill_cbuf() in io.c calls rb_econv_convert() from transcode.c directly.
On the other hand, fallback_func is called in transcode_loop(), which is called by str_encode().

Since transcode_loop() also calls rb_econv_convert(), I wrote a small patch which moves some codes from
transcode_loop() to rb_econv_convert() to fix the problem.

The attached file is the patch. Hope this helps.

[naruse (Yui NARUSE)]

• Category set to M17N
• Status changed from Open to Assigned
• Assignee set to akr (Akira Tanaka)
• Target version set to 2.1.0

Is this intentional or just unexpected inconsistency?

[hsbt (Hiroshi SHIBATA)]

• Target version changed from 2.1.0 to 2.2.0

[naruse (Yui NARUSE)]

• Target version deleted (2.2.0)

[ko1 (Koichi Sasada)]

• Description updated (diff)
• Backport deleted (1.9.3: UNKNOWN, 2.0.0: UNKNOWN)

[JesseJohnson (Jesse Johnson)]

I can replicate this in 3.2.2.