1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

ast.c: AST.of checks if a given method object is defined in C

Browse Source 
[Bug #18178]
This commit is contained in:
Yusuke Endoh 2021-09-18 21:28:35 +09:00
parent 1c07c98229
commit ed9d9cee76
2 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ast.c
View File

@ -213,6 +213,9 @@ ast_s_of(rb_execution_context_t *ec, VALUE module, VALUE body, VALUE keep_script
else { else {
iseq = rb_method_iseq(body); iseq = rb_method_iseq(body);
} }
if (!iseq) {
rb_raise(rb_eArgError, "cannot get AST for method that is not defined in Ruby");
}
if (rb_iseq_from_eval_p(iseq)) { if (rb_iseq_from_eval_p(iseq)) {
rb_raise(rb_eArgError, "cannot get AST for method defined in eval"); rb_raise(rb_eArgError, "cannot get AST for method defined in eval");
} }

5
test/ruby/test_ast.rb
View File

@ -231,6 +231,11 @@ class TestAst < Test::Unit::TestCase
assert_raise(ArgumentError) { RubyVM::AbstractSyntaxTree.of(method) } assert_raise(ArgumentError) { RubyVM::AbstractSyntaxTree.of(method) }
end end
def test_of_c_method
c = Class.new { attr_reader :foo }
assert_raise(ArgumentError) { RubyVM::AbstractSyntaxTree.of(c.instance_method(:foo)) }
end
def test_scope_local_variables def test_scope_local_variables
node = RubyVM::AbstractSyntaxTree.parse("_x = 0") node = RubyVM::AbstractSyntaxTree.parse("_x = 0")
lv, _, body = *node.children lv, _, body = *node.children