mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
* marshal.c (id2encidx): register encoding name.
(r_object0): register object before encoding name. [ruby-core:24882] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@25387 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
775174bb76
commit
79a8955f8d
3 changed files with 78 additions and 18 deletions
|
@ -1,3 +1,9 @@
|
|||
Sat Oct 17 22:11:03 2009 Nobuyoshi Nakada <nobu@ruby-lang.org>
|
||||
|
||||
* marshal.c (id2encidx): register encoding name.
|
||||
(r_object0): register object before encoding name.
|
||||
[ruby-core:24882]
|
||||
|
||||
Sat Oct 17 17:56:58 2009 Yuki Sonoda (Yugui) <yugui@yugui.jp>
|
||||
|
||||
* test/rake/test_fileutils.rb (Rake::TestFileUtils#test_sh): uses
|
||||
|
|
47
marshal.c
47
marshal.c
|
@ -1000,11 +1000,21 @@ static const rb_data_type_t load_arg_data = {
|
|||
mark_load_arg, free_load_arg, memsize_load_arg
|
||||
};
|
||||
|
||||
static VALUE r_entry(VALUE v, struct load_arg *arg);
|
||||
#define r_entry(v, arg) r_entry0(v, (arg)->data->num_entries, arg)
|
||||
static VALUE r_entry0(VALUE v, st_index_t num, struct load_arg *arg);
|
||||
static VALUE r_object(struct load_arg *arg);
|
||||
static ID r_symbol(struct load_arg *arg);
|
||||
static VALUE path2class(VALUE path);
|
||||
|
||||
static st_index_t
|
||||
r_prepare(struct load_arg *arg)
|
||||
{
|
||||
st_index_t idx = arg->data->num_entries;
|
||||
|
||||
st_insert(arg->data, (st_data_t)idx, (st_data_t)Qundef);
|
||||
return idx;
|
||||
}
|
||||
|
||||
static int
|
||||
r_byte(struct load_arg *arg)
|
||||
{
|
||||
|
@ -1109,10 +1119,12 @@ r_bytes0(long len, struct load_arg *arg)
|
|||
}
|
||||
|
||||
static int
|
||||
id2encidx(ID id, VALUE val)
|
||||
id2encidx(ID id, VALUE val, struct load_arg *arg)
|
||||
{
|
||||
if (id == rb_id_encoding()) {
|
||||
return rb_enc_find_index(StringValueCStr(val));
|
||||
int idx = rb_enc_find_index(StringValueCStr(val));
|
||||
r_entry(val, arg);
|
||||
return idx;
|
||||
}
|
||||
else if (id == rb_intern("E")) {
|
||||
if (val == Qfalse) return rb_usascii_encindex();
|
||||
|
@ -1145,7 +1157,7 @@ r_symreal(struct load_arg *arg, int ivar)
|
|||
long num = r_long(arg);
|
||||
while (num-- > 0) {
|
||||
id = r_symbol(arg);
|
||||
idx = id2encidx(id, r_object(arg));
|
||||
idx = id2encidx(id, r_object(arg), arg);
|
||||
}
|
||||
}
|
||||
if (idx < 0) idx = rb_usascii_encindex();
|
||||
|
@ -1192,14 +1204,14 @@ r_string(struct load_arg *arg)
|
|||
}
|
||||
|
||||
static VALUE
|
||||
r_entry(VALUE v, struct load_arg *arg)
|
||||
r_entry0(VALUE v, st_index_t num, struct load_arg *arg)
|
||||
{
|
||||
st_data_t real_obj = (VALUE)Qundef;
|
||||
if (st_lookup(arg->compat_tbl, v, &real_obj)) {
|
||||
st_insert(arg->data, arg->data->num_entries, (st_data_t)real_obj);
|
||||
st_insert(arg->data, num, (st_data_t)real_obj);
|
||||
}
|
||||
else {
|
||||
st_insert(arg->data, arg->data->num_entries, (st_data_t)v);
|
||||
st_insert(arg->data, num, (st_data_t)v);
|
||||
}
|
||||
if (arg->taint) {
|
||||
OBJ_TAINT(v);
|
||||
|
@ -1246,7 +1258,7 @@ r_ivar(VALUE obj, struct load_arg *arg)
|
|||
do {
|
||||
ID id = r_symbol(arg);
|
||||
VALUE val = r_object(arg);
|
||||
int idx = id2encidx(id, val);
|
||||
int idx = id2encidx(id, val, arg);
|
||||
if (idx >= 0) {
|
||||
rb_enc_associate_index(obj, idx);
|
||||
}
|
||||
|
@ -1311,7 +1323,7 @@ has_encoding(struct load_arg *arg)
|
|||
long offset = arg->offset;
|
||||
r_long(arg);
|
||||
switch (r_byte(arg)) {
|
||||
case ':':
|
||||
case TYPE_SYMBOL:
|
||||
switch (r_byte(arg)) {
|
||||
case 6:
|
||||
if (r_byte(arg) == 'E') res = TRUE;
|
||||
|
@ -1321,7 +1333,7 @@ has_encoding(struct load_arg *arg)
|
|||
break;
|
||||
}
|
||||
break;
|
||||
case ';':
|
||||
case TYPE_SYMLINK:
|
||||
{
|
||||
ID id = r_symlink(arg);
|
||||
if (id == rb_intern("E") || id == rb_id_encoding())
|
||||
|
@ -1552,14 +1564,12 @@ r_object0(struct load_arg *arg, int *ivp, VALUE extmod)
|
|||
|
||||
case TYPE_STRUCT:
|
||||
{
|
||||
VALUE klass, mem;
|
||||
VALUE values;
|
||||
VALUE mem, values;
|
||||
volatile long i; /* gcc 2.7.2.3 -O2 bug?? */
|
||||
long len;
|
||||
ID slot;
|
||||
|
||||
klass = path2class(r_unique(arg));
|
||||
len = r_long(arg);
|
||||
st_index_t idx = r_prepare(arg);
|
||||
VALUE klass = path2class(r_unique(arg));
|
||||
long len = r_long(arg);
|
||||
|
||||
v = rb_obj_alloc(klass);
|
||||
if (TYPE(v) != T_STRUCT) {
|
||||
|
@ -1571,7 +1581,7 @@ r_object0(struct load_arg *arg, int *ivp, VALUE extmod)
|
|||
rb_class2name(klass));
|
||||
}
|
||||
|
||||
v = r_entry(v, arg);
|
||||
v = r_entry0(v, idx, arg);
|
||||
values = rb_ary_new2(len);
|
||||
for (i=0; i<len; i++) {
|
||||
slot = r_symbol(arg);
|
||||
|
@ -1636,11 +1646,12 @@ r_object0(struct load_arg *arg, int *ivp, VALUE extmod)
|
|||
|
||||
case TYPE_OBJECT:
|
||||
{
|
||||
st_index_t idx = r_prepare(arg);
|
||||
v = obj_alloc_by_path(r_unique(arg), arg);
|
||||
if (TYPE(v) != T_OBJECT) {
|
||||
rb_raise(rb_eArgError, "dump format error");
|
||||
}
|
||||
v = r_entry(v, arg);
|
||||
v = r_entry0(v, idx, arg);
|
||||
r_ivar(v, arg);
|
||||
v = r_leave(v, arg);
|
||||
}
|
||||
|
|
|
@ -200,9 +200,52 @@ class TestMarshal < Test::Unit::TestCase
|
|||
end
|
||||
|
||||
ClassUTF8 = eval("class R\u{e9}sum\u{e9}; self; end")
|
||||
|
||||
iso_8859_1 = Encoding::ISO_8859_1
|
||||
|
||||
structISO8859_1 = Struct.new("r\xe9sum\xe9".force_encoding(iso_8859_1).intern)
|
||||
const_set("R\xe9sum\xe9".force_encoding(iso_8859_1), structISO8859_1)
|
||||
structISO8859_1.name
|
||||
StructISO8859_1 = structISO8859_1
|
||||
classISO8859_1 = Class.new do
|
||||
attr_accessor "r\xe9sum\xe9".force_encoding(iso_8859_1)
|
||||
eval("def initialize(x) @r\xe9sum\xe9 = x; end".force_encoding(iso_8859_1))
|
||||
end
|
||||
const_set("R\xe9sum\xe92".force_encoding(iso_8859_1), classISO8859_1)
|
||||
classISO8859_1.name
|
||||
ClassISO8859_1 = classISO8859_1
|
||||
|
||||
def test_class_nonascii
|
||||
a = ClassUTF8.new
|
||||
assert_instance_of(ClassUTF8, Marshal.load(Marshal.dump(a)), '[ruby-core:24790]')
|
||||
|
||||
bug1932 = '[ruby-core:24882]'
|
||||
|
||||
a = StructISO8859_1.new(10)
|
||||
assert_nothing_raised(bug1932) do
|
||||
assert_equal(a, Marshal.load(Marshal.dump(a)), bug1932)
|
||||
end
|
||||
a.__send__("#{StructISO8859_1.members[0]}=", a)
|
||||
assert_nothing_raised(bug1932) do
|
||||
assert_equal(a, Marshal.load(Marshal.dump(a)), bug1932)
|
||||
end
|
||||
|
||||
a = ClassISO8859_1.new(10)
|
||||
assert_nothing_raised(bug1932) do
|
||||
b = Marshal.load(Marshal.dump(a))
|
||||
assert_equal(ClassISO8859_1, b.class, bug1932)
|
||||
assert_equal(a.instance_variables, b.instance_variables, bug1932)
|
||||
a.instance_variables.each do |i|
|
||||
assert_equal(a.instance_variable_get(i), b.instance_variable_get(i), bug1932)
|
||||
end
|
||||
end
|
||||
a.__send__(a.methods(true).grep(/=\z/)[0], a)
|
||||
assert_nothing_raised(bug1932) do
|
||||
b = Marshal.load(Marshal.dump(a))
|
||||
assert_equal(ClassISO8859_1, b.class, bug1932)
|
||||
assert_equal(a.instance_variables, b.instance_variables, bug1932)
|
||||
assert_equal(b, b.instance_variable_get(a.instance_variables[0]), bug1932)
|
||||
end
|
||||
end
|
||||
|
||||
def test_regexp
|
||||
|
|
Loading…
Reference in a new issue