[Mono-list] Bug 72149

Jackson Harper jackson@ximian.com
Fri, 04 Feb 2005 12:29:55 -0800


Hello,

Thanks for the patch. I will test and apply this weekend.

Sol Good mofo,
Jackson


On Thu, 2005-02-03 at 10:11 -0800, Sohail Somani wrote:
> Hi,
> 
> If .locals init (...) has duplicate symbols, mono ilasm croaks because
> it assumes symbols are unique.
> 
> .NET doesnt because it basically seems to ignore symbols and work on
> stack slots. 
> 
> http://bugzilla.ximian.com/show_bug.cgi?id=72149
> 
> I have attached a test case that can be used for regression (and
> demonstrate the problem). The patch:
> 
> Index: codegen/MethodDef.cs
> ===================================================================
> --- codegen/MethodDef.cs	(revision 39963)
> +++ codegen/MethodDef.cs	(working copy)
> @@ -183,7 +183,8 @@
>                                  slot_pos++;
>                                  if (local.Name == null)
>                                          continue;
> -                                named_local_table.Add (local.Name, local);
> +				if(!named_local_table.Contains(local.Name))
> +				  named_local_table.Add (local.Name, local);
>                          }
>  
>                          this.local_list.AddRange (local_list);
> 
> only comes into effect in the case that there are already duplicate
> symbols. In other cases, its business as usual
> 
> Thank you kind sirs
> 
> WHAT UP B-DOG!
> 
> _______________________________________________
> Mono-list maillist  -  Mono-list@lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-list