diff options
author | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-11-02 21:50:24 +0000 |
---|---|---|
committer | normal <normal@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2015-11-02 21:50:24 +0000 |
commit | ca86c3abb867aea615e2d33bcadf4bb9bee8576d (patch) | |
tree | bd36587e0c53478e6432ce11f7b3bf7f4aae49e2 /variable.c | |
parent | 5aef9e1d782b858f687f3ce2a640b4d1861b24d5 (diff) | |
download | ruby-ca86c3abb867aea615e2d33bcadf4bb9bee8576d.tar.gz |
variable.c: remove casts for rb_class_ivar_set callers
Casting any arguments of rb_class_ivar_set to st_data_t is wrong
as the function does not take any st_data_t parameters anymore.
There's no functional change, as ID, VALUE, and st_data_t are
all the same type, but this reduces confusion and improves
maintainability for future type changes.
* variable.c (find_class_path): remove cast for rb_class_ivar_set
(rb_ivar_set): ditto
(rb_cvar_set): ditto
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@52427 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'variable.c')
-rw-r--r-- | variable.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/variable.c b/variable.c index bedfb63e8b..000c48ddd5 100644 --- a/variable.c +++ b/variable.c @@ -159,7 +159,7 @@ find_class_path(VALUE klass, ID preferred) if (!RCLASS_IV_TBL(klass)) { RCLASS_IV_TBL(klass) = st_init_numtable(); } - rb_class_ivar_set(klass, (st_data_t)classpath, arg.path); + rb_class_ivar_set(klass, classpath, arg.path); st_delete(RCLASS_IV_TBL(klass), &tmp, 0); return arg.path; @@ -1412,7 +1412,7 @@ rb_ivar_set(VALUE obj, ID id, VALUE val) case T_CLASS: case T_MODULE: if (!RCLASS_IV_TBL(obj)) RCLASS_IV_TBL(obj) = st_init_numtable(); - rb_class_ivar_set(obj, (st_data_t)id, val); + rb_class_ivar_set(obj, id, val); break; default: generic_ivar_set(obj, id, val); @@ -2802,7 +2802,7 @@ rb_cvar_set(VALUE klass, ID id, VALUE val) RCLASS_IV_TBL(target) = st_init_numtable(); } - rb_class_ivar_set(target, (st_data_t)id, (st_data_t)val); + rb_class_ivar_set(target, id, val); } VALUE |