forgot to remove dead code
[mono.git] / mcs / class / Mono.CSharp.Debugger / MonoSymbolTable.cs
index 12c5fe417a28722ba337c27d28dba3875b2349fd..723643f52cab0dc361d43cd804bc2fe04280ff9c 100644 (file)
@@ -18,7 +18,7 @@ namespace Mono.CSharp.Debugger
 {
        public struct OffsetTable
        {
-               public const int  Version = 34;
+               public const int  Version = 35;
                public const long Magic   = 0x45e82623fd7fa614;
 
                public int TotalFileSize;
@@ -447,6 +447,7 @@ namespace Mono.CSharp.Debugger
                public readonly int NumLocals;
                public readonly int NumLineNumbers;
                public readonly int NamespaceID;
+               public readonly bool LocalNamesAmbiguous;
 
                int NameOffset;
                int FullNameOffset;
@@ -457,12 +458,12 @@ namespace Mono.CSharp.Debugger
                int LexicalBlockTableOffset;
                #endregion
 
-               int index;
                int file_offset;
                string name;
                string full_name;
                MethodIndexEntry index_entry;
 
+               public readonly int Index;
                public readonly SourceFileEntry SourceFile;
                public readonly LineNumberEntry[] LineNumbers;
                public readonly int[] ParamTypeIndices;
@@ -492,7 +493,7 @@ namespace Mono.CSharp.Debugger
                internal MethodEntry (MonoSymbolFile file, BinaryReader reader, int index)
                {
                        this.SymbolFile = file;
-                       this.index = index;
+                       this.Index = index;
                        SourceFileIndex = reader.ReadInt32 ();
                        Token = reader.ReadInt32 ();
                        StartRow = reader.ReadInt32 ();
@@ -509,6 +510,7 @@ namespace Mono.CSharp.Debugger
                        NumLexicalBlocks = reader.ReadInt32 ();
                        LexicalBlockTableOffset = reader.ReadInt32 ();
                        NamespaceID = reader.ReadInt32 ();
+                       LocalNamesAmbiguous = reader.ReadInt32 () != 0;
 
                        name = file.ReadString (NameOffset);
                        full_name = file.ReadString (FullNameOffset);
@@ -578,7 +580,7 @@ namespace Mono.CSharp.Debugger
                                      int namespace_id)
                {
                        this.SymbolFile = file;
-                       index = file.GetNextMethodIndex ();
+                       Index = file.GetNextMethodIndex ();
 
                        Token = token;
                        SourceFileIndex = source.Index;
@@ -589,28 +591,36 @@ namespace Mono.CSharp.Debugger
                        LexicalBlocks = blocks;
                        NumLexicalBlocks = LexicalBlocks.Length;
 
-                       LineNumbers = BuildLineNumberTable (lines);
+                       LineNumbers = lines;
                        NumLineNumbers = LineNumbers.Length;
 
                        ParameterInfo[] parameters = method.GetParameters ();
                        if (parameters == null)
                                parameters = new ParameterInfo [0];
-
-                       StringBuilder sb = new StringBuilder ();
-                       sb.Append (method.DeclaringType.FullName);
-                       sb.Append (".");
-                       sb.Append (method.Name);
-                       sb.Append ("(");
-                       for (int i = 0; i < parameters.Length; i++) {
-                               if (i > 0)
-                                       sb.Append (",");
-                               sb.Append (parameters [i].ParameterType.FullName);
+                       
+                       if (parameters.Length == 0)
+                               full_name = method.DeclaringType.FullName + "." + method.Name + "()";
+                       else if (parameters.Length == 1)
+                               full_name = method.DeclaringType.FullName + "." + method.Name + "(" + parameters [0].ParameterType.FullName +  ")";
+                       else if (parameters.Length == 2)
+                               full_name = method.DeclaringType.FullName + "." + method.Name + "(" + parameters [0].ParameterType.FullName + "," + parameters [1].ParameterType.FullName + ")";
+                       else {
+                               StringBuilder sb = new StringBuilder ();
+                               sb.Append (method.DeclaringType.FullName);
+                               sb.Append (".");
+                               sb.Append (method.Name);
+                               sb.Append ("(");
+                               for (int i = 0; i < parameters.Length; i++) {
+                                       if (i > 0)
+                                               sb.Append (",");
+                                       sb.Append (parameters [i].ParameterType.FullName);
+                               }
+                               sb.Append (")");
+                               full_name = sb.ToString ();
                        }
-                       sb.Append (")");
 
                        name = method.Name;
-                       full_name = sb.ToString ();
-
+                       
                        NumParameters = parameters.Length;
                        ParamTypeIndices = new int [NumParameters];
                        for (int i = 0; i < NumParameters; i++)
@@ -619,6 +629,34 @@ namespace Mono.CSharp.Debugger
                        NumLocals = locals.Length;
                        Locals = locals;
 
+                       if (NumLocals <= 32) {
+                               // Most of the time, the O(n^2) factor is actually
+                               // less than the cost of allocating the hash table,
+                               // 32 is a rough number obtained through some testing.
+                               
+                               for (int i = 0; i < NumLocals; i ++) {
+                                       string nm = locals [i].Name;
+                                       
+                                       for (int j = i + 1; j < NumLocals; j ++) {
+                                               if (locals [j].Name == nm) {
+                                                       LocalNamesAmbiguous = true;
+                                                       goto locals_check_done;
+                                               }
+                                       }
+                               }
+                       locals_check_done :
+                               ;
+                       } else {
+                               Hashtable local_names = new Hashtable ();
+                               foreach (LocalVariableEntry local in locals) {
+                                       if (local_names.Contains (local.Name)) {
+                                               LocalNamesAmbiguous = true;
+                                               break;
+                                       }
+                                       local_names.Add (local.Name, local);
+                               }
+                       }
+
                        LocalTypeIndices = new int [NumLocals];
                        for (int i = 0; i < NumLocals; i++)
                                LocalTypeIndices [i] = file.GetNextTypeIndex ();
@@ -626,33 +664,6 @@ namespace Mono.CSharp.Debugger
                        ClassTypeIndex = file.DefineType (method.ReflectedType);
                }
 
-               LineNumberEntry[] BuildLineNumberTable (LineNumberEntry[] line_numbers)
-               {
-                       ArrayList list = new ArrayList ();
-                       int last_offset = -1;
-                       int last_row = -1;
-
-                       for (int i = 0; i < line_numbers.Length; i++) {
-                               LineNumberEntry line = (LineNumberEntry) line_numbers [i];
-
-                               if (line.Offset > last_offset) {
-                                       if (last_row >= 0)
-                                               list.Add (new LineNumberEntry (last_row, last_offset));
-                                       last_row = line.Row;
-                                       last_offset = line.Offset;
-                               } else if (line.Row > last_row) {
-                                       last_row = line.Row;
-                               }
-                       }
-
-                       if (last_row >= 0)
-                               list.Add (new LineNumberEntry (last_row, last_offset));
-
-                       LineNumberEntry[] retval = new LineNumberEntry [list.Count];
-                       list.CopyTo (retval, 0);
-                       return retval;
-               }
-
                internal MethodSourceEntry Write (MonoSymbolFile file, BinaryWriter bw)
                {
                        NameOffset = (int) bw.BaseStream.Position;
@@ -701,8 +712,9 @@ namespace Mono.CSharp.Debugger
                        bw.Write (NumLexicalBlocks);
                        bw.Write (LexicalBlockTableOffset);
                        bw.Write (NamespaceID);
+                       bw.Write (LocalNamesAmbiguous ? 1 : 0);
 
-                       return new MethodSourceEntry (index, file_offset, StartRow, EndRow);
+                       return new MethodSourceEntry (Index, file_offset, StartRow, EndRow);
                }
 
                internal void WriteIndex (BinaryWriter bw)
@@ -713,7 +725,7 @@ namespace Mono.CSharp.Debugger
                public override string ToString ()
                {
                        return String.Format ("[Method {0}:{1}:{2}:{3}:{4} - {7}:{8}:{9}:{10} - {5} - {6}]",
-                                             index, Token, SourceFileIndex, StartRow, EndRow,
+                                             Index, Token, SourceFileIndex, StartRow, EndRow,
                                              SourceFile, FullName, ClassTypeIndex, NumParameters,
                                              NumLocals, NumLineNumbers);
                }