From 131422ceea4e1970e68907c337976b898a797d8a Mon Sep 17 00:00:00 2001 From: Nobuyoshi Nakada Date: Tue, 28 Jun 2022 19:40:27 +0900 Subject: [PATCH] [ruby/rdoc] Support attributes defined by `rb_struct_define` https://github.com/ruby/rdoc/commit/854b370763 --- lib/rdoc/parser/c.rb | 10 +++++++++- test/rdoc/test_rdoc_parser_c.rb | 14 ++++++++++++++ 2 files changed, 23 insertions(+), 1 deletion(-) diff --git a/lib/rdoc/parser/c.rb b/lib/rdoc/parser/c.rb index 83a216a8ac..a03e4663e4 100644 --- a/lib/rdoc/parser/c.rb +++ b/lib/rdoc/parser/c.rb @@ -372,12 +372,20 @@ class RDoc::Parser::C < RDoc::Parser next end + var_name = $~[:var_name] type = $~[:module] ? :module : :class class_name = $~[:class_name] parent_name = $~[:parent_name] || $~[:path] under = $~[:under] + attributes = $~[:attributes] - handle_class_module($~[:var_name], type, class_name, parent_name, under) + handle_class_module(var_name, type, class_name, parent_name, under) + if attributes and !parent_name # rb_struct_define *not* without_accessor + true_flag = 'Qtrue' + attributes.scan(/"\K\w+(?=")/) do |attr_name| + handle_attr var_name, attr_name, true_flag, true_flag + end + end end end diff --git a/test/rdoc/test_rdoc_parser_c.rb b/test/rdoc/test_rdoc_parser_c.rb index b7e1648cd6..7cd04b52ef 100644 --- a/test/rdoc/test_rdoc_parser_c.rb +++ b/test/rdoc/test_rdoc_parser_c.rb @@ -311,6 +311,12 @@ VALUE cFoo = rb_struct_define( klass = util_get_class content, 'cFoo' assert_equal "this is the Foo class", klass.comment.text + + attributes = klass.attributes + assert_equal 3, attributes.size, -> {attributes} + ["some", "various", "fields"].zip(attributes) do |name, attr| + assert_equal RDoc::Attr.new("", name, "RW", ""), attr + end end def test_do_classes_struct_under @@ -326,6 +332,12 @@ VALUE cFoo = rb_struct_define_under( klass = util_get_class content, 'cFoo' assert_equal 'Kernel::Foo', klass.full_name assert_equal "this is the Foo class under Kernel", klass.comment.text + + attributes = klass.attributes + assert_equal 3, attributes.size, -> {attributes} + ["some", "various", "fields"].zip(attributes) do |name, attr| + assert_equal RDoc::Attr.new("", name, "RW", ""), attr + end end def test_do_classes_struct_without_accessor @@ -340,6 +352,7 @@ VALUE cFoo = rb_struct_define_without_accessor( klass = util_get_class content, 'cFoo' assert_equal "this is the Foo class", klass.comment.text + assert_empty klass.attributes end def test_do_classes_struct_without_accessor_under @@ -355,6 +368,7 @@ VALUE cFoo = rb_struct_define_without_accessor_under( klass = util_get_class content, 'cFoo' assert_equal 'Kernel::Foo', klass.full_name assert_equal "this is the Foo class under Kernel", klass.comment.text + assert_empty klass.attributes end def test_do_classes_class_under