2006-08-17 Atsushi Enomoto <atsushi@ximian.com>
[mono.git] / mcs / mbas / class.cs
index e4f0352502de752c369915afbf52fa59c81d8513..535d3d5ac6755228a083badbcba8f08eadfda0b5 100644 (file)
@@ -83,10 +83,7 @@ namespace Mono.MonoBASIC {
                // Holds the events
                ArrayList events;
 
-               // Holds the indexers
-               /* ArrayList indexers; */
-
-               // Holds AddHandlers stements for events
+               // Holds AddHandlers statements for events
                ArrayList handlers;
 
                // The emit context for toplevel objects.
@@ -141,22 +138,23 @@ namespace Mono.MonoBASIC {
                //
                // The indexer name for this class
                //
-               public string IndexerName;
+               public string DefaultPropName;
 
                public TypeContainer (TypeContainer parent, string name, Attributes attrs, Location l)
                        : base (parent, name, attrs, l)
                {
-                       string n;
                        types = new ArrayList ();
+                       
+                       base_class_name = null;
 
+                       /* string n;
+                         
                        if (parent == null)
                                n = "";
                        else
                                n = parent.Name;
 
-                       base_class_name = null;
-                       
-                       //Console.WriteLine ("New class " + name + " inside " + n);
+                       Console.WriteLine ("New class " + name + " inside " + n);*/
                }
 
                public AdditionResult AddConstant (Const constant)
@@ -258,9 +256,6 @@ namespace Mono.MonoBASIC {
                        if (value != null && (!(value is Method)))
                                return AdditionResult.NameExists;
 
-                       if (basename == Basename)
-                               return AdditionResult.EnclosingClash;
-                       
                        if (methods == null)
                                methods = new ArrayList ();
 
@@ -367,6 +362,13 @@ namespace Mono.MonoBASIC {
                        AdditionResult res;
                        string basename = prop.Name;
 
+                       string fullname = Name + "." + basename;
+
+                       Object value = defined_names [fullname];
+
+                       if (value != null && (!(value is Property)))
+                               return AdditionResult.NameExists;
+
                        if ((res = IsValid (basename)) != AdditionResult.Success)
                                return res;
 
@@ -377,7 +379,9 @@ namespace Mono.MonoBASIC {
                                properties.Insert (0, prop);
                        else
                                properties.Add (prop);
-                       DefineName (Name + "." + basename, prop);
+
+                       if (value == null)
+                               DefineName (Name + "." + basename, prop);
 
                        return AdditionResult.Success;
                }
@@ -399,21 +403,6 @@ namespace Mono.MonoBASIC {
                        return AdditionResult.Success;
                }
 
-               /*
-               public AdditionResult AddIndexer (Indexer i)
-               {
-                       if (indexers == null)
-                               indexers = new ArrayList ();
-
-                       if (i.InterfaceType != null)
-                               indexers.Insert (0, i);
-                       else
-                               indexers.Add (i);
-
-                       return AdditionResult.Success;
-               }
-               */              
-
                public AdditionResult AddEventHandler (Statement stmt)
                {
                        if (handlers == null)
@@ -432,17 +421,6 @@ namespace Mono.MonoBASIC {
                }
                public override void ApplyAttributeBuilder (Attribute a, CustomAttributeBuilder cb)
                {
-                       if (a.Type == TypeManager.default_member_type) {
-                               /*
-                               if (Indexers != null) {
-                                       Report.Error (646, a.Location,
-                                                     "Cannot specify the DefaultMember attribute on" +
-                                                     " a type containing an indexer");
-                                       return;
-                               }
-                               */
-                       }
-                       
                        base.ApplyAttributeBuilder (a, cb);
                } 
 
@@ -526,14 +504,6 @@ namespace Mono.MonoBASIC {
                        }
                }
 
-               /* 
-               public ArrayList Indexers {
-                       get {
-                               return indexers;
-                       }
-               }
-               */ 
-
                public ArrayList Delegates {
                        get {
                                return delegates;
@@ -564,7 +534,7 @@ namespace Mono.MonoBASIC {
                public bool EmitFieldInitializers (EmitContext ec)
                {
                        ArrayList fields;
-                       ILGenerator ig = ec.ig;
+                       //ILGenerator ig = ec.ig;
                        Expression instance_expr;
                        
                        if (ec.IsStatic){
@@ -718,7 +688,7 @@ namespace Mono.MonoBASIC {
                                                                "'NotInheritable' class " + TypeManager.MonoBASIC_Name (parent));
                                        
                                if (!AsAccessible (parent, ModFlags))
-                                       Report.Error (30389, Location,
+                                        Report.Error (30389, Location,
                                                      "Inconsistent accessibility: base class `" +
                                                      TypeManager.MonoBASIC_Name (parent) + "' is less " +
                                                      "accessible than class `" +
@@ -741,7 +711,7 @@ namespace Mono.MonoBASIC {
                                }
 
                                if (is_class == false && !t.IsInterface){
-                                       Report.Error (527, "In Struct `" + Name + "', type `"+
+                                       Report.Error (527, Location, "In Struct `" + Name + "', type `"+
                                                      name +"' is not an interface");
                                        error = true;
                                        return null;
@@ -749,10 +719,10 @@ namespace Mono.MonoBASIC {
                        
                                if (t.IsSealed) {
                                        if (t.IsValueType)
-                                               Report.Error (30258, "class `"+ Name +
+                                               Report.Error (30258, Location, "class `"+ Name +
                                                        "': a class can not inherit from a struct/enum");
                                                        
-                                       /*Report.Error (509, "class `"+ Name +
+                                       /*Report.Error (509, Location, "class `"+ Name +
                                                      "': Cannot inherit from sealed class `"+
                                                      bases [i]);*/
                                        error = true;
@@ -761,7 +731,7 @@ namespace Mono.MonoBASIC {
 
                                if (t.IsClass) {
                                        if (parent != null){
-                                               Report.Error (30121, Name + ": A class cannot inherit " +
+                                               Report.Error (30121, Location, Name + ": A class cannot inherit " +
                                                        "more than one class");
                                                error = true;
                                                return null;
@@ -770,7 +740,7 @@ namespace Mono.MonoBASIC {
 
                                for (int x = 0; x < j; x++) {
                                        if (t == ifaces [x]) {
-                                               Report.Error (528, "`" + name + "' is already listed in interface list");
+                                               Report.Error (528,Location, "`" + name + "' is already listed in interface list");
                                                error = true;
                                                return null;
                                        }
@@ -898,8 +868,14 @@ namespace Mono.MonoBASIC {
                            (base_class_type == TypeManager.attribute_type ||
                             base_class_type.IsSubclassOf (TypeManager.attribute_type))) {
                                RootContext.RegisterAttribute (this);
-                       } else
-                               RootContext.RegisterOrder (this); 
+                       }
+                       else
+                       {
+                               if ( this is Interface)
+                                       RootContext.RegisterOrder ((Interface) this); 
+                               else    
+                                       RootContext.RegisterOrder (this); 
+                       }
                                
                        if (Interfaces != null) {
                                foreach (Interface iface in Interfaces)
@@ -1025,49 +1001,6 @@ namespace Mono.MonoBASIC {
                        remove_list.Clear ();
                }
 
-               //
-               // Defines the indexers, and also verifies that the IndexerNameAttribute in the
-               // class is consisten.  Either it is `Item' or it is the name defined by all the
-               // indexers with the `IndexerName' attribute.
-               //
-               // Turns out that the IndexerNameAttribute is applied to each indexer,
-               // but it is never emitted, instead a DefaultName attribute is attached
-               // to the class.
-               //
-               void DefineIndexers ()
-               {
-                       string class_indexer_name = null;
-
-                       /*
-                       foreach (Indexer i in Indexers){
-                               string name;
-                               
-                               i.Define (this);
-
-                               name = i.IndexerName;
-
-                               if (i.InterfaceType != null)
-                                       continue;
-
-                               if (class_indexer_name == null){
-                                       class_indexer_name = name;
-                                       continue;
-                               }
-                               
-                               if (name == class_indexer_name)
-                                       continue;
-                               
-                               Report.Error (
-                                       668, "Two indexers have different names, " +
-                                       " you should use the same name for all your indexers");
-                       }
-                       */
-                       
-                       if (class_indexer_name == null)
-                               class_indexer_name = "Item";
-                       IndexerName = class_indexer_name;
-               }
-
                static void Error_KeywordNotAllowed (Location loc)
                {
                        Report.Error (1530, loc, "Keyword new not allowed for namespace elements");
@@ -1156,13 +1089,6 @@ namespace Mono.MonoBASIC {
                        if (events != null)
                                DefineMembers (events, defined_names);
 
-                       /*
-                       if (indexers != null) {
-                               DefineIndexers ();
-                       } else
-                       */
-                               IndexerName = "Item";
-
                        if (enums != null)
                                DefineMembers (enums, defined_names);
                        
@@ -1329,27 +1255,6 @@ namespace Mono.MonoBASIC {
                                                        members.Add (b);
                                        }
                                }
-
-                               /*
-                               if (indexers != null){
-                                       foreach (Indexer ix in indexers){
-                                               if ((ix.ModFlags & modflags) == 0)
-                                                       continue;
-                                               if ((ix.ModFlags & static_mask) != static_flags)
-                                                       continue;
-                                               
-                                               MethodBuilder b;
-
-                                               b = ix.GetBuilder;
-                                               if (b != null && filter (b, criteria) == true)
-                                                       members.Add (b);
-
-                                               b = ix.SetBuilder;
-                                               if (b != null && filter (b, criteria) == true)
-                                                       members.Add (b);
-                                       }
-                               }
-                               */
                        }
 
                        if ((mt & MemberTypes.Event) != 0) {
@@ -1373,27 +1278,12 @@ namespace Mono.MonoBASIC {
                                                        continue;
                                                if ((p.ModFlags & static_mask) != static_flags)
                                                        continue;
-
+                                               
                                                MemberInfo pb = p.PropertyBuilder;
 
                                                if (pb != null && filter (pb, criteria) == true)
                                                        members.Add (p.PropertyBuilder);
                                        }
-
-                               /* 
-                               if (indexers != null)
-                                       foreach (Indexer ix in indexers) {
-                                               if ((ix.ModFlags & modflags) == 0)
-                                                       continue;
-                                               if ((ix.ModFlags & static_mask) != static_flags)
-                                                       continue;
-
-                                               MemberInfo ib = ix.PropertyBuilder;
-                                               if (ib != null && filter (ib, criteria) == true) {
-                                                       members.Add (ix.PropertyBuilder);
-                                               }
-                                       }
-                               */                                      
                        }
                        
                        if ((mt & MemberTypes.NestedType) != 0) {
@@ -1540,16 +1430,10 @@ namespace Mono.MonoBASIC {
                                foreach (Property p in properties)
                                        p.Emit (this);
 
-                       /*
-                       if (indexers != null){
-                               foreach (Indexer ix in indexers)
-                                       ix.Emit (this);
-                               
-                               CustomAttributeBuilder cb = Interface.EmitDefaultMemberAttr (
-                                       this, IndexerName, ModFlags, Location);
+                       if (this.DefaultPropName != null) {
+                               CustomAttributeBuilder cb = new CustomAttributeBuilder (TypeManager.default_member_ctor, new string [] { DefaultPropName });
                                TypeBuilder.SetCustomAttribute (cb);
                        }
-                       */
                        
                        if (fields != null)
                                foreach (Field f in fields)
@@ -1574,6 +1458,9 @@ namespace Mono.MonoBASIC {
                                foreach (Field f in fields) {
                                        if ((f.ModFlags & Modifiers.PUBLIC) != 0)
                                                continue;
+                                               
+                                       if (f.Name.StartsWith(@"$STATIC$"))
+                                               continue;
 
                                        if (f.status == 0){
                                                Report.Warning (
@@ -1685,7 +1572,7 @@ namespace Mono.MonoBASIC {
                {
                        const int vao = (Modifiers.VIRTUAL | Modifiers.ABSTRACT | Modifiers.OVERRIDE);
                        const int va = (Modifiers.VIRTUAL | Modifiers.ABSTRACT);
-                       const int nv = (Modifiers.SHADOWS | Modifiers.VIRTUAL);
+                       //const int nv = (Modifiers.SHADOWS | Modifiers.VIRTUAL);
                        bool ok = true;
                        string name = MakeName (n);
                        
@@ -1760,7 +1647,7 @@ namespace Mono.MonoBASIC {
                        if ((flags & Modifiers.PRIVATE) != 0){
                                if ((flags & vao) != 0){
                                        Report.Error (
-                                               31408, loc, name +
+                                               30266, loc, name +
                                                ": Members marked as Overridable or Overrides can not be Private");
                                        ok = false;
                                }
@@ -1799,7 +1686,7 @@ namespace Mono.MonoBASIC {
                // and return the new level.
                static AccessLevel CheckAccessLevel (AccessLevel level, int flags)
                {
-                       AccessLevel old_level = level;
+                       //AccessLevel old_level = level;
 
                        if ((flags & Modifiers.INTERNAL) != 0) {
                                if ((flags & Modifiers.PROTECTED) != 0) {
@@ -2141,80 +2028,6 @@ namespace Mono.MonoBASIC {
                        
                        return cc;
                }
-
-               public void LabelParameters (EmitContext ec, Type [] parameters, MethodBase builder)
-               {
-                       LabelParameters (ec, parameters, builder, null);
-               }
-
-               public void LabelParameters (EmitContext ec, Type [] parameters, MethodBase builder, Parameters p_params)
-               {
-                       //
-                       // Define each type attribute (in/out/ref) and
-                       // the argument names.
-                       //
-                       Parameter [] p = p_params == null ? Parameters.FixedParameters : p_params.FixedParameters;
-                       int i = 0;
-                       
-                       MethodBuilder mb = null;
-                       ConstructorBuilder cb = null;
-
-                       if (builder is MethodBuilder)
-                               mb = (MethodBuilder) builder;
-                       else
-                               cb = (ConstructorBuilder) builder;
-
-                       if (p != null){
-                               for (i = 0; i < p.Length; i++) {
-                                       ParameterBuilder pb;
-                                       
-                                       if (mb == null)
-                                               pb = cb.DefineParameter (
-                                                       i + 1, p [i].Attributes, p [i].Name);
-                                       else 
-                                               pb = mb.DefineParameter (
-                                                       i + 1, p [i].Attributes, p [i].Name);
-
-                                       if (p [i].ParameterInitializer != null) {
-                                               if (p [i].ParameterInitializer is MemberAccess) {
-                                                       MemberAccess ma = p [i].ParameterInitializer as MemberAccess;
-
-                                                       Expression const_ex = ma.Resolve(ec);
-                                                       if (const_ex is EnumConstant)
-                                                               pb.SetConstant (((EnumConstant) const_ex).Child.GetValue());
-                                                       else
-                                                               Report.Error(-1,
-                                                                       "Internal error - Non supported argument type in optional parameter");
-                                               }
-                                               else
-                                                       pb.SetConstant (((Constant) p [i].ParameterInitializer).GetValue());
-                                       }
-
-                                       Attributes attr = p [i].OptAttributes;
-                                       if (attr != null)
-                                               Attribute.ApplyAttributes (ec, pb, pb, attr, Location);
-                               }
-                       }
-
-                       if (Parameters.ArrayParameter != null){
-                               ParameterBuilder pb;
-                               Parameter array_param = Parameters.ArrayParameter;
-                               
-                               if (mb == null)
-                                       pb = cb.DefineParameter (
-                                               i + 1, array_param.Attributes,
-                                               array_param.Name);
-                               else
-                                       pb = mb.DefineParameter (
-                                               i + 1, array_param.Attributes,
-                                               array_param.Name);
-                                       
-                               CustomAttributeBuilder a = new CustomAttributeBuilder (
-                                       TypeManager.cons_param_array_attribute, new object [0]);
-                               
-                               pb.SetCustomAttribute (a);
-                       }
-               }
        }
        
        public class Method : MethodCore {
@@ -2395,7 +2208,7 @@ namespace Mono.MonoBASIC {
                                                WarningNotHiding (parent);*/
 
                                        if ((ModFlags & Modifiers.OVERRIDE) != 0){
-                                               Report.Error (30284, Location,
+                                               Report.Error (30398, Location,
                                                              parent.MakeName (Name) +
                                                              " : No suitable methods found to override");
                                        }
@@ -2772,7 +2585,7 @@ namespace Mono.MonoBASIC {
                                ec.IsStatic = false;
                        }
 
-                       LabelParameters (ec, ParameterTypes, ConstructorBuilder);
+                       Parameters.LabelParameters (ec, ConstructorBuilder, Location);
                        
                        //
                        // Classes can have base initializers and instance field initializers.
@@ -2826,11 +2639,11 @@ namespace Mono.MonoBASIC {
                //
                // Protected data.
                //
-               protected MemberBase member;
-               protected int modifiers;
-               protected MethodAttributes flags;
-               protected bool is_method;
-               protected string accessor_name;
+               readonly MemberBase member;
+               readonly int modifiers;
+               MethodAttributes flags;
+               readonly bool is_method;
+               readonly string accessor_name;
                ArrayList conditionals;
 
                MethodBuilder builder = null;
@@ -3034,35 +2847,29 @@ namespace Mono.MonoBASIC {
                        bool IsImplementing = false;
                        Type current_iface_type = iface_type;
 
-                               /*
-                               if (member is Indexer)
-                                       implementing = parent.Pending.IsAbstractIndexer (
-                                       current_iface_type , ReturnType, ParameterTypes);
-                               else
-                               */
-                                       implementing = parent.Pending.IsAbstractMethod (
-                                       current_iface_type, method_name, ReturnType, ParameterTypes);
+                       implementing = parent.Pending.IsAbstractMethod (
+                       current_iface_type, method_name, ReturnType, ParameterTypes);
 
-                               if (implementing != null) {
-                                       if (!implementing_list.Contains (implementing)) {
-                                               implementing_list.Add (implementing);
-                                               implementing_iface.Add(current_iface_type);
-                                       }
-                                       IsImplementing = true;
-                               } else {
-                                       Type[] current_iface_types = current_iface_type.GetInterfaces();
-                                       if (current_iface_types.Length == 0)
-                                               return false;
+                       if (implementing != null) {
+                               if (!implementing_list.Contains (implementing)) {
+                                       implementing_list.Add (implementing);
+                                       implementing_iface.Add(current_iface_type);
+                               }
+                               IsImplementing = true;
+                       } else {
+                               Type[] current_iface_types = current_iface_type.GetInterfaces();
+                               if (current_iface_types.Length == 0)
+                                       return false;
 
-                                       foreach (Type curr_iface_type in current_iface_types) {
-                                               IsImplementing = SearchBasesForAbstractMethods (
-                                                       parent, curr_iface_type, method_name, 
-                                                       ref implementing_list, ref implementing_iface);
+                               foreach (Type curr_iface_type in current_iface_types) {
+                                       IsImplementing = SearchBasesForAbstractMethods (
+                                               parent, curr_iface_type, method_name, 
+                                               ref implementing_list, ref implementing_iface);
 
-                                               if (IsImplementing)
-                                                       break;
-                                       }
+                                       if (IsImplementing)
+                                               break;
                                }
+                       }
 
                        return IsImplementing;
                }
@@ -3090,12 +2897,6 @@ namespace Mono.MonoBASIC {
                        if ((member.ModFlags & Modifiers.OVERRIDE) != 0) {
                                if (parent.Pending == null)
                                        implementing = null;
-                               /*
-                               else if (member is Indexer)
-                                       implementing = parent.Pending.IsAbstractIndexer (
-                                               (Type) parent.TypeBuilder.BaseType, 
-                                               ReturnType, ParameterTypes);
-                               */                                              
                                else
                                        implementing = parent.Pending.IsAbstractMethod (
                                                (Type) parent.TypeBuilder.BaseType, name, 
@@ -3283,7 +3084,7 @@ namespace Mono.MonoBASIC {
                                Attribute.ApplyAttributes (ec, builder, kind, OptAttributes, Location);
 
                        if (member is MethodCore)
-                               ((MethodCore) member).LabelParameters (ec, ParameterTypes, MethodBuilder);
+                               ((MethodCore) member).Parameters.LabelParameters (ec, MethodBuilder, ((MethodCore) member).Location);
 
                        //
                        // abstract or extern methods have no bodies
@@ -3463,18 +3264,10 @@ namespace Mono.MonoBASIC {
                                if (parent.AsAccessible (partype, ModFlags))
                                        continue;
 
-                               /*
-                               if (this is Indexer)
-                                       Report.Error (55, Location,
-                                                     "Inconsistent accessibility: parameter type `" +
-                                                     TypeManager.MonoBASIC_Name (partype) + "' is less " +
-                                                     "accessible than indexer `" + Name + "'");
-                               else
-                               */
-                                       Report.Error (51, Location,
-                                                     "Inconsistent accessibility: parameter type `" +
-                                                     TypeManager.MonoBASIC_Name (partype) + "' is less " +
-                                                     "accessible than method `" + Name + "'");
+                               Report.Error (51, Location,
+                                                   "Inconsistent accessibility: parameter type `" +
+                                                   TypeManager.MonoBASIC_Name (partype) + "' is less " +
+                                                   "accessible than method `" + Name + "'");
                                error = true;
                        }
 
@@ -3517,11 +3310,14 @@ namespace Mono.MonoBASIC {
                                                TypeManager.ExpandAllInterfaces (tbases, ref bases);
 
                                                foreach (Type tbase in bases) {
+                                                       /*
                                                        string bname = tbase.Name;
                                                        if (bname.LastIndexOf(".") != -1)
                                                                bname = bname.Substring(bname.LastIndexOf("."));
+                                                       */
 
-                                                       if (bname == iname)     {
+                                                       //if (bname == iname)   {
+                                                       if (tbase == InterfaceType) {
                                                                iface_found = true;
                                                                break;
                                                        }
@@ -3543,15 +3339,8 @@ namespace Mono.MonoBASIC {
                                                      "Inconsistent accessibility: property type `" +
                                                      TypeManager.MonoBASIC_Name (MemberType) + "' is less " +
                                                      "accessible than property `" + Name + "'");
-                               /*
-                               else if (this is Indexer)
-                                       Report.Error (54, Location,
-                                                     "Inconsistent accessibility: indexer return type `" +
-                                                     TypeManager.MonoBASIC_Name (MemberType) + "' is less " +
-                                                     "accessible than indexer `" + Name + "'");
-                               */                                                    
                                else if (this is Method)
-                                       Report.Error (50, Location,
+                                       Report.Error (30508, Location,
                                                      "Inconsistent accessibility: return type `" +
                                                      TypeManager.MonoBASIC_Name (MemberType) + "' is less " +
                                                      "accessible than method `" + Name + "'");
@@ -3766,8 +3555,14 @@ namespace Mono.MonoBASIC {
                                              "' causes a cycle in the structure layout");
                                return false;
                        }
-                       FieldBuilder = parent.TypeBuilder.DefineField (
-                               Name, t, Modifiers.FieldAttr (ModFlags));
+                       
+                       //Local Static Variable
+                        if (Name.StartsWith(@"$STATIC$"))
+                                FieldBuilder = parent.TypeBuilder.DefineField (
+                                        Name, t, Modifiers.FieldAttr (ModFlags) | FieldAttributes.SpecialName);
+                        else
+                                FieldBuilder = parent.TypeBuilder.DefineField (
+                                        Name, t, Modifiers.FieldAttr (ModFlags));
 
                        TypeManager.RegisterFieldBase (FieldBuilder, this);
                        return true;
@@ -3806,11 +3601,155 @@ namespace Mono.MonoBASIC {
                }
        }
 
+       public class GetMethod {
+               private Accessor Get;
+               public MethodData GetData;
+               public MethodBuilder GetBuilder;
+
+               Property parent_property;
+               Location Location;
+
+               Parameters get_params;
+               
+               public GetMethod (Property parent_property, Accessor get_block, Attributes attrs, Parameters p_get, Location loc)
+               {
+                       this.parent_property = parent_property;
+                       Get = get_block;
+                       get_params = p_get;
+                       Location = loc;
+               }               
+               
+               public MethodBuilder Define (TypeContainer parent)
+               {
+                       Type [] g_parameters=null;
+                       Parameter [] g_parms;
+                       InternalParameters g_ip=null;
+
+                       if (get_params == Parameters.EmptyReadOnlyParameters) {
+                               g_parameters = TypeManager.NoTypes;
+                               g_ip = new InternalParameters (parent, Parameters.EmptyReadOnlyParameters);
+                       } else  {
+                               g_parameters = new Type [get_params.FixedParameters.Length];
+                               for (int i = 0; i < get_params.FixedParameters.Length; i ++) {
+                                       g_parameters[i] = get_params.FixedParameters[i].ParameterType;
+                               }
+                               g_parms = new Parameter [get_params.FixedParameters.Length];
+                               for (int i = 0; i < get_params.FixedParameters.Length; i ++) {
+                                       Parameter tp = get_params.FixedParameters[i];
+                                       g_parms[i] = new Parameter (tp.TypeName, tp.Name,
+                                               Parameter.Modifier.NONE, null);
+                               }
+                               g_ip = new InternalParameters (
+                                       parent, new Parameters (g_parms, null, Location));
+                       }
+
+                       GetData = new MethodData (parent_property, "get", parent_property.MemberType,
+                                                 g_parameters, g_ip, CallingConventions.Standard,
+                                                 Get.OptAttributes, parent_property.ModFlags, parent_property.MethodAttributeFlags, false);
+
+                       if (!GetData.Define (parent))
+                                       return null;
+
+                       GetBuilder = GetData.MethodBuilder;
+                       
+                       return GetBuilder;
+               }
+
+               public void Emit (TypeContainer tc)
+               {
+                       if (GetData != null) 
+                       {
+                               parent_property.Parameters = get_params;
+                               GetData.Emit (tc, Get.Block, Get);
+                       }
+               }
+       }
+
+       public class SetMethod {
+               Accessor Set;
+               MethodData SetData;
+               MethodBuilder SetBuilder;
+
+               Property parent_property;
+               Location Location;
+
+               string set_parameter_name;
+               Parameters set_params;
+               
+               public SetMethod (Property parent_property, Accessor set_block, string set_name, 
+                               Parameters p_set, Location loc)
+               {
+                       this.parent_property = parent_property;
+                       Set = set_block;
+                       set_params = p_set;
+                       set_parameter_name = set_name;
+                       this.Location = loc;
+               }               
+               
+               public MethodBuilder Define (TypeContainer parent)
+               {
+                       Type [] s_parameters=null;
+                       Parameter [] s_parms;
+                       InternalParameters s_ip=null;
+                       
+                       if (set_params == Parameters.EmptyReadOnlyParameters) 
+                       {
+                               s_parameters = new Type [1];
+                               s_parameters [0] = parent_property.MemberType;
+
+                               s_parms = new Parameter [1];
+                               s_parms [0] = new Parameter (parent_property.Type, set_parameter_name, 
+                                       Parameter.Modifier.NONE, null);
+                       } else {
+                               s_parameters = new Type [set_params.FixedParameters.Length];
+                               for (int i = 0; i < set_params.FixedParameters.Length; i ++) {
+                                       s_parameters[i] = set_params.FixedParameters[i].ParameterType;
+                               }
+
+                               s_parms = new Parameter [set_params.FixedParameters.Length];
+                               for (int i = 0; i < set_params.FixedParameters.Length; i ++) {
+                                       Parameter tp = set_params.FixedParameters[i];
+                                       s_parms[i] = new Parameter (tp.TypeName, tp.Name,
+                                               Parameter.Modifier.NONE, null);
+                               }
+                       }
+
+                       s_ip = new InternalParameters (
+                               parent, new Parameters (s_parms, null, Location));
+
+                       SetData = new MethodData (parent_property, "set", TypeManager.void_type,
+                               s_parameters, s_ip, CallingConventions.Standard,
+                               Set.OptAttributes, parent_property.ModFlags, parent_property.MethodAttributeFlags, false);
+
+                       if (!SetData.Define (parent))
+                               return null;
+
+                       SetBuilder = SetData.MethodBuilder;
+                       SetBuilder.DefineParameter (1, ParameterAttributes.None, 
+                               set_parameter_name); 
+
+                       return SetBuilder;
+               }
+
+               public void Emit (TypeContainer tc)
+               {
+                       if (SetData != null) 
+                       {
+                               parent_property.Parameters = set_params;
+                               SetData.Emit (tc, Set.Block, Set);
+                       }
+               }
+       }
+
        public class Property : MethodCore {
-               public Accessor Get, Set;
+               GetMethod GetMethod;
+               SetMethod SetMethod;
+               
+               public Accessor Set;
                public PropertyBuilder PropertyBuilder;
-               public MethodBuilder GetBuilder, SetBuilder;
-               public MethodData GetData, SetData;
+               public MethodBuilder GetBuilder;
+               public MethodBuilder SetBuilder;
+               public MethodData SetData;
 
                protected EmitContext ec;
 
@@ -3876,7 +3815,7 @@ namespace Mono.MonoBASIC {
                        if ((ModFlags & Modifiers.READONLY) != 0)
                                retval = MemberType;
 
-                       string report_name;
+                       //string report_name;
                        MethodSignature base_ms;
                        
                        /*
@@ -3889,7 +3828,7 @@ namespace Mono.MonoBASIC {
                                base_name = TypeManager.IndexerPropertyName (container.TypeBuilder.BaseType);
                                base_ms = new MethodSignature (base_name, retval, ParameterTypes);
                        } else */ {
-                               report_name = Name;
+                               //report_name = Name;
                                ms = base_ms = new MethodSignature (Name, retval, ParameterTypes);
                        }
 
@@ -3916,7 +3855,6 @@ namespace Mono.MonoBASIC {
                                MethodSignature.inheritable_property_signature_filter,
                                base_ms);
 
-
                        if (mi_instance.Count > 0)
                                mi = mi_instance;
                        else if (mi_static.Count > 0)
@@ -3926,7 +3864,6 @@ namespace Mono.MonoBASIC {
 
                        if (mi != null && mi.Count > 0) 
                                parent_member = (PropertyInfo) mi [0];
-
                        if (parent_member is PropertyInfo) {
                                PropertyInfo parent_property = (PropertyInfo)parent_member;
 
@@ -3963,20 +3900,6 @@ namespace Mono.MonoBASIC {
                                /*if ((ModFlags & Modifiers.NEW) != 0)
                                        WarningNotHiding (container);
                                */
-                               if ((ModFlags & Modifiers.OVERRIDE) != 0) {
-
-                                       /*
-                                       if (this is Indexer)
-                                               Report.Error (115, Location,
-                                                       container.MakeName (Name) +
-                                                       " no suitable indexers found to override");
-                                       else
-                                       */
-                                               Report.Error (115, Location,
-                                                       container.MakeName (Name) +
-                                                       " no suitable properties found to override");
-                                       return false;
-                               }
 
                                if ((ModFlags & ( Modifiers.NEW | Modifiers.SHADOWS | Modifiers.OVERRIDE )) == 0) {
                                        if ((ModFlags & Modifiers.NONVIRTUAL) != 0)     {
@@ -4016,9 +3939,9 @@ namespace Mono.MonoBASIC {
                        Modifiers.WRITEONLY |
                        Modifiers.SHADOWS;
 
-               string set_parameter_name;
-               Parameters get_params;
-               Parameters set_params;
+               //string set_parameter_name;
+               //Parameters get_params;
+               //Parameters set_params;
                
                public Property (Expression type, string name, int mod_flags,
                                Accessor get_block, Accessor set_block,
@@ -4027,12 +3950,11 @@ namespace Mono.MonoBASIC {
                        : base (type, mod_flags, AllowedModifiers, name, attrs, p_set, loc)
                
                {
-                       Get = get_block;
-                       get_params = p_get;
+                       if (get_block != null)
+                               GetMethod = new GetMethod (this, get_block, attrs, p_get, loc);
 
-                       Set = set_block;
-                       set_params = p_set;
-                       set_parameter_name = set_name;
+                       if (set_block != null)
+                               SetMethod = new SetMethod (this, set_block, set_name, p_set, loc);
                        
                        Implements = impl_what;
                }               
@@ -4047,15 +3969,21 @@ namespace Mono.MonoBASIC {
 
                public override bool Define (TypeContainer parent)
                {
-                       Type [] g_parameters=null, s_parameters=null;
-                       Parameter [] g_parms, s_parms;
-                       InternalParameters g_ip=null, s_ip=null;
-
+                       /*Type [] s_parameters=null;
+                       Parameter [] s_parms;
+                       InternalParameters s_ip=null;*/
+                       
                        if ((parent is Struct) && ((ModFlags & Modifiers.PROTECTED) != 0))
                                Report.Error (30435, Location,
                                        "'Property' inside a 'Structure' can not be declared as " +
                                        "'Protected' or 'Protected Friend'");
 
+                       if (((ModFlags & Modifiers.DEFAULT) != 0) && (Parameters == null || Parameters.CountStandardParams () == 0)) {
+                               Report.Error (31048, Location, "Properties with no required " +
+                                       "parameters cannot be declared 'Default'");
+                               return false;
+                       }
+
                        if (!DoDefine (parent))
                                return false;
 
@@ -4064,43 +3992,15 @@ namespace Mono.MonoBASIC {
 
                        flags |= MethodAttributes.HideBySig | MethodAttributes.SpecialName;
 
-                       if (Get == null) {
+                       if (GetMethod == null) {
                                if ((ModFlags & Modifiers.WRITEONLY) == 0)
                                        Report.Error (
                                                30124, Location,
                                                "Property without 'Get' accessor must have a 'WriteOnly' modifier");
-                       }
-                       else {
-                               if (get_params == Parameters.EmptyReadOnlyParameters) {
-                                       g_parameters = TypeManager.NoTypes;
-                                       g_ip = new InternalParameters (
-                                                       parent, Parameters.EmptyReadOnlyParameters);
-                               } else  {
-                                       g_parameters = new Type [get_params.FixedParameters.Length];
-                                       for (int i = 0; i < get_params.FixedParameters.Length; i ++) {
-                                               g_parameters[i] = get_params.FixedParameters[i].ParameterType;
-                                       }
-                                       g_parms = new Parameter [get_params.FixedParameters.Length];
-                                       for (int i = 0; i < get_params.FixedParameters.Length; i ++) {
-                                               Parameter tp = get_params.FixedParameters[i];
-                                               g_parms[i] = new Parameter (tp.TypeName, tp.Name,
-                                                       Parameter.Modifier.NONE, null);
-                                       }
-                                       g_ip = new InternalParameters (
-                                               parent, new Parameters (g_parms, null, Location));
-                               }
-
-                               GetData = new MethodData (this, "get", MemberType,
-                                                         g_parameters, g_ip, CallingConventions.Standard,
-                                                         Get.OptAttributes, ModFlags, flags, false);
-
-                               if (!GetData.Define (parent))
-                                       return false;
-
-                               GetBuilder = GetData.MethodBuilder;
-                       }
+                       } else
+                               GetBuilder = GetMethod.Define (parent);
 
-                       if (Set == null) {
+                       if (SetMethod== null) {
                                if ((ModFlags & Modifiers.READONLY) == 0)
                                        Report.Error (
                                                30124, Location,
@@ -4108,43 +4008,7 @@ namespace Mono.MonoBASIC {
                                                
                        }
                        else 
-                       {
-                               if (set_params == Parameters.EmptyReadOnlyParameters) 
-                               {
-                                       s_parameters = new Type [1];
-                                       s_parameters [0] = MemberType;
-
-                                       s_parms = new Parameter [1];
-                                       s_parms [0] = new Parameter (Type, set_parameter_name, 
-                                               Parameter.Modifier.NONE, null);
-                               } else {
-                                       s_parameters = new Type [set_params.FixedParameters.Length];
-                                       for (int i = 0; i < set_params.FixedParameters.Length; i ++) {
-                                               s_parameters[i] = set_params.FixedParameters[i].ParameterType;
-                                       }
-
-                                       s_parms = new Parameter [set_params.FixedParameters.Length];
-                                       for (int i = 0; i < set_params.FixedParameters.Length; i ++) {
-                                               Parameter tp = set_params.FixedParameters[i];
-                                               s_parms[i] = new Parameter (tp.TypeName, tp.Name,
-                                                       Parameter.Modifier.NONE, null);
-                                       }
-                               }
-
-                               s_ip = new InternalParameters (
-                                       parent, new Parameters (s_parms, null, Location));
-
-                               SetData = new MethodData (this, "set", TypeManager.void_type,
-                                       s_parameters, s_ip, CallingConventions.Standard,
-                                       Set.OptAttributes, ModFlags, flags, false);
-
-                               if (!SetData.Define (parent))
-                                       return false;
-
-                               SetBuilder = SetData.MethodBuilder;
-                               SetBuilder.DefineParameter (1, ParameterAttributes.None, 
-                                       set_parameter_name); 
-                       }
+                               SetBuilder = SetMethod.Define (parent);
 
                        // FIXME - PropertyAttributes.HasDefault ?
                        
@@ -4185,19 +4049,23 @@ namespace Mono.MonoBASIC {
                        if (PropertyBuilder != null && OptAttributes != null)
                                OptAttributes.Emit (ec, this);
                        
-                       if (GetData != null) 
+                       if (GetMethod != null) 
                        {
-                               Parameters = get_params;
-                               GetData.Emit (tc, Get.Block, Get);
+                               GetMethod.Emit (tc);
                        }
 
-                       if (SetData != null) 
+                       if (SetMethod != null) 
                        {
-                               Parameters = set_params;
-                               SetData.Emit (tc, Set.Block, Set);
+                               SetMethod.Emit (tc);
                        }
-                               
                }
+
+               public MethodAttributes MethodAttributeFlags {
+                       get {
+                               return flags;
+                       }
+               }
+
        }
 
        /// </summary>
@@ -4334,6 +4202,98 @@ namespace Mono.MonoBASIC {
                        }
                }
        }
+
+       
+       public class AddDelegateMethod {
+               MethodBuilder AddBuilder;
+               MethodData AddData;
+               Event parent_event;
+               Location Location;
+               
+               public AddDelegateMethod (Event parent_event, Location loc)
+               {
+                       this.parent_event = parent_event;
+                       Location = loc;
+               }
+
+               public MethodBuilder Define (TypeContainer parent)
+               {
+                       Type [] parameter_types = new Type [1];
+                       parameter_types [0] = parent_event.MemberType;
+
+                       Parameter [] parms = new Parameter [1];
+                       parms [0] = new Parameter (parent_event.Type, /* was "value" */ parent_event.Name, Parameter.Modifier.NONE, null);
+                       InternalParameters ip = new InternalParameters (
+                               parent, new Parameters (parms, null, Location)); 
+
+                       AddData = new MethodData (parent_event, "add", TypeManager.void_type,
+                                                 parameter_types, ip, CallingConventions.Standard, null,
+                                                 parent_event.ModFlags, parent_event.MethodAttributeFlags, false);
+
+                       if (!AddData.Define (parent))
+                               return null;
+
+                       AddBuilder = AddData.MethodBuilder;
+                       AddBuilder.DefineParameter (1, ParameterAttributes.None, /* was "value" */ parent_event.Name);
+
+
+                       return AddBuilder;
+               }
+
+
+               public void Emit (TypeContainer tc)
+               {
+                       ILGenerator ig = AddData.MethodBuilder.GetILGenerator ();
+                       EmitContext ec = new EmitContext (tc, Location, ig, TypeManager.void_type, parent_event.ModFlags);
+                       parent_event.EmitDefaultMethod (ec, true);
+               }
+       }
+
+       public class RemoveDelegateMethod {
+               MethodBuilder RemoveBuilder;
+               MethodData RemoveData;
+               Event parent_event;
+               Location Location;
+               
+               public RemoveDelegateMethod (Event parent_event, Location loc)
+               {
+                       this.parent_event = parent_event;
+                       Location = loc;
+               }
+
+               public MethodBuilder Define (TypeContainer parent)
+               {
+                       //EventAttributes e_attr = EventAttributes.RTSpecialName | EventAttributes.SpecialName;
+
+                       Type [] parameter_types = new Type [1];
+                       parameter_types [0] = parent_event.MemberType;
+
+                       Parameter [] parms = new Parameter [1];
+                       parms [0] = new Parameter (parent_event.Type, /* was "value" */ parent_event.Name, Parameter.Modifier.NONE, null);
+                       InternalParameters ip = new InternalParameters (
+                               parent, new Parameters (parms, null, Location)); 
+
+
+                       RemoveData = new MethodData (parent_event, "remove", TypeManager.void_type,
+                                                    parameter_types, ip, CallingConventions.Standard, null,
+                                                    parent_event.ModFlags, parent_event.MethodAttributeFlags, false);
+
+                       if (!RemoveData.Define (parent))
+                               return null;
+
+                       RemoveBuilder = RemoveData.MethodBuilder;
+                       RemoveBuilder.DefineParameter (1, ParameterAttributes.None, /* was "value" */ parent_event.Name);
+
+                       return RemoveBuilder;
+               }
+
+               public void Emit (TypeContainer tc)
+               {
+                       ILGenerator ig = RemoveData.MethodBuilder.GetILGenerator ();
+                       EmitContext ec = new EmitContext (tc, Location, ig, TypeManager.void_type, parent_event.ModFlags);
+                       parent_event.EmitDefaultMethod (ec, false);
+               }
+       }
        
        public class Event : FieldBase {
                const int AllowedModifiers =
@@ -4349,28 +4309,27 @@ namespace Mono.MonoBASIC {
                        Modifiers.UNSAFE |
                        Modifiers.ABSTRACT;
 
-               public readonly Accessor  Add;
-               public readonly Accessor  Remove;
+               public readonly AddDelegateMethod  Add;
+               public readonly RemoveDelegateMethod  Remove;
                public MyEventBuilder     EventBuilder;
 
                MethodBuilder AddBuilder, RemoveBuilder;
-               MethodData AddData, RemoveData;
                
-               public Event (Expression type, string name, Object init, int mod, Accessor add,
-                             Accessor remove, Attributes attrs, Location loc)
-                       : base (type, mod, AllowedModifiers, name, init, attrs, loc)
+               public Event (Expression type, string name, Object init, int mod, Attributes attrs, Location loc)
+                               : base (type, mod, AllowedModifiers, name, init, attrs, loc)
                {
-                       Add = add;
-                       Remove = remove;
+                       Add = new AddDelegateMethod (this, loc);
+                       Remove = new RemoveDelegateMethod (this, loc);
+
                        Implements = null;
                }
 
-               public Event (Expression type, string name, Object init, int mod, Accessor add,
-                       Accessor remove, Attributes attrs, ArrayList impl_what, Location loc)
+               public Event (Expression type, string name, Object init, int mod,  Attributes attrs, ArrayList impl_what, Location loc)
                        : base (type, mod, AllowedModifiers, name, init, attrs, loc)
                {
-                       Add = add;
-                       Remove = remove;
+                       Add = new AddDelegateMethod (this, loc);
+                       Remove = new RemoveDelegateMethod (this, loc);
+
                        Implements = impl_what;
                }
 
@@ -4403,58 +4362,34 @@ namespace Mono.MonoBASIC {
 
                        Parameter [] parms = new Parameter [1];
                        parms [0] = new Parameter (Type, /* was "value" */ this.Name, Parameter.Modifier.NONE, null);
-                       InternalParameters ip = new InternalParameters (
+                       /*InternalParameters ip = new InternalParameters (
                                parent, new Parameters (parms, null, Location)); 
+                       */
 
                        if (!CheckBase (parent))
                                return false;
 
-                       //
-                       // Now define the accessors
-                       //
-                       AddData = new MethodData (this, "add", TypeManager.void_type,
-                                                 parameter_types, ip, CallingConventions.Standard,
-                                                 (Add != null) ? Add.OptAttributes : null,
-                                                 ModFlags, flags, false);
-
-                       if (!AddData.Define (parent))
-                               return false;
-
-                       AddBuilder = AddData.MethodBuilder;
-                       AddBuilder.DefineParameter (1, ParameterAttributes.None, /* was "value" */ this.Name);
-
-                       RemoveData = new MethodData (this, "remove", TypeManager.void_type,
-                                                    parameter_types, ip, CallingConventions.Standard,
-                                                    (Remove != null) ? Remove.OptAttributes : null,
-                                                    ModFlags, flags, false);
-
-                       if (!RemoveData.Define (parent))
-                               return false;
-
-                       RemoveBuilder = RemoveData.MethodBuilder;
-                       RemoveBuilder.DefineParameter (1, ParameterAttributes.None, /* was "value" */ this.Name);
+                       AddBuilder = Add.Define (parent);
+                       RemoveBuilder = Remove.Define (parent);
+                       
 
                        if (!IsExplicitImpl){
                                EventBuilder = new MyEventBuilder (
                                        parent.TypeBuilder, Name, e_attr, MemberType);
                                        
-                               if (Add == null && Remove == null) {
-                                       FieldBuilder = parent.TypeBuilder.DefineField (
-                                               Name, MemberType,
+                               FieldBuilder = parent.TypeBuilder.DefineField (Name, MemberType,
                                                FieldAttributes.FamANDAssem | ((ModFlags & Modifiers.STATIC) != 0 ? FieldAttributes.Static : 0));
-                                       TypeManager.RegisterPrivateFieldOfEvent (
-                                               (EventInfo) EventBuilder, FieldBuilder);
-                                       TypeManager.RegisterFieldBase (FieldBuilder, this);
-                               }
+                               TypeManager.RegisterPrivateFieldOfEvent ((EventInfo) EventBuilder, FieldBuilder);
+                               TypeManager.RegisterFieldBase (FieldBuilder, this);
                        
                                EventBuilder.SetAddOnMethod (AddBuilder);
                                EventBuilder.SetRemoveOnMethod (RemoveBuilder);
 
                                if (!TypeManager.RegisterEvent (EventBuilder, AddBuilder, RemoveBuilder)) {
                                        Report.Error (111, Location,
-                                                     "Class `" + parent.Name +
-                                                     "' already contains a definition for the event `" +
-                                                     Name + "'");
+                                                   "Class `" + parent.Name +
+                                                "' already contains a definition for the event `" +
+                                                Name + "'");
                                        return false;
                                }
                        }
@@ -4462,7 +4397,7 @@ namespace Mono.MonoBASIC {
                        return true;
                }
 
-               void EmitDefaultMethod (EmitContext ec, bool is_add)
+               public void EmitDefaultMethod (EmitContext ec, bool is_add)
                {
                        ILGenerator ig = ec.ig;
                        MethodInfo method = null;
@@ -4499,213 +4434,17 @@ namespace Mono.MonoBASIC {
                        if (OptAttributes != null)
                                OptAttributes.Emit (ec, this);
 
-                       if (Add != null)
-                               AddData.Emit (tc, Add.Block, Add);
-                       else {
-                               ILGenerator ig = AddData.MethodBuilder.GetILGenerator ();
-                               ec = new EmitContext (tc, Location, ig, TypeManager.void_type, ModFlags);
-                               EmitDefaultMethod (ec, true);
-                       }
-
-                       if (Remove != null)
-                               RemoveData.Emit (tc, Remove.Block, Remove);
-                       else {
-                               ILGenerator ig = RemoveData.MethodBuilder.GetILGenerator ();
-                               ec = new EmitContext (tc, Location, ig, TypeManager.void_type, ModFlags);
-                               EmitDefaultMethod (ec, false);
-                       }
-               }
-               
-       }
-
-       //
-       // FIXME: This does not handle:
-       //
-       //   int INTERFACENAME [ args ]
-       //   Does not 
-       //
-       // Only:
-       // 
-       // int this [ args ]
-
-
-       /*
-       public class Indexer : PropertyBase {
-
-               const int AllowedModifiers =
-                       Modifiers.NEW |
-                       Modifiers.PUBLIC |
-                       Modifiers.PROTECTED |
-                       Modifiers.INTERNAL |
-                       Modifiers.PRIVATE |
-                       Modifiers.VIRTUAL |
-                       Modifiers.SEALED |
-                       Modifiers.OVERRIDE |
-                       Modifiers.UNSAFE |
-                       Modifiers.EXTERN |
-                       Modifiers.ABSTRACT;
-
-               public string IndexerName;
-               public string InterfaceIndexerName;
-
-               //
-               // Are we implementing an interface ?
-               //
-               bool IsImplementing = false;
-               
-               public Indexer (Expression type, string int_type, int flags, Parameters parameters,
-                               Accessor get_block, Accessor set_block, Attributes attrs, Location loc)
-                       : base (type, "", flags, AllowedModifiers, parameters, get_block, set_block,
-                               attrs, loc)
-               {
-                       ExplicitInterfaceName = int_type;
+                       Add.Emit (tc);
+                       Remove.Emit (tc);
                }
 
-               public override bool Define (TypeContainer parent)
-               {
-                       PropertyAttributes prop_attr =
-                               PropertyAttributes.RTSpecialName |
-                               PropertyAttributes.SpecialName;
-                       
-                       if (!DoDefine (parent))
-                               return false;
-
-                       IndexerName = Attribute.ScanForIndexerName (ec, OptAttributes);
-                       if (IndexerName == null)
-                               IndexerName = "Item";
-                       else if (IsExplicitImpl)
-                               Report.Error (592, Location,
-                                             "Attribute 'IndexerName' is not valid on this declaration " +
-                                             "type. It is valid on `property' declarations only.");
-
-                       ShortName = IndexerName;
-                       if (IsExplicitImpl) {
-                               InterfaceIndexerName = TypeManager.IndexerPropertyName (InterfaceType);
-                               Name = InterfaceType.FullName + "." + IndexerName;
-                       } else {
-                               InterfaceIndexerName = IndexerName;
-                               Name = ShortName;
-                       }
-
-                       if (!CheckBase (parent))
-                               return false;
-
-                       if (Get != null){
-                                InternalParameters ip = new InternalParameters (parent, Parameters);
-
-                               GetData = new MethodData (this, "get", MemberType,
-                                                         ParameterTypes, ip, CallingConventions.Standard,
-                                                         Get.OptAttributes, ModFlags, flags, false);
-
-                               if (!GetData.Define (parent))
-                                       return false;
-
-                               GetBuilder = GetData.MethodBuilder;
-                       }
-                       
-                       if (Set != null){
-                               int top = ParameterTypes.Length;
-                               Type [] set_pars = new Type [top + 1];
-                               ParameterTypes.CopyTo (set_pars, 0);
-                               set_pars [top] = MemberType;
-
-                               Parameter [] fixed_parms = Parameters.FixedParameters;
-
-                               if (fixed_parms == null){
-                                       throw new Exception ("We currently do not support only array arguments in an indexer");
-                                       // BUG BUG BUG BUG BUG BUG BUG BUG BUG BUG
-                                       // BUG BUG BUG BUG BUG BUG BUG BUG BUG BUG
-                                       //
-                                       // Here is the problem: the `value' parameter has
-                                       // to come *after* the array parameter in the declaration
-                                       // like this:
-                                       // X (object [] x, Type value)
-                                       // .param [0]
-                                       //
-                                       // BUG BUG BUG BUG BUG BUG BUG BUG BUG BUG
-                                       // BUG BUG BUG BUG BUG BUG BUG BUG BUG BUG
-                                       
-                               }
-                               
-                               Parameter [] tmp = new Parameter [fixed_parms.Length + 1];
-
-
-                               fixed_parms.CopyTo (tmp, 0);
-                               tmp [fixed_parms.Length] = new Parameter (
-                                       Type, this.Name, Parameter.Modifier.NONE, null);
-
-                               Parameters set_formal_params = new Parameters (tmp, null, Location);
-                               
-                               InternalParameters ip = new InternalParameters (parent, set_formal_params);
-
-                               SetData = new MethodData (this, "set", TypeManager.void_type,
-                                                         set_pars, ip, CallingConventions.Standard,
-                                                         Set.OptAttributes, ModFlags, flags, false);
-
-                               if (!SetData.Define (parent))
-                                       return false;
-
-                               SetBuilder = SetData.MethodBuilder;
-                       }
-
-                       //
-                       // Now name the parameters
-                       //
-                       Parameter [] p = Parameters.FixedParameters;
-                       if (p != null) {
-                               int i;
-                               
-                               for (i = 0; i < p.Length; ++i) {
-                                       if (Get != null)
-                                               GetBuilder.DefineParameter (
-                                                       i + 1, p [i].Attributes, p [i].Name);
-
-                                       if (Set != null)
-                                               SetBuilder.DefineParameter (
-                                                       i + 1, p [i].Attributes, p [i].Name);
-                               }
-                               
-
-                               if (Set != null)
-                                       SetBuilder.DefineParameter (
-                                               i + 1, ParameterAttributes.None, this.Name);
-                                       
-                               if (i != ParameterTypes.Length) {
-                                       Parameter array_param = Parameters.ArrayParameter;
-                                       SetBuilder.DefineParameter (
-                                               i + 1, array_param.Attributes, array_param.Name);
-                               }
-                       }
-
-                       if (GetData != null)
-                               IsImplementing = GetData.IsImplementing;
-                       else if (SetData != null)
-                               IsImplementing = SetData.IsImplementing;
-
-                       //
-                       // Define the PropertyBuilder if one of the following conditions are met:
-                       // a) we're not implementing an interface indexer.
-                       // b) the indexer has a different IndexerName and this is no
-                       //    explicit interface implementation.
-                       //
-                       if (!IsExplicitImpl) {
-                               PropertyBuilder = parent.TypeBuilder.DefineProperty (
-                                       IndexerName, prop_attr, MemberType, ParameterTypes);
-
-                               if (GetData != null)
-                                       PropertyBuilder.SetGetMethod (GetBuilder);
-
-                               if (SetData != null)
-                                       PropertyBuilder.SetSetMethod (SetBuilder);
-                               
-                               TypeManager.RegisterIndexer (PropertyBuilder, GetBuilder, SetBuilder,
-                                                            ParameterTypes);
+               public MethodAttributes MethodAttributeFlags {
+                       get {
+                               return flags;
                        }
-
-                       return true;
                }
+               
        }
-       */
 
        struct MethodSignature {
                public string Name;