[Mono-bugs] [Bug 75091][Nor] Changed - wrong base types

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by dsilva at ccs.neu.edu.

http://bugzilla.ximian.com/show_bug.cgi?id=75091

--- shadow/75091	2005-05-29 17:33:00.000000000 -0400
+++ shadow/75091.tmp.5785	2005-05-30 05:11:41.000000000 -0400
@@ -1,14 +1,14 @@
 Bug#: 75091
 Product: Mono: Class Libraries
 Version: 1.1
-OS: 
+OS: unknown
 OS Details: 
 Status: NEW   
 Resolution: 
-Severity: 
+Severity: Unknown
 Priority: Normal
 Component: Mono.Cecil
 AssignedTo: mono-bugs at ximian.com                            
 ReportedBy: dsilva at ccs.neu.edu               
 QAContact: mono-bugs at ximian.com
 TargetMilestone: ---
@@ -304,6 +304,31 @@
 	IL_0000:  ldarg.0 
 	IL_0001:  call instance void class [Foo]Foo::.ctor()
 	IL_0006:  ret 
     } // end of method Bar::instance default void .ctor () 
 
   } // end of class Bar
+
+------- Additional Comments From dsilva at ccs.neu.edu  2005-05-30 05:11 -------
+Found the problem.  Here's the bugfix.
+  	  	  	
+cecil$ svn diff
+Index: lib/Mono.Cecil.Implem/ReflectionReader.cs
+===================================================================
+--- lib/Mono.Cecil.Implem/ReflectionReader.cs   (revision 45181)
++++ lib/Mono.Cecil.Implem/ReflectionReader.cs   (working copy)
+@@ -229,9 +229,12 @@
+             ReadTypeSpecs ();
+
+             // set base types
+-            for (int i = 1; i < typesTable.Rows.Count; i++) {
++               // BUGFIX 75091: changed the loop to start at 0,
++               //               and changed m_typeDefs[i-1] to
+m_typeDefs[i]
++               //               -- daniel
++            for (int i = 0; i < typesTable.Rows.Count; i++) {
+                 TypeDefRow type = typesTable [i];
+-                TypeDefinition child = m_typeDefs [i - 1];
++                TypeDefinition child = m_typeDefs [i];
+                 child.BaseType = GetTypeDefOrRef (type.Extends);
+             }
+