diff options
author | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-02-25 08:10:23 +0000 |
---|---|---|
committer | naruse <naruse@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2012-02-25 08:10:23 +0000 |
commit | f7f7f93a03f013fb957a0e3e99237f16b5bc1fef (patch) | |
tree | bf659f16ad0a53f078185e0b19d388f3b4ee96c7 | |
parent | 7e2e3dc85d8c02672bc7486630424e7df95ef28b (diff) |
merge revision(s) 34795:
* dir.c (dir_inspect), io.c (rb_io_inspect): keep encoding of path.
[Bug #6072]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_3@34810 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | dir.c | 8 | ||||
-rw-r--r-- | io.c | 22 | ||||
-rw-r--r-- | test/ruby/test_dir_m17n.rb | 13 | ||||
-rw-r--r-- | test/ruby/test_io_m17n.rb | 11 | ||||
-rw-r--r-- | version.h | 2 |
6 files changed, 45 insertions, 16 deletions
@@ -1,3 +1,8 @@ +Sat Feb 25 17:09:53 2012 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * dir.c (dir_inspect), io.c (rb_io_inspect): keep encoding of path. + [Bug #6072] + Sat Feb 25 16:39:13 2012 NARUSE, Yui <naruse@ruby-lang.org> * complex.c (nucomp_marshal_load): raise error on invalid data. @@ -486,8 +486,12 @@ dir_inspect(VALUE dir) TypedData_Get_Struct(dir, struct dir_data, &dir_data_type, dirp); if (!NIL_P(dirp->path)) { - const char *c = rb_obj_classname(dir); - return rb_sprintf("#<%s:%s>", c, RSTRING_PTR(dirp->path)); + VALUE str = rb_str_new_cstr("#<"); + rb_str_append(str, rb_class_name(CLASS_OF(dir))); + rb_str_cat2(str, ":"); + rb_str_append(str, dirp->path); + rb_str_cat2(str, ">"); + return str; } return rb_funcall(dir, rb_intern("to_s"), 0, 0); } @@ -1618,31 +1618,29 @@ static VALUE rb_io_inspect(VALUE obj) { rb_io_t *fptr; - const char *cname; - char fd_desc[4+sizeof(int)*3]; - const char *path; - const char *st = ""; + VALUE result; + static const char closed[] = " (closed)"; fptr = RFILE(rb_io_taint_check(obj))->fptr; if (!fptr) return rb_any_to_s(obj); - cname = rb_obj_classname(obj); + result = rb_str_new_cstr("#<"); + rb_str_append(result, rb_class_name(CLASS_OF(obj))); + rb_str_cat2(result, ":"); if (NIL_P(fptr->pathv)) { if (fptr->fd < 0) { - path = ""; - st = "(closed)"; + rb_str_cat(result, closed+1, strlen(closed)-1); } else { - snprintf(fd_desc, sizeof(fd_desc), "fd %d", fptr->fd); - path = fd_desc; + rb_str_catf(result, "fd %d", fptr->fd); } } else { - path = RSTRING_PTR(fptr->pathv); + rb_str_append(result, fptr->pathv); if (fptr->fd < 0) { - st = " (closed)"; + rb_str_cat(result, closed, strlen(closed)); } } - return rb_sprintf("#<%s:%s%s>", cname, path, st); + return rb_str_cat2(result, ">"); } /* diff --git a/test/ruby/test_dir_m17n.rb b/test/ruby/test_dir_m17n.rb index 6701e0b1a5..33ba9e1557 100644 --- a/test/ruby/test_dir_m17n.rb +++ b/test/ruby/test_dir_m17n.rb @@ -213,5 +213,16 @@ class TestDir_M17N < Test::Unit::TestCase EOS } end -end + def test_inspect_nonascii + bug6072 = '[ruby-dev:45280]' + paths = ["\u{3042}".encode("sjis"), "\u{ff}".encode("iso-8859-1")] + encs = with_tmpdir { + paths.map {|path| + Dir.mkdir(path) + Dir.open(path) {|d| d.inspect.encoding} + } + } + assert_equal(paths.map(&:encoding), encs, bug6072) + end +end diff --git a/test/ruby/test_io_m17n.rb b/test/ruby/test_io_m17n.rb index b7460bf63e..13d792b633 100644 --- a/test/ruby/test_io_m17n.rb +++ b/test/ruby/test_io_m17n.rb @@ -2325,4 +2325,15 @@ EOT end } end if /mswin|mingw/ =~ RUBY_PLATFORM + + def test_inspect_nonascii + bug6072 = '[ruby-dev:45280]' + paths = ["\u{3042}".encode("sjis"), "\u{ff}".encode("iso-8859-1")] + encs = with_tmpdir { + paths.map {|path| + open(path, "wb") {|f| f.inspect.encoding} + } + } + assert_equal(paths.map(&:encoding), encs, bug6072) + end end @@ -1,5 +1,5 @@ #define RUBY_VERSION "1.9.3" -#define RUBY_PATCHLEVEL 144 +#define RUBY_PATCHLEVEL 145 #define RUBY_RELEASE_DATE "2012-02-25" #define RUBY_RELEASE_YEAR 2012 |