Merge pull request #893 from nobu/struct-attributes

Support attributes defined by `rb_struct_define`

Author: nobu

lib/rdoc/parser/c.rb

@@ -372,12 +372,20 @@ def do_classes_and_modules
         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
 

test/rdoc/test_rdoc_parser_c.rb

@@ -311,6 +311,12 @@ def test_do_classes_struct
 
     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 @@ def test_do_classes_struct_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 @@ def test_do_classes_struct_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 @@ def test_do_classes_struct_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