summaryrefslogtreecommitdiff
path: root/ext/psych/psych_yaml_tree.c
diff options
context:
space:
mode:
authorCharles Oliver Nutter <headius@headius.com>2020-07-16 10:21:18 -0500
committerHiroshi SHIBATA <hsbt@ruby-lang.org>2020-09-25 13:11:32 +0900
commit33641e00cd0a1a2e4c497f70365f06bf5c5f3f6a (patch)
tree165ed8b1c322d24dd01f3e2421b80a126a26eb11 /ext/psych/psych_yaml_tree.c
parent6081ba4a871f857eabdcb1e51b68f11bb10c49af (diff)
Remove private_iv_get
The only remaining use of this function was to get the internal message object from an exception's hidden `mesg` instance variable to allow it to be dumped wiithout converting to a string. As discussed in #103, this exposes internal implementation details of CRuby, and ultimately does not provide any real utility to the user since they can't directly inspect this hidden variable. The test change here is to reflect CRuby behavior that denies equality if the internal message objects do not match, as is the case after the exception has been loaded and now has a simple String value. The impact to users is that exceptions with special hidden message objects will convert those objects to String during marshaling through YAML. I believe this only affects NameError and its descendants, since users can't set this field directly on their own exception types. Fixes #103.
Diffstat (limited to 'ext/psych/psych_yaml_tree.c')
-rw-r--r--ext/psych/psych_yaml_tree.c12
1 files changed, 0 insertions, 12 deletions
diff --git a/ext/psych/psych_yaml_tree.c b/ext/psych/psych_yaml_tree.c
index 7aca9114c9e..225655d1270 100644
--- a/ext/psych/psych_yaml_tree.c
+++ b/ext/psych/psych_yaml_tree.c
@@ -2,23 +2,11 @@
VALUE cPsychVisitorsYamlTree;
-/*
- * call-seq: private_iv_get(target, prop)
- *
- * Get the private instance variable +prop+ from +target+
- */
-static VALUE private_iv_get(VALUE self, VALUE target, VALUE prop)
-{
- return rb_attr_get(target, rb_intern(StringValueCStr(prop)));
-}
-
void Init_psych_yaml_tree(void)
{
VALUE psych = rb_define_module("Psych");
VALUE visitors = rb_define_module_under(psych, "Visitors");
VALUE visitor = rb_define_class_under(visitors, "Visitor", rb_cObject);
cPsychVisitorsYamlTree = rb_define_class_under(visitors, "YAMLTree", visitor);
-
- rb_define_private_method(cPsychVisitorsYamlTree, "private_iv_get", private_iv_get, 2);
}
/* vim: set noet sws=4 sw=4: */