Actually tests to see if a ClassObject has constructors and, if so, generates a __doc__ string.

Barton Cline · Barton Cline · commit ba61f4217e00 · 2010-10-17T18:24:04.000Z
Formerly, the __doc__ string was set to an empty string. Now it remains None and test_enum.py checks for that.
This is also a step toward SomeObject.Overloads[]() handling which, of course, should not be attempted if the ClassObject doesn't have constructors. The current readme.html details the direction I'm heading with that one.
diff --git a/pythonnet/src/runtime/classmanager.cs b/pythonnet/src/runtime/classmanager.cs
@@ -129,10 +129,14 @@ private static ClassBase CreateClass(Type type) {
             // If class has constructors, generate an __doc__ attribute.
 
             ClassObject co = impl as ClassObject;
+            // If this is a ClassObject AND it has constructors, generate a __doc__ attribute.
+            // required that the ClassObject.ctors be changed to internal
             if (co != null) {
-                IntPtr doc = co.GetDocString();
-                Runtime.PyDict_SetItemString(dict, "__doc__", doc);
-                Runtime.Decref(doc);
+                if (co.ctors.Length > 0) {
+                    IntPtr doc = co.GetDocString();
+                    Runtime.PyDict_SetItemString(dict, "__doc__", doc);
+                    Runtime.Decref(doc);
+                }
             }
 
             return impl;
diff --git a/pythonnet/src/runtime/classobject.cs b/pythonnet/src/runtime/classobject.cs
@@ -21,8 +21,8 @@ namespace Python.Runtime {
 
     internal class ClassObject : ClassBase {
 
-        ConstructorBinder binder;
-        ConstructorInfo[] ctors;
+        internal ConstructorBinder binder;
+        internal ConstructorInfo[] ctors;
 
         internal ClassObject(Type tp) : base(tp) {
             ctors = type.GetConstructors();
diff --git a/pythonnet/src/tests/test_enum.py b/pythonnet/src/tests/test_enum.py
@@ -20,7 +20,7 @@ def testEnumStandardAttrs(self):
         self.failUnless(DayOfWeek.__name__ == 'DayOfWeek')
         self.failUnless(DayOfWeek.__module__ == 'System')
         self.failUnless(type(DayOfWeek.__dict__) == types.DictProxyType)
-        self.failUnless(DayOfWeek.__doc__ == '')
+        self.failUnless(DayOfWeek.__doc__ == None)
 
 
     def testEnumGetMember(self):
