X-Git-Url: http://wien.tomnetworks.com/gitweb/?a=blobdiff_plain;f=mcs%2Fmcs%2Fdelegate.cs;h=e78c6e1dbfdb9a4083874c889ae4c35083d00b7b;hb=2b808a73a435ae7c50a5edbd98eb12ba96fef4b5;hp=989fa917525a04fa16e2d5f94781d3746e543cf1;hpb=a1df5e44c85602c95913a487a0de022a1d6cafc6;p=mono.git diff --git a/mcs/mcs/delegate.cs b/mcs/mcs/delegate.cs index 989fa917525..e78c6e1dbfd 100644 --- a/mcs/mcs/delegate.cs +++ b/mcs/mcs/delegate.cs @@ -20,11 +20,10 @@ namespace Mono.CSharp { /// /// Holds Delegates /// - public class Delegate : MemberCore { - public readonly string ReturnType; + public class Delegate : DeclSpace { + public Expression ReturnType; public Parameters Parameters; public Attributes OptAttributes; - public TypeBuilder TypeBuilder; public ConstructorBuilder ConstructorBuilder; public MethodBuilder InvokeBuilder; @@ -42,49 +41,62 @@ namespace Mono.CSharp { Modifiers.PUBLIC | Modifiers.PROTECTED | Modifiers.INTERNAL | + Modifiers.UNSAFE | Modifiers.PRIVATE; - public Delegate (string type, int mod_flags, string name, Parameters param_list, + public Delegate (TypeContainer parent, Expression type, int mod_flags, + string name, Parameters param_list, Attributes attrs, Location l) - : base (name, l) + : base (parent, name, l) { this.ReturnType = type; - ModFlags = Modifiers.Check (AllowedModifiers, mod_flags, Modifiers.PUBLIC); + ModFlags = Modifiers.Check (AllowedModifiers, mod_flags, + IsTopLevel ? Modifiers.INTERNAL : + Modifiers.PRIVATE, l); Parameters = param_list; OptAttributes = attrs; } - public void DefineDelegate (object parent_builder) + public override TypeBuilder DefineType () { TypeAttributes attr; - string name = Name.Substring (1 + Name.LastIndexOf ('.')); + + if (TypeBuilder != null) + return TypeBuilder; - if (parent_builder is ModuleBuilder) { - ModuleBuilder builder = (ModuleBuilder) parent_builder; + if (IsTopLevel) { + ModuleBuilder builder = CodeGen.ModuleBuilder; attr = TypeAttributes.Public | TypeAttributes.Class | TypeAttributes.Sealed; TypeBuilder = builder.DefineType ( - name, attr, TypeManager.multicast_delegate_type); + Name, attr, TypeManager.multicast_delegate_type); } else { - // FIXME: We could just use TypeBuilder here. - TypeBuilder builder = (System.Reflection.Emit.TypeBuilder) parent_builder; + TypeBuilder builder = Parent.TypeBuilder; attr = TypeAttributes.NestedPublic | TypeAttributes.Class | TypeAttributes.Sealed; + string name = Name.Substring (1 + Name.LastIndexOf ('.')); TypeBuilder = builder.DefineNestedType ( name, attr, TypeManager.multicast_delegate_type); } - RootContext.TypeManager.AddDelegateType (Name, TypeBuilder, this); + TypeManager.AddDelegateType (Name, TypeBuilder, this); + + return TypeBuilder; + } + + public override bool DefineMembers (TypeContainer container) + { + return true; } - public override bool Define (TypeContainer parent) + public override bool Define (TypeContainer container) { MethodAttributes mattr; int i; - + // FIXME: POSSIBLY make this static, as it is always constant - // + // Type [] const_arg_types = new Type [2]; const_arg_types [0] = TypeManager.object_type; const_arg_types [1] = TypeManager.intptr_type; @@ -115,27 +127,46 @@ namespace Mono.CSharp { ConstructorBuilder.SetImplementationFlags (MethodImplAttributes.Runtime); - - // Here the various methods like Invoke, BeginInvoke etc are defined // - // Invoke method + // Here the various methods like Invoke, BeginInvoke etc are defined // - param_types = Parameters.GetParameterInfo (parent); + // First, call the `out of band' special method for + // defining recursively any types we need: + + if (!Parameters.ComputeAndDefineParameterTypes (this)) + return false; + + param_types = Parameters.GetParameterInfo (this); if (param_types == null) return false; + // + // Invoke method + // + // Check accessibility foreach (Type partype in param_types) - if (!TypeContainer.AsAccessible (partype, ModFlags)) + if (!container.AsAccessible (partype, ModFlags)) { + Report.Error (59, Location, + "Inconsistent accessibility: parameter type `" + + TypeManager.CSharpName (partype) + "` is less " + + "accessible than delegate `" + Name + "'"); return false; + } - ret_type = RootContext.LookupType (parent, ReturnType, false, Location); + ReturnType = ResolveTypeExpr (ReturnType, false, Location); + ret_type = ReturnType.Type; if (ret_type == null) return false; - if (!TypeContainer.AsAccessible (ret_type, ModFlags)) + if (!container.AsAccessible (ret_type, ModFlags)) { + Report.Error (58, Location, + "Inconsistent accessibility: return type `" + + TypeManager.CSharpName (ret_type) + "` is less " + + "accessible than delegate `" + Name + "'"); return false; + } // // We don't have to check any others because they are all @@ -152,16 +183,29 @@ namespace Mono.CSharp { ret_type, param_types); - for (i = 0 ; i < param_types.Length; i++) { - Parameter p = Parameters.FixedParameters [i]; - string name = p.Name; - ParameterBuilder pb = InvokeBuilder.DefineParameter (i+1, p.Attributes, name); + i = 0; + if (Parameters.FixedParameters != null){ + int top = Parameters.FixedParameters.Length; + Parameter p; + + for (; i < top; i++) { + p = Parameters.FixedParameters [i]; + + InvokeBuilder.DefineParameter ( + i+1, p.Attributes, p.Name); + } + } + if (Parameters.ArrayParameter != null){ + Parameter p = Parameters.ArrayParameter; + + InvokeBuilder.DefineParameter ( + i+1, p.Attributes, p.Name); } InvokeBuilder.SetImplementationFlags (MethodImplAttributes.Runtime); TypeManager.RegisterMethod (InvokeBuilder, - new InternalParameters (parent, Parameters), + new InternalParameters (container, Parameters), param_types); // @@ -175,8 +219,8 @@ namespace Mono.CSharp { async_param_types [params_num] = TypeManager.asynccallback_type; async_param_types [params_num + 1] = TypeManager.object_type; - mattr = MethodAttributes.Public | MethodAttributes.HideBySig | MethodAttributes.Virtual | - MethodAttributes.NewSlot; + mattr = MethodAttributes.Public | MethodAttributes.HideBySig | + MethodAttributes.Virtual | MethodAttributes.NewSlot; BeginInvokeBuilder = TypeBuilder.DefineMethod ("BeginInvoke", mattr, @@ -184,30 +228,52 @@ namespace Mono.CSharp { TypeManager.iasyncresult_type, async_param_types); - for (i = 0 ; i < param_types.Length; i++) { - Parameter p = Parameters.FixedParameters [i]; - string name = p.Name; - BeginInvokeBuilder.DefineParameter (i + 1, p.Attributes, name); + i = 0; + if (Parameters.FixedParameters != null){ + int top = Parameters.FixedParameters.Length; + Parameter p; + + for (i = 0 ; i < top; i++) { + p = Parameters.FixedParameters [i]; + + BeginInvokeBuilder.DefineParameter ( + i+1, p.Attributes, p.Name); + } } - + if (Parameters.ArrayParameter != null){ + Parameter p = Parameters.ArrayParameter; + + BeginInvokeBuilder.DefineParameter ( + i+1, p.Attributes, p.Name); + i++; + } + BeginInvokeBuilder.DefineParameter (i + 1, ParameterAttributes.None, "callback"); BeginInvokeBuilder.DefineParameter (i + 2, ParameterAttributes.None, "object"); BeginInvokeBuilder.SetImplementationFlags (MethodImplAttributes.Runtime); Parameter [] async_params = new Parameter [params_num + 2]; - if (params_num > 0) + int n = 0; + if (Parameters.FixedParameters != null){ Parameters.FixedParameters.CopyTo (async_params, 0); - - async_params [params_num] = new Parameter ("System.AsyncCallback", "callback", + n = Parameters.FixedParameters.Length; + } + if (Parameters.ArrayParameter != null) + async_params [n] = Parameters.ArrayParameter; + + async_params [params_num] = new Parameter ( + TypeManager.system_asynccallback_expr, "callback", Parameter.Modifier.NONE, null); - async_params [params_num + 1] = new Parameter ("System.IAsyncResult", "object", + async_params [params_num + 1] = new Parameter ( + TypeManager.system_object_expr, "object", Parameter.Modifier.NONE, null); + Parameters async_parameters = new Parameters (async_params, null, Location); + + async_parameters.ComputeAndDefineParameterTypes (this); TypeManager.RegisterMethod (BeginInvokeBuilder, - new InternalParameters ( - parent, - new Parameters (async_params, null, Location)), + new InternalParameters (container, async_parameters), async_param_types); // @@ -226,13 +292,15 @@ namespace Mono.CSharp { EndInvokeBuilder.SetImplementationFlags (MethodImplAttributes.Runtime); Parameter [] end_params = new Parameter [1]; - end_params [0] = new Parameter ("System.IAsyncResult", "result", + end_params [0] = new Parameter ( + TypeManager.system_iasyncresult_expr, "result", Parameter.Modifier.NONE, null); - TypeManager.RegisterMethod (EndInvokeBuilder, - new InternalParameters ( - parent, - new Parameters (end_params, null, Location)), + TypeManager.RegisterMethod ( + EndInvokeBuilder, new InternalParameters ( + container, + new Parameters ( + end_params, null, Location)), end_param_types); return true; @@ -247,10 +315,13 @@ namespace Mono.CSharp { { ParameterData pd = Invocation.GetParameterData (mb); - Expression ml = Expression.MemberLookup (ec, delegate_type, "Invoke", false, loc); + int pd_count = pd.Count; + + Expression ml = Expression.MemberLookup ( + ec, delegate_type, "Invoke", loc); if (!(ml is MethodGroupExpr)) { - Report.Error (-100, loc, "Internal error : could not find Invoke method!"); + Report.Error (-100, loc, "Internal error: could not find Invoke method!"); return null; } @@ -258,46 +329,32 @@ namespace Mono.CSharp { ParameterData invoke_pd = Invocation.GetParameterData (invoke_mb); - bool mismatch = false; - for (int i = pd.Count; i > 0; ) { + if (invoke_pd.Count != pd_count) + return null; + + for (int i = pd_count; i > 0; ) { i--; - if (invoke_pd.ParameterType (i) == pd.ParameterType (i)) + if (invoke_pd.ParameterType (i) == pd.ParameterType (i) && + invoke_pd.ParameterModifier (i) == pd.ParameterModifier (i)) continue; - else { - mismatch = true; - break; - } - } - - if (mismatch) { - Report.Error ( - 123, loc, "Method '" + Invocation.FullMethodDesc (mb) + - "' does not match delegate '" + - FullDelegateDesc (delegate_type, invoke_mb, invoke_pd) + "'"); - return null; + else + return null; } if (((MethodInfo) invoke_mb).ReturnType == ((MethodInfo) mb).ReturnType) return mb; else - mismatch = true; - - if (mismatch) { - Report.Error (123, loc, "Method '" + Invocation.FullMethodDesc (mb) + - "' does not match delegate '" + - FullDelegateDesc (delegate_type, invoke_mb, invoke_pd) + "'"); return null; - } - - return null; } // // Verifies whether the invocation arguments are compatible with the // delegate's target method // - public static bool VerifyApplicability (EmitContext ec, Type delegate_type, ArrayList args, + public static bool VerifyApplicability (EmitContext ec, + Type delegate_type, + ArrayList args, Location loc) { int arg_count; @@ -307,75 +364,55 @@ namespace Mono.CSharp { else arg_count = args.Count; - Expression ml = Expression.MemberLookup (ec, delegate_type, "Invoke", false, loc); - + Expression ml = Expression.MemberLookup ( + ec, delegate_type, "Invoke", loc); + if (!(ml is MethodGroupExpr)) { - Report.Error (-100, loc, "Internal error : could not find Invoke method!"); + Report.Error (-100, loc, "Internal error: could not find Invoke method!" + delegate_type); return false; } MethodBase mb = ((MethodGroupExpr) ml).Methods [0]; - ParameterData pd = Invocation.GetParameterData (mb); - - if (pd.Count != arg_count) { + + int pd_count = pd.Count; + + bool not_params_method = (pd_count == 0) || + (pd.ParameterModifier (pd_count - 1) != Parameter.Modifier.PARAMS); + + if (not_params_method && pd_count != arg_count) { Report.Error (1593, loc, "Delegate '" + delegate_type.ToString () + "' does not take '" + arg_count + "' arguments"); return false; } - for (int i = arg_count; i > 0;) { - i--; - Expression conv; - Argument a = (Argument) args [i]; - Expression a_expr = a.Expr; - - if (pd.ParameterType (i) != a_expr.Type) { - - conv = Expression.ConvertImplicitStandard (ec, a_expr, pd.ParameterType (i), loc); - - if (conv == null) { - Report.Error (1594, loc, - "Delegate '" + delegate_type.ToString () + - "' has some invalid arguments."); - - Report.Error (1503, loc, - "Argument " + (i+1) + - ": Cannot convert from '" + - TypeManager.CSharpName (a_expr.Type) - + "' to '" + TypeManager.CSharpName (pd.ParameterType (i)) + "'"); - return false; - } - - if (a_expr != conv) - a.Expr = conv; - } - } - - return true; + return Invocation.VerifyArgumentsCompat (ec, args, arg_count, mb, !not_params_method, + delegate_type, loc); } - + /// /// Verifies whether the delegate in question is compatible with this one in /// order to determine if instantiation from the same is possible. /// public static bool VerifyDelegate (EmitContext ec, Type delegate_type, Type probe_type, Location loc) { - Expression ml = Expression.MemberLookup (ec, delegate_type, "Invoke", false, loc); + Expression ml = Expression.MemberLookup ( + ec, delegate_type, "Invoke", loc); if (!(ml is MethodGroupExpr)) { - Report.Error (-100, loc, "Internal error : could not find Invoke method!"); + Report.Error (-100, loc, "Internal error: could not find Invoke method!"); return false; } MethodBase mb = ((MethodGroupExpr) ml).Methods [0]; ParameterData pd = Invocation.GetParameterData (mb); - Expression probe_ml = Expression.MemberLookup (ec, delegate_type, "Invoke", false, loc); + Expression probe_ml = Expression.MemberLookup ( + ec, delegate_type, "Invoke", loc); if (!(probe_ml is MethodGroupExpr)) { - Report.Error (-100, loc, "Internal error : could not find Invoke method!"); + Report.Error (-100, loc, "Internal error: could not find Invoke method!"); return false; } @@ -410,8 +447,8 @@ namespace Mono.CSharp { for (int i = length; i > 0; ) { i--; - - sb.Append (TypeManager.CSharpName (pd.ParameterType (length - i - 1))); + + sb.Append (pd.ParameterDesc (length - i - 1)); if (i != 0) sb.Append (", "); } @@ -422,40 +459,38 @@ namespace Mono.CSharp { } // Hack around System.Reflection as found everywhere else - public MemberInfo [] FindMembers (MemberTypes mt, BindingFlags bf, MemberFilter filter, object criteria) + public override MemberList FindMembers (MemberTypes mt, BindingFlags bf, + MemberFilter filter, object criteria) { ArrayList members = new ArrayList (); if ((mt & MemberTypes.Method) != 0) { + if (ConstructorBuilder != null) if (filter (ConstructorBuilder, criteria)) members.Add (ConstructorBuilder); + if (InvokeBuilder != null) if (filter (InvokeBuilder, criteria)) members.Add (InvokeBuilder); + if (BeginInvokeBuilder != null) if (filter (BeginInvokeBuilder, criteria)) members.Add (BeginInvokeBuilder); + if (EndInvokeBuilder != null) if (filter (EndInvokeBuilder, criteria)) members.Add (EndInvokeBuilder); } - int count = members.Count; + return new MemberList (members); + } - if (count > 0) { - MemberInfo [] mi = new MemberInfo [count]; - members.CopyTo (mi, 0); - return mi; + public override MemberCache MemberCache { + get { + return null; } - - return null; } - - public void CloseDelegate () - { - TypeBuilder.CreateType (); - } - + public Expression InstanceExpression { get { return instance_expr; @@ -496,58 +531,94 @@ namespace Mono.CSharp { MethodBase delegate_method; Expression delegate_instance_expr; - Location Location; - public NewDelegate (Type type, ArrayList Arguments, Location loc) { this.type = type; this.Arguments = Arguments; - this.Location = loc; + this.loc = loc; } public override Expression DoResolve (EmitContext ec) { if (Arguments == null) { - Report.Error (-11, Location, + Report.Error (-11, loc, "Delegate creation expression takes only one argument"); return null; } - + if (Arguments.Count != 1) { - Report.Error (-11, Location, + Report.Error (-11, loc, "Delegate creation expression takes only one argument"); return null; } - Expression ml = Expression.MemberLookup (ec, type, ".ctor", false, Location); + Expression ml = Expression.MemberLookup ( + ec, type, ".ctor", loc); if (!(ml is MethodGroupExpr)) { - Report.Error (-100, Location, "Internal error : Could not find delegate constructor!"); + Report.Error (-100, loc, "Internal error: Could not find delegate constructor!"); return null; } constructor_method = ((MethodGroupExpr) ml).Methods [0]; - Argument a = (Argument) Arguments [0]; - if (!a.Resolve (ec, Location)) + if (!a.ResolveMethodGroup (ec, Location)) return null; Expression e = a.Expr; + Expression invoke_method = Expression.MemberLookup ( + ec, type, "Invoke", MemberTypes.Method, + Expression.AllBindingFlags, loc); + + if (invoke_method == null) { + Report.Error (-200, loc, "Internal error ! Could not find Invoke method!"); + return null; + } + if (e is MethodGroupExpr) { MethodGroupExpr mg = (MethodGroupExpr) e; - if (mg.Methods.Length > 1) { - Report.Error (-14, Location, "Ambiguous method reference in delegate creation"); - return null; + foreach (MethodInfo mi in mg.Methods){ + delegate_method = Delegate.VerifyMethod (ec, type, mi, loc); + + if (delegate_method != null) + break; } - - delegate_method = Delegate.VerifyMethod (ec, type, mg.Methods [0], Location); - - if (delegate_method == null) + + if (delegate_method == null) { + string method_desc; + if (mg.Methods.Length > 1) + method_desc = mg.Methods [0].Name; + else + method_desc = Invocation.FullMethodDesc (mg.Methods [0]); + + MethodBase dm = ((MethodGroupExpr) invoke_method).Methods [0]; + ParameterData param = Invocation.GetParameterData (dm); + string delegate_desc = Delegate.FullDelegateDesc (type, dm, param); + + Report.Error (123, loc, "Method '" + method_desc + "' does not " + + "match delegate '" + delegate_desc + "'"); + return null; + } + // + // Check safe/unsafe of the delegate + // + if (!ec.InUnsafe){ + ParameterData param = Invocation.GetParameterData (delegate_method); + int count = param.Count; + + for (int i = 0; i < count; i++){ + if (param.ParameterType (i).IsPointer){ + Expression.UnsafeError (loc); + return null; + } + } + } + if (mg.InstanceExpression != null) delegate_instance_expr = mg.InstanceExpression.Resolve (ec); else { @@ -568,7 +639,7 @@ namespace Mono.CSharp { Type e_type = e.Type; if (!TypeManager.IsDelegateType (e_type)) { - Report.Error (-12, Location, "Cannot create a delegate from something " + + Report.Error (-12, loc, "Cannot create a delegate from something " + "not a delegate or a method."); return null; } @@ -576,24 +647,14 @@ namespace Mono.CSharp { // This is what MS' compiler reports. We could always choose // to be more verbose and actually give delegate-level specifics - if (!Delegate.VerifyDelegate (ec, type, e_type, Location)) { - Report.Error (29, Location, "Cannot implicitly convert type '" + e_type + "' " + + if (!Delegate.VerifyDelegate (ec, type, e_type, loc)) { + Report.Error (29, loc, "Cannot implicitly convert type '" + e_type + "' " + "to type '" + type + "'"); return null; } - - Expression invoke_method = Expression.MemberLookup (ec, e_type, "Invoke", false, - MemberTypes.Method, - Expression.AllBindingFlags, - Location); - - if (invoke_method == null) { - Report.Error (-200, Location, "Internal error ! COuld not find Invoke method!"); - return null; - } delegate_instance_expr = e; - delegate_method = ((MethodGroupExpr) invoke_method).Methods [0]; + delegate_method = ((MethodGroupExpr) invoke_method).Methods [0]; eclass = ExprClass.Value; return this; @@ -601,12 +662,17 @@ namespace Mono.CSharp { public override void Emit (EmitContext ec) { - if (delegate_instance_expr == null) + if (delegate_instance_expr == null || + delegate_method.IsStatic) ec.ig.Emit (OpCodes.Ldnull); else delegate_instance_expr.Emit (ec); - ec.ig.Emit (OpCodes.Ldftn, (MethodInfo) delegate_method); + if (delegate_method.IsVirtual) { + ec.ig.Emit (OpCodes.Dup); + ec.ig.Emit (OpCodes.Ldvirtftn, (MethodInfo) delegate_method); + } else + ec.ig.Emit (OpCodes.Ldftn, (MethodInfo) delegate_method); ec.ig.Emit (OpCodes.Newobj, (ConstructorInfo) constructor_method); } } @@ -615,7 +681,6 @@ namespace Mono.CSharp { public Expression InstanceExpr; public ArrayList Arguments; - public Location Location; MethodBase method; @@ -623,40 +688,58 @@ namespace Mono.CSharp { { this.InstanceExpr = instance_expr; this.Arguments = args; - this.Location = loc; + this.loc = loc; } public override Expression DoResolve (EmitContext ec) { + if (InstanceExpr is EventExpr) { + + EventInfo ei = ((EventExpr) InstanceExpr).EventInfo; + + Expression ml = MemberLookup ( + ec, ec.ContainerType, ei.Name, + MemberTypes.Event, AllBindingFlags | BindingFlags.DeclaredOnly, loc); + + if (ml == null) { + // + // If this is the case, then the Event does not belong + // to this Type and so, according to the spec + // cannot be accessed directly + // + // Note that target will not appear as an EventExpr + // in the case it is being referenced within the same type container; + // it will appear as a FieldExpr in that case. + // + + Assign.error70 (ei, loc); + return null; + } + } + + Type del_type = InstanceExpr.Type; - if (del_type == null) return null; if (Arguments != null){ - for (int i = Arguments.Count; i > 0;){ - --i; - Argument a = (Argument) Arguments [i]; - - if (!a.Resolve (ec, Location)) + foreach (Argument a in Arguments){ + if (!a.Resolve (ec, loc)) return null; } } - if (!Delegate.VerifyApplicability (ec, del_type, Arguments, Location)) + if (!Delegate.VerifyApplicability (ec, del_type, Arguments, loc)) return null; - Expression ml = Expression.MemberLookup (ec, del_type, "Invoke", false, Location); - - if (!(ml is MethodGroupExpr)) { - Report.Error (-100, Location, "Internal error : could not find Invoke method!"); + Expression lookup = Expression.MemberLookup (ec, del_type, "Invoke", loc); + if (!(lookup is MethodGroupExpr)) { + Report.Error (-100, loc, "Internal error: could not find Invoke method!"); return null; } - method = ((MethodGroupExpr) ml).Methods [0]; - + method = ((MethodGroupExpr) lookup).Methods [0]; type = ((MethodInfo) method).ReturnType; - eclass = ExprClass.Value; return this; @@ -670,7 +753,7 @@ namespace Mono.CSharp { // Invocation on delegates call the virtual Invoke member // so we are always `instance' calls // - Invocation.EmitCall (ec, false, false, InstanceExpr, method, Arguments); + Invocation.EmitCall (ec, false, false, InstanceExpr, method, Arguments, loc); } public override void EmitStatement (EmitContext ec)