X-Git-Url: http://wien.tomnetworks.com/gitweb/?a=blobdiff_plain;f=mcs%2Fmcs%2Fdecl.cs;h=5f04e33fab416635e4205d59006fb63a1877774c;hb=55c4f6f6cfdf1df8233333d76f73274fd6d3a832;hp=1bcaf63ed6a72a6880efbaefe89a9624e512cb59;hpb=2332d6ff00c77598def60262d6c620b4e340db9c;p=mono.git
diff --git a/mcs/mcs/decl.cs b/mcs/mcs/decl.cs
index 1bcaf63ed6a..5f04e33fab4 100644
--- a/mcs/mcs/decl.cs
+++ b/mcs/mcs/decl.cs
@@ -13,12 +13,12 @@
using System;
using System.Text;
-using System.Collections;
+using System.Collections.Generic;
using System.Globalization;
using System.Reflection.Emit;
using System.Reflection;
-#if BOOTSTRAP_WITH_OLDLIB || NET_2_1
+#if NET_2_1
using XmlElement = System.Object;
#else
using System.Xml;
@@ -31,7 +31,7 @@ namespace Mono.CSharp {
//
public class MemberName {
public readonly string Name;
- public readonly TypeArguments TypeArguments;
+ public TypeArguments TypeArguments;
public readonly MemberName Left;
public readonly Location Location;
@@ -103,6 +103,12 @@ namespace Mono.CSharp {
return GetName (false);
}
+ public int Arity {
+ get {
+ return TypeArguments == null ? 0 : TypeArguments.Count;
+ }
+ }
+
public bool IsGeneric {
get {
if (TypeArguments != null)
@@ -127,7 +133,7 @@ namespace Mono.CSharp {
{
if (Left == null) {
if (TypeArguments != null)
- return new SimpleName (Basename, TypeArguments, Location);
+ return new SimpleName (Name, TypeArguments, Location);
return new SimpleName (Name, Location);
}
@@ -231,11 +237,25 @@ namespace Mono.CSharp {
}
}
+ public class SimpleMemberName
+ {
+ public string Value;
+ public Location Location;
+
+ public SimpleMemberName (string name, Location loc)
+ {
+ this.Value = name;
+ this.Location = loc;
+ }
+ }
+
///
/// Base representation for members. This is used to keep track
/// of Name, Location and Modifier flags, and handling Attributes.
///
- public abstract class MemberCore : Attributable, IMemberContext {
+ [System.Diagnostics.DebuggerDisplay ("{GetSignatureForError()}")]
+ public abstract class MemberCore : Attributable, IMemberContext, IMemberDefinition
+ {
///
/// Public name
///
@@ -250,6 +270,12 @@ namespace Mono.CSharp {
}
}
+ string IMemberDefinition.Name {
+ get {
+ return member_name.Name;
+ }
+ }
+
// Is not readonly because of IndexerName attribute
private MemberName member_name;
public MemberName MemberName {
@@ -259,8 +285,8 @@ namespace Mono.CSharp {
///
/// Modifier flags that the user specified in the source code
///
- private int mod_flags;
- public int ModFlags {
+ private Modifiers mod_flags;
+ public Modifiers ModFlags {
set {
mod_flags = value;
if ((value & Modifiers.COMPILER_GENERATED) != 0)
@@ -271,7 +297,7 @@ namespace Mono.CSharp {
}
}
- public /*readonly*/ DeclSpace Parent;
+ public /*readonly*/ TypeContainer Parent;
///
/// Location where this declaration happens
@@ -300,7 +326,7 @@ namespace Mono.CSharp {
CloseTypeCreated = 1 << 4, // Tracks whether we have Closed the type
HasCompliantAttribute_Undetected = 1 << 5, // Presence of CLSCompliantAttribute has not been detected
HasClsCompliantAttribute = 1 << 6, // Type has CLSCompliantAttribute
- ClsCompliantAttributeTrue = 1 << 7, // Type has CLSCompliant (true)
+ ClsCompliantAttributeFalse = 1 << 7, // Member has CLSCompliant(false)
Excluded_Undetected = 1 << 8, // Conditional attribute has not been detected yet
Excluded = 1 << 9, // Method is conditional
MethodOverloadsExist = 1 << 10, // Test for duplication must be performed
@@ -318,12 +344,16 @@ namespace Mono.CSharp {
public MemberCore (DeclSpace parent, MemberName name, Attributes attrs)
{
- this.Parent = parent;
+ this.Parent = parent as TypeContainer;
member_name = name;
caching_flags = Flags.Obsolete_Undetected | Flags.ClsCompliance_Undetected | Flags.HasCompliantAttribute_Undetected | Flags.Excluded_Undetected;
AddAttributes (attrs, this);
}
+ public virtual Assembly Assembly {
+ get { return Parent.Module.Assembly; }
+ }
+
protected virtual void SetMemberName (MemberName new_name)
{
member_name = new_name;
@@ -332,7 +362,7 @@ namespace Mono.CSharp {
protected bool CheckAbstractAndExtern (bool has_block)
{
- if (Parent.PartialContainer.Kind == Kind.Interface)
+ if (Parent.PartialContainer.Kind == MemberKind.Interface)
return true;
if (has_block) {
@@ -348,7 +378,7 @@ namespace Mono.CSharp {
return false;
}
} else {
- if ((ModFlags & (Modifiers.ABSTRACT | Modifiers.EXTERN | Modifiers.PARTIAL)) == 0) {
+ if ((ModFlags & (Modifiers.ABSTRACT | Modifiers.EXTERN | Modifiers.PARTIAL)) == 0 && !(Parent is Delegate)) {
if (RootContext.Version >= LanguageVersion.V_3) {
Property.PropertyMethod pm = this as Property.PropertyMethod;
if (pm is Indexer.GetIndexerMethod || pm is Indexer.SetIndexerMethod)
@@ -371,12 +401,12 @@ namespace Mono.CSharp {
return true;
}
- public void CheckProtectedModifier ()
+ protected void CheckProtectedModifier ()
{
if ((ModFlags & Modifiers.PROTECTED) == 0)
return;
- if (Parent.PartialContainer.Kind == Kind.Struct) {
+ if (Parent.PartialContainer.Kind == MemberKind.Struct) {
Report.Error (666, Location, "`{0}': Structs cannot contain protected members",
GetSignatureForError ());
return;
@@ -426,8 +456,7 @@ namespace Mono.CSharp {
if (!RootContext.VerifyClsCompliance)
return;
- if (Report.WarningLevel > 0)
- VerifyClsCompliance ();
+ VerifyClsCompliance ();
}
public bool IsCompilerGenerated {
@@ -439,19 +468,36 @@ namespace Mono.CSharp {
}
}
+ public bool IsImported {
+ get {
+ return false;
+ }
+ }
+
public virtual bool IsUsed {
get { return (caching_flags & Flags.IsUsed) != 0; }
}
- public void SetMemberIsUsed ()
+ protected Report Report {
+ get {
+ return Compiler.Report;
+ }
+ }
+
+ public void SetIsUsed ()
{
caching_flags |= Flags.IsUsed;
}
+ public void SetIsAssigned ()
+ {
+ caching_flags |= Flags.IsAssigned;
+ }
+
///
/// Returns instance of ObsoleteAttribute for this MemberCore
///
- public virtual ObsoleteAttribute GetObsoleteAttribute ()
+ public virtual ObsoleteAttribute GetAttributeObsolete ()
{
if ((caching_flags & (Flags.Obsolete_Undetected | Flags.Obsolete)) == 0)
return null;
@@ -479,72 +525,15 @@ namespace Mono.CSharp {
///
public virtual void CheckObsoleteness (Location loc)
{
- ObsoleteAttribute oa = GetObsoleteAttribute ();
+ ObsoleteAttribute oa = GetAttributeObsolete ();
if (oa != null)
- AttributeTester.Report_ObsoleteMessage (oa, GetSignatureForError (), loc);
- }
-
- // Access level of a type.
- const int X = 1;
- enum AccessLevel
- { // Each column represents `is this scope larger or equal to Blah scope'
- // Public Assembly Protected
- Protected = (0 << 0) | (0 << 1) | (X << 2),
- Public = (X << 0) | (X << 1) | (X << 2),
- Private = (0 << 0) | (0 << 1) | (0 << 2),
- Internal = (0 << 0) | (X << 1) | (0 << 2),
- ProtectedOrInternal = (0 << 0) | (X << 1) | (X << 2),
- }
-
- static AccessLevel GetAccessLevelFromModifiers (int flags)
- {
- if ((flags & Modifiers.INTERNAL) != 0) {
-
- if ((flags & Modifiers.PROTECTED) != 0)
- return AccessLevel.ProtectedOrInternal;
- else
- return AccessLevel.Internal;
-
- } else if ((flags & Modifiers.PROTECTED) != 0)
- return AccessLevel.Protected;
- else if ((flags & Modifiers.PRIVATE) != 0)
- return AccessLevel.Private;
- else
- return AccessLevel.Public;
- }
-
- //
- // Returns the access level for type `t'
- //
- static AccessLevel GetAccessLevelFromType (Type t)
- {
- if (t.IsPublic)
- return AccessLevel.Public;
- if (t.IsNestedPrivate)
- return AccessLevel.Private;
- if (t.IsNotPublic)
- return AccessLevel.Internal;
-
- if (t.IsNestedPublic)
- return AccessLevel.Public;
- if (t.IsNestedAssembly)
- return AccessLevel.Internal;
- if (t.IsNestedFamily)
- return AccessLevel.Protected;
- if (t.IsNestedFamORAssem)
- return AccessLevel.ProtectedOrInternal;
- if (t.IsNestedFamANDAssem)
- throw new NotImplementedException ("NestedFamANDAssem not implemented, cant make this kind of type from c# anyways");
-
- // nested private is taken care of
-
- throw new Exception ("I give up, what are you?");
+ AttributeTester.Report_ObsoleteMessage (oa, GetSignatureForError (), loc, Report);
}
//
// Checks whether the type P is as accessible as this member
//
- public bool IsAccessibleAs (Type p)
+ public bool IsAccessibleAs (TypeSpec p)
{
//
// if M is private, its accessibility is the same as this declspace.
@@ -557,69 +546,73 @@ namespace Mono.CSharp {
while (TypeManager.HasElementType (p))
p = TypeManager.GetElementType (p);
- if (TypeManager.IsGenericParameter (p))
+ if (p.IsGenericParameter)
return true;
- if (TypeManager.IsGenericType (p)) {
- foreach (Type t in TypeManager.GetTypeArguments (p)) {
- if (!IsAccessibleAs (t))
- return false;
+ for (TypeSpec p_parent; p != null; p = p_parent) {
+ p_parent = p.DeclaringType;
+
+ if (p.IsGeneric) {
+ foreach (TypeSpec t in p.TypeArguments) {
+ if (!IsAccessibleAs (t))
+ return false;
+ }
}
- }
- for (Type p_parent = null; p != null; p = p_parent) {
- p_parent = p.DeclaringType;
- AccessLevel pAccess = GetAccessLevelFromType (p);
- if (pAccess == AccessLevel.Public)
+ var pAccess = p.Modifiers & Modifiers.AccessibilityMask;
+ if (pAccess == Modifiers.PUBLIC)
continue;
bool same_access_restrictions = false;
for (MemberCore mc = this; !same_access_restrictions && mc != null && mc.Parent != null; mc = mc.Parent) {
- AccessLevel al = GetAccessLevelFromModifiers (mc.ModFlags);
+ var al = mc.ModFlags & Modifiers.AccessibilityMask;
switch (pAccess) {
- case AccessLevel.Internal:
- if (al == AccessLevel.Private || al == AccessLevel.Internal)
- same_access_restrictions = TypeManager.IsThisOrFriendAssembly (p.Assembly);
+ case Modifiers.INTERNAL:
+ if (al == Modifiers.PRIVATE || al == Modifiers.INTERNAL)
+ same_access_restrictions = TypeManager.IsThisOrFriendAssembly (Parent.Module.Assembly, p.Assembly);
break;
-
- case AccessLevel.Protected:
- if (al == AccessLevel.Protected) {
- same_access_restrictions = mc.Parent.IsBaseType (p_parent);
+
+ case Modifiers.PROTECTED:
+ if (al == Modifiers.PROTECTED) {
+ same_access_restrictions = mc.Parent.IsBaseTypeDefinition (p_parent);
break;
}
-
- if (al == AccessLevel.Private) {
+
+ if (al == Modifiers.PRIVATE) {
//
// When type is private and any of its parents derives from
// protected type then the type is accessible
//
while (mc.Parent != null) {
- if (mc.Parent.IsBaseType (p_parent))
+ if (mc.Parent.IsBaseTypeDefinition (p_parent))
same_access_restrictions = true;
mc = mc.Parent;
}
}
break;
-
- case AccessLevel.ProtectedOrInternal:
- if (al == AccessLevel.Protected)
- same_access_restrictions = mc.Parent.IsBaseType (p_parent);
- else if (al == AccessLevel.Internal)
- same_access_restrictions = TypeManager.IsThisOrFriendAssembly (p.Assembly);
- else if (al == AccessLevel.ProtectedOrInternal)
- same_access_restrictions = mc.Parent.IsBaseType (p_parent) &&
- TypeManager.IsThisOrFriendAssembly (p.Assembly);
-
+
+ case Modifiers.PROTECTED | Modifiers.INTERNAL:
+ if (al == Modifiers.INTERNAL)
+ same_access_restrictions = TypeManager.IsThisOrFriendAssembly (Parent.Module.Assembly, p.Assembly);
+ else if (al == Modifiers.PROTECTED)
+ same_access_restrictions = mc.Parent.IsBaseTypeDefinition (p_parent);
+ else if (al == (Modifiers.PROTECTED | Modifiers.INTERNAL))
+ same_access_restrictions = mc.Parent.IsBaseTypeDefinition (p_parent) &&
+ TypeManager.IsThisOrFriendAssembly (Parent.Module.Assembly, p.Assembly);
break;
-
- case AccessLevel.Private:
+
+ case Modifiers.PRIVATE:
//
// Both are private and share same parent
//
- if (al == AccessLevel.Private)
- same_access_restrictions = TypeManager.IsEqual (mc.Parent.TypeBuilder, p_parent);
+ if (al == Modifiers.PRIVATE) {
+ var decl = mc.Parent;
+ do {
+ same_access_restrictions = decl.CurrentType == p_parent;
+ } while (!same_access_restrictions && !decl.IsTopLevel && (decl = decl.Parent) != null);
+ }
break;
@@ -643,16 +636,29 @@ namespace Mono.CSharp {
if ((caching_flags & Flags.ClsCompliance_Undetected) == 0)
return (caching_flags & Flags.ClsCompliant) != 0;
- if (GetClsCompliantAttributeValue () && IsExposedFromAssembly ()) {
- caching_flags &= ~Flags.ClsCompliance_Undetected;
+ caching_flags &= ~Flags.ClsCompliance_Undetected;
+
+ if (HasClsCompliantAttribute) {
+ if ((caching_flags & Flags.ClsCompliantAttributeFalse) != 0)
+ return false;
+
+ caching_flags |= Flags.ClsCompliant;
+ return true;
+ }
+
+ if (Parent.PartialContainer.IsClsComplianceRequired ()) {
caching_flags |= Flags.ClsCompliant;
return true;
}
- caching_flags &= ~Flags.ClsCompliance_Undetected;
return false;
}
+ public virtual string[] ConditionalConditions ()
+ {
+ return null;
+ }
+
///
/// Returns true when MemberCore is exposed from assembly.
///
@@ -661,7 +667,7 @@ namespace Mono.CSharp {
if ((ModFlags & (Modifiers.PUBLIC | Modifiers.PROTECTED)) == 0)
return false;
- DeclSpace parentContainer = Parent;
+ DeclSpace parentContainer = Parent.PartialContainer;
while (parentContainer != null && parentContainer.ModFlags != 0) {
if ((parentContainer.ModFlags & (Modifiers.PUBLIC | Modifiers.PROTECTED)) == 0)
return false;
@@ -670,9 +676,9 @@ namespace Mono.CSharp {
return true;
}
- public virtual ExtensionMethodGroupExpr LookupExtensionMethod (Type extensionType, string name, Location loc)
+ public virtual ExtensionMethodGroupExpr LookupExtensionMethod (TypeSpec extensionType, string name, int arity, Location loc)
{
- return Parent.LookupExtensionMethod (extensionType, name, loc);
+ return Parent.LookupExtensionMethod (extensionType, name, arity, loc);
}
public virtual FullNamedExpression LookupNamespaceAlias (string name)
@@ -680,42 +686,34 @@ namespace Mono.CSharp {
return Parent.NamespaceEntry.LookupNamespaceAlias (name);
}
- public virtual FullNamedExpression LookupNamespaceOrType (string name, Location loc, bool ignore_cs0104)
+ public virtual FullNamedExpression LookupNamespaceOrType (string name, int arity, Location loc, bool ignore_cs0104)
{
- return Parent.LookupNamespaceOrType (name, loc, ignore_cs0104);
+ return Parent.LookupNamespaceOrType (name, arity, loc, ignore_cs0104);
}
///
/// Goes through class hierarchy and gets value of first found CLSCompliantAttribute.
/// If no is attribute exists then assembly CLSCompliantAttribute is returned.
///
- public virtual bool GetClsCompliantAttributeValue ()
+ public bool IsNotCLSCompliant ()
{
if ((caching_flags & Flags.HasCompliantAttribute_Undetected) == 0)
- return (caching_flags & Flags.ClsCompliantAttributeTrue) != 0;
+ return (caching_flags & Flags.ClsCompliantAttributeFalse) != 0;
caching_flags &= ~Flags.HasCompliantAttribute_Undetected;
if (OptAttributes != null) {
- Attribute cls_attribute = OptAttributes.Search (
- PredefinedAttributes.Get.CLSCompliant);
+ Attribute cls_attribute = OptAttributes.Search (PredefinedAttributes.Get.CLSCompliant);
if (cls_attribute != null) {
caching_flags |= Flags.HasClsCompliantAttribute;
- bool value = cls_attribute.GetClsCompliantAttributeValue ();
- if (value)
- caching_flags |= Flags.ClsCompliantAttributeTrue;
- return value;
+ if (cls_attribute.GetClsCompliantAttributeValue ())
+ return false;
+
+ caching_flags |= Flags.ClsCompliantAttributeFalse;
+ return true;
}
}
-
- // It's null for TypeParameter
- if (Parent == null)
- return false;
- if (Parent.GetClsCompliantAttributeValue ()) {
- caching_flags |= Flags.ClsCompliantAttributeTrue;
- return true;
- }
return false;
}
@@ -725,7 +723,7 @@ namespace Mono.CSharp {
protected bool HasClsCompliantAttribute {
get {
if ((caching_flags & Flags.HasCompliantAttribute_Undetected) != 0)
- GetClsCompliantAttributeValue ();
+ IsNotCLSCompliant ();
return (caching_flags & Flags.HasClsCompliantAttribute) != 0;
}
@@ -747,41 +745,55 @@ namespace Mono.CSharp {
///
protected virtual bool VerifyClsCompliance ()
{
- if (!IsClsComplianceRequired ()) {
- if (HasClsCompliantAttribute && Report.WarningLevel >= 2) {
- if (!IsExposedFromAssembly ()) {
- Attribute a = OptAttributes.Search (PredefinedAttributes.Get.CLSCompliant);
- Report.Warning (3019, 2, a.Location, "CLS compliance checking will not be performed on `{0}' because it is not visible from outside this assembly", GetSignatureForError ());
- }
-
- if (!CodeGen.Assembly.IsClsCompliant) {
- Attribute a = OptAttributes.Search (PredefinedAttributes.Get.CLSCompliant);
- Report.Warning (3021, 2, a.Location, "`{0}' does not need a CLSCompliant attribute because the assembly is not marked as CLS-compliant", GetSignatureForError ());
+ if (HasClsCompliantAttribute) {
+ if (CodeGen.Assembly.ClsCompliantAttribute == null) {
+ Attribute a = OptAttributes.Search (PredefinedAttributes.Get.CLSCompliant);
+ if ((caching_flags & Flags.ClsCompliantAttributeFalse) != 0) {
+ Report.Warning (3021, 2, a.Location,
+ "`{0}' does not need a CLSCompliant attribute because the assembly is not marked as CLS-compliant",
+ GetSignatureForError ());
+ } else {
+ Report.Warning (3014, 1, a.Location,
+ "`{0}' cannot be marked as CLS-compliant because the assembly is not marked as CLS-compliant",
+ GetSignatureForError ());
}
+ return false;
}
- return false;
- }
- if (HasClsCompliantAttribute) {
- if (CodeGen.Assembly.ClsCompliantAttribute == null && !CodeGen.Assembly.IsClsCompliant) {
+ if (!IsExposedFromAssembly ()) {
Attribute a = OptAttributes.Search (PredefinedAttributes.Get.CLSCompliant);
- Report.Warning (3014, 1, a.Location,
- "`{0}' cannot be marked as CLS-compliant because the assembly is not marked as CLS-compliant",
- GetSignatureForError ());
+ Report.Warning (3019, 2, a.Location, "CLS compliance checking will not be performed on `{0}' because it is not visible from outside this assembly", GetSignatureForError ());
+ return false;
+ }
+
+ if ((caching_flags & Flags.ClsCompliantAttributeFalse) != 0) {
+ if (Parent.Kind == MemberKind.Interface && Parent.IsClsComplianceRequired ()) {
+ Report.Warning (3010, 1, Location, "`{0}': CLS-compliant interfaces must have only CLS-compliant members", GetSignatureForError ());
+ } else if (Parent.Kind == MemberKind.Class && (ModFlags & Modifiers.ABSTRACT) != 0 && Parent.IsClsComplianceRequired ()) {
+ Report.Warning (3011, 1, Location, "`{0}': only CLS-compliant members can be abstract", GetSignatureForError ());
+ }
+
return false;
}
- if (!Parent.IsClsComplianceRequired ()) {
+ if (Parent.Parent != null && !Parent.IsClsComplianceRequired ()) {
Attribute a = OptAttributes.Search (PredefinedAttributes.Get.CLSCompliant);
- Report.Warning (3018, 1, a.Location, "`{0}' cannot be marked as CLS-compliant because it is a member of non CLS-compliant type `{1}'",
+ Report.Warning (3018, 1, a.Location, "`{0}' cannot be marked as CLS-compliant because it is a member of non CLS-compliant type `{1}'",
GetSignatureForError (), Parent.GetSignatureForError ());
return false;
}
+ } else {
+ if (!IsExposedFromAssembly ())
+ return false;
+
+ if (!Parent.PartialContainer.IsClsComplianceRequired ())
+ return false;
}
if (member_name.Name [0] == '_') {
Report.Warning (3008, 1, Location, "Identifier `{0}' is not CLS-compliant", GetSignatureForError () );
}
+
return true;
}
@@ -812,7 +824,7 @@ namespace Mono.CSharp {
internal virtual void GenerateDocComment (DeclSpace ds)
{
try {
- DocUtil.GenerateDocComment (this, ds);
+ DocUtil.GenerateDocComment (this, ds, Report);
} catch (Exception e) {
throw new InternalErrorException (this, e);
}
@@ -820,25 +832,29 @@ namespace Mono.CSharp {
#region IMemberContext Members
- public virtual Type CurrentType {
+ public virtual CompilerContext Compiler {
+ get { return Parent.Compiler; }
+ }
+
+ public virtual TypeSpec CurrentType {
get { return Parent.CurrentType; }
}
- public virtual TypeContainer CurrentTypeDefinition {
- get { return Parent.CurrentTypeDefinition; }
+ public MemberCore CurrentMemberDefinition {
+ get { return this; }
}
public virtual TypeParameter[] CurrentTypeParameters {
get { return null; }
}
- public DeclSpace DeclContainer {
- get { return Parent; }
+ public virtual bool HasUnresolvedConstraints {
+ get { return false; }
}
public bool IsObsolete {
get {
- if (GetObsoleteAttribute () != null)
+ if (GetAttributeObsolete () != null)
return true;
return Parent == null ? false : Parent.IsObsolete;
@@ -855,20 +871,266 @@ namespace Mono.CSharp {
}
public bool IsStatic {
- get { return (ModFlags & Modifiers.STATIC) != 0; }
+ get {
+ return (ModFlags & Modifiers.STATIC) != 0;
+ }
}
#endregion
}
- ///
- /// Base class for structs, classes, enumerations and interfaces.
- ///
- ///
- /// They all create new declaration spaces. This
- /// provides the common foundation for managing those name
- /// spaces.
- ///
+ //
+ // Base member specification. A member specification contains
+ // member details which can alter in the context (e.g. generic instances)
+ //
+ public abstract class MemberSpec
+ {
+ [Flags]
+ protected enum StateFlags
+ {
+ Obsolete_Undetected = 1, // Obsolete attribute has not been detected yet
+ Obsolete = 1 << 1, // Member has obsolete attribute
+ CLSCompliant_Undetected = 1 << 3, // CLSCompliant attribute has not been detected yet
+ CLSCompliant = 1 << 4, // Member is CLS Compliant
+
+ IsAccessor = 1 << 9, // Method is an accessor
+ IsGeneric = 1 << 10, // Member contains type arguments
+
+ PendingMetaInflate = 1 << 12,
+ PendingMakeMethod = 1 << 13,
+ PendingMemberCacheMembers = 1 << 14,
+ PendingBaseTypeInflate = 1 << 15,
+ InterfacesExpanded = 1 << 16,
+ IsNotRealProperty = 1 << 17,
+ }
+
+ protected Modifiers modifiers;
+ protected StateFlags state;
+ protected IMemberDefinition definition;
+ public readonly MemberKind Kind;
+ protected TypeSpec declaringType;
+
+#if DEBUG
+ static int counter;
+ public int ID = counter++;
+#endif
+
+ protected MemberSpec (MemberKind kind, TypeSpec declaringType, IMemberDefinition definition, Modifiers modifiers)
+ {
+ this.Kind = kind;
+ this.declaringType = declaringType;
+ this.definition = definition;
+ this.modifiers = modifiers;
+
+ state = StateFlags.Obsolete_Undetected | StateFlags.CLSCompliant_Undetected;
+ }
+
+ #region Properties
+
+ public Assembly Assembly {
+ get {
+ return definition.Assembly;
+ }
+ }
+
+ public virtual int Arity {
+ get {
+ return 0;
+ }
+ }
+
+ public TypeSpec DeclaringType {
+ get {
+ return declaringType;
+ }
+ set {
+ declaringType = value;
+ }
+ }
+
+ public IMemberDefinition MemberDefinition {
+ get {
+ return definition;
+ }
+ }
+
+ public Modifiers Modifiers {
+ get {
+ return modifiers;
+ }
+ set {
+ modifiers = value;
+ }
+ }
+
+ public virtual string Name {
+ get {
+ return definition.Name;
+ }
+ }
+
+ public bool IsAbstract {
+ get { return (modifiers & Modifiers.ABSTRACT) != 0; }
+ }
+
+ public bool IsAccessor {
+ get {
+ return (state & StateFlags.IsAccessor) != 0;
+ }
+ set {
+ state = value ? state | StateFlags.IsAccessor : state & ~StateFlags.IsAccessor;
+ }
+ }
+
+ //
+ // Return true when this member is a generic in C# terms
+ // A nested non-generic type of generic type will return false
+ //
+ public bool IsGeneric {
+ get {
+ return (state & StateFlags.IsGeneric) != 0;
+ }
+ set {
+ state = value ? state | StateFlags.IsGeneric : state & ~StateFlags.IsGeneric;
+ }
+ }
+
+ public bool IsPrivate {
+ get { return (modifiers & Modifiers.PRIVATE) != 0; }
+ }
+
+ public bool IsStatic {
+ get {
+ return (modifiers & Modifiers.STATIC) != 0;
+ }
+ }
+
+ #endregion
+
+ public virtual ObsoleteAttribute GetAttributeObsolete ()
+ {
+ if ((state & (StateFlags.Obsolete | StateFlags.Obsolete_Undetected)) == 0)
+ return null;
+
+ state &= ~StateFlags.Obsolete_Undetected;
+
+ var oa = definition.GetAttributeObsolete ();
+ if (oa != null)
+ state |= StateFlags.Obsolete;
+
+ return oa;
+ }
+
+ protected virtual bool IsNotCLSCompliant ()
+ {
+ return MemberDefinition.IsNotCLSCompliant ();
+ }
+
+ public virtual string GetSignatureForError ()
+ {
+ var bf = MemberDefinition as Property.BackingField;
+ var name = bf == null ? Name : bf.OriginalName;
+ return DeclaringType.GetSignatureForError () + "." + name;
+ }
+
+ public virtual MemberSpec InflateMember (TypeParameterInflator inflator)
+ {
+ var inflated = (MemberSpec) MemberwiseClone ();
+ inflated.declaringType = inflator.TypeInstance;
+ inflated.state |= StateFlags.PendingMetaInflate;
+#if DEBUG
+ if (inflated.ID > 0)
+ inflated.ID = -inflated.ID;
+#endif
+ return inflated;
+ }
+
+ //
+ // Returns member CLS compliance based on full member hierarchy
+ //
+ public bool IsCLSCompliant ()
+ {
+ if ((state & StateFlags.CLSCompliant_Undetected) != 0) {
+ state &= ~StateFlags.CLSCompliant_Undetected;
+
+ if (IsNotCLSCompliant ())
+ return false;
+
+ bool compliant;
+ if (DeclaringType != null) {
+ compliant = DeclaringType.IsCLSCompliant ();
+ } else {
+ // TODO: NEED AssemblySpec
+ if (MemberDefinition.IsImported) {
+ var attr = MemberDefinition.Assembly.GetCustomAttributes (typeof (CLSCompliantAttribute), false);
+ compliant = attr.Length > 0 && ((CLSCompliantAttribute) attr[0]).IsCompliant;
+ } else {
+ compliant = CodeGen.Assembly.IsClsCompliant;
+ }
+ }
+
+ if (compliant)
+ state |= StateFlags.CLSCompliant;
+ }
+
+ return (state & StateFlags.CLSCompliant) != 0;
+ }
+
+ public bool IsConditionallyExcluded (Location loc)
+ {
+ if ((Kind & (MemberKind.Class | MemberKind.Method)) == 0)
+ return false;
+
+ var conditions = MemberDefinition.ConditionalConditions ();
+ if (conditions == null)
+ return false;
+
+ foreach (var condition in conditions) {
+ if (loc.CompilationUnit.IsConditionalDefined (condition))
+ return false;
+ }
+
+ return true;
+ }
+
+ public override string ToString ()
+ {
+ return GetSignatureForError ();
+ }
+ }
+
+ //
+ // Member details which are same between all member
+ // specifications
+ //
+ public interface IMemberDefinition
+ {
+ Assembly Assembly { get; }
+ string Name { get; }
+ bool IsImported { get; }
+
+ string[] ConditionalConditions ();
+ ObsoleteAttribute GetAttributeObsolete ();
+ bool IsNotCLSCompliant ();
+ void SetIsAssigned ();
+ void SetIsUsed ();
+ }
+
+ public interface IParametersMember : IInterfaceMemberSpec
+ {
+ AParametersCollection Parameters { get; }
+ }
+
+ public interface IInterfaceMemberSpec
+ {
+ TypeSpec MemberType { get; }
+ }
+
+ //
+ // Base type container declaration. It exists to handle partial types
+ // which share same definition (PartialContainer) but have different
+ // resolve scopes
+ //
public abstract class DeclSpace : MemberCore {
///
/// This points to the actual definition that is being
@@ -876,24 +1138,17 @@ namespace Mono.CSharp {
///
public TypeBuilder TypeBuilder;
- ///
- /// If we are a generic type, this is the type we are
- /// currently defining. We need to lookup members on this
- /// instead of the TypeBuilder.
- ///
- protected Type currentType;
-
//
// This is the namespace in which this typecontainer
// was declared. We use this to resolve names.
//
public NamespaceEntry NamespaceEntry;
- private Hashtable Cache = new Hashtable ();
+ private Dictionary Cache = new Dictionary ();
public readonly string Basename;
- protected Hashtable defined_names;
+ protected Dictionary defined_names;
public TypeContainer PartialContainer;
@@ -924,7 +1179,7 @@ namespace Mono.CSharp {
{
NamespaceEntry = ns;
Basename = name.Basename;
- defined_names = new Hashtable ();
+ defined_names = new Dictionary ();
PartialContainer = null;
if (name.TypeArguments != null) {
is_generic = true;
@@ -939,9 +1194,8 @@ namespace Mono.CSharp {
///
protected virtual bool AddToContainer (MemberCore symbol, string name)
{
- MemberCore mc = (MemberCore) defined_names [name];
-
- if (mc == null) {
+ MemberCore mc;
+ if (!defined_names.TryGetValue (name, out mc)) {
defined_names.Add (name, symbol);
return true;
}
@@ -952,6 +1206,10 @@ namespace Mono.CSharp {
if (symbol.EnableOverloadChecks (mc))
return true;
+ InterfaceMemberBase im = mc as InterfaceMemberBase;
+ if (im != null && im.IsExplicitImpl)
+ return true;
+
Report.SymbolRelatedToPreviousError (mc);
if ((mc.ModFlags & Modifiers.PARTIAL) != 0 && (symbol is ClassOrStruct || symbol is Interface)) {
Error_MissingPartialModifier (symbol);
@@ -986,13 +1244,11 @@ namespace Mono.CSharp {
///
public MemberCore GetDefinition (string name)
{
- return (MemberCore)defined_names [name];
- }
-
- public bool IsStaticClass {
- get { return (ModFlags & Modifiers.STATIC) != 0; }
+ MemberCore mc = null;
+ defined_names.TryGetValue (name, out mc);
+ return mc;
}
-
+
//
// root_types contains all the types. All TopLevel types
// hence have a parent that points to `root_types', that is
@@ -1007,27 +1263,6 @@ namespace Mono.CSharp {
return false;
}
- public virtual void CloseType ()
- {
- if ((caching_flags & Flags.CloseTypeCreated) == 0){
- try {
- TypeBuilder.CreateType ();
- } catch {
- //
- // The try/catch is needed because
- // nested enumerations fail to load when they
- // are defined.
- //
- // Even if this is the right order (enumerations
- // declared after types).
- //
- // Note that this still creates the type and
- // it is possible to save it
- }
- caching_flags |= Flags.CloseTypeCreated;
- }
- }
-
protected virtual TypeAttributes TypeAttr {
get { return Module.DefaultCharSetType; }
}
@@ -1044,171 +1279,71 @@ namespace Mono.CSharp {
type.GetSignatureForError ());
}
- public override void Emit ()
+ public override string GetSignatureForError ()
{
- if (type_params != null) {
- int offset = count_type_params - type_params.Length;
- for (int i = offset; i < type_params.Length; i++)
- CurrentTypeParameters [i - offset].Emit ();
- }
-
- if ((ModFlags & Modifiers.COMPILER_GENERATED) != 0 && !Parent.IsCompilerGenerated)
- PredefinedAttributes.Get.CompilerGenerated.EmitAttribute (TypeBuilder);
-
- base.Emit ();
- }
-
- public override string GetSignatureForError ()
- {
return MemberName.GetSignatureForError ();
}
- public bool CheckAccessLevel (Type check_type)
+ public bool CheckAccessLevel (TypeSpec check_type)
{
- Type tb = TypeBuilder;
-
- if (this is GenericMethod) {
- tb = Parent.TypeBuilder;
-
- // FIXME: Generic container does not work with nested generic
- // anonymous method stories
- if (TypeBuilder == null)
- return true;
- }
-
- check_type = TypeManager.DropGenericTypeArguments (check_type);
- if (check_type == tb)
- return true;
-
- // TODO: When called from LocalUsingAliasEntry tb is null
- // because we are in RootDeclSpace
- if (tb == null)
- tb = typeof (RootDeclSpace);
-
- //
- // Broken Microsoft runtime, return public for arrays, no matter what
- // the accessibility is for their underlying class, and they return
- // NonPublic visibility for pointers
- //
- if (TypeManager.HasElementType (check_type))
- return CheckAccessLevel (TypeManager.GetElementType (check_type));
-
- if (TypeManager.IsGenericParameter (check_type))
- return true;
+ TypeSpec tb = PartialContainer.Definition;
+ check_type = check_type.GetDefinition ();
- TypeAttributes check_attr = check_type.Attributes & TypeAttributes.VisibilityMask;
+ var check_attr = check_type.Modifiers & Modifiers.AccessibilityMask;
switch (check_attr){
- case TypeAttributes.Public:
+ case Modifiers.PUBLIC:
return true;
- case TypeAttributes.NotPublic:
- return TypeManager.IsThisOrFriendAssembly (check_type.Assembly);
+ case Modifiers.INTERNAL:
+ return TypeManager.IsThisOrFriendAssembly (Assembly, check_type.Assembly);
- case TypeAttributes.NestedPublic:
- return CheckAccessLevel (check_type.DeclaringType);
-
- case TypeAttributes.NestedPrivate:
- Type declaring = check_type.DeclaringType;
- return tb == declaring || TypeManager.IsNestedChildOf (tb, declaring);
+ case Modifiers.PRIVATE:
+ TypeSpec declaring = check_type.DeclaringType;
+ return tb == declaring.GetDefinition () || TypeManager.IsNestedChildOf (tb, declaring);
- case TypeAttributes.NestedFamily:
+ case Modifiers.PROTECTED:
//
// Only accessible to methods in current type or any subtypes
//
- return FamilyAccessible (tb, check_type);
+ return TypeManager.IsNestedFamilyAccessible (tb, check_type.DeclaringType);
- case TypeAttributes.NestedFamANDAssem:
- return TypeManager.IsThisOrFriendAssembly (check_type.Assembly) &&
- FamilyAccessible (tb, check_type);
-
- case TypeAttributes.NestedFamORAssem:
- return FamilyAccessible (tb, check_type) ||
- TypeManager.IsThisOrFriendAssembly (check_type.Assembly);
-
- case TypeAttributes.NestedAssembly:
- return TypeManager.IsThisOrFriendAssembly (check_type.Assembly);
- }
-
- throw new NotImplementedException (check_attr.ToString ());
- }
-
- static bool FamilyAccessible (Type tb, Type check_type)
- {
- Type declaring = check_type.DeclaringType;
- return TypeManager.IsNestedFamilyAccessible (tb, declaring);
- }
-
- public bool IsBaseType (Type baseType)
- {
- if (TypeManager.IsInterfaceType (baseType))
- throw new NotImplementedException ();
-
- Type type = TypeBuilder;
- while (type != null) {
- if (TypeManager.IsEqual (type, baseType))
+ case Modifiers.PROTECTED | Modifiers.INTERNAL:
+ if (TypeManager.IsThisOrFriendAssembly (Assembly, check_type.Assembly))
return true;
- type = type.BaseType;
+ goto case Modifiers.PROTECTED;
}
- return false;
+ throw new NotImplementedException (check_attr.ToString ());
}
- private Type LookupNestedTypeInHierarchy (string name)
+ private TypeSpec LookupNestedTypeInHierarchy (string name, int arity)
{
- Type t = null;
- // if the member cache has been created, lets use it.
- // the member cache is MUCH faster.
- if (MemberCache != null) {
- t = MemberCache.FindNestedType (name);
- if (t == null)
- return null;
-
- //
- // FIXME: This hack is needed because member cache does not work
- // with nested base generic types, it does only type name copy and
- // not type construction
- //
-#if !GMCS_SOURCE
- return t;
-#endif
- }
-
- // no member cache. Do it the hard way -- reflection
- for (Type current_type = TypeBuilder;
- current_type != null && current_type != TypeManager.object_type;
- current_type = current_type.BaseType) {
-
- Type ct = TypeManager.DropGenericTypeArguments (current_type);
- if (ct is TypeBuilder) {
- TypeContainer tc = ct == TypeBuilder
- ? PartialContainer : TypeManager.LookupTypeContainer (ct);
- if (tc != null)
- t = tc.FindNestedType (name);
- } else {
- t = TypeManager.GetNestedType (ct, name);
- }
+ // TODO: GenericMethod only
+ if (PartialContainer == null)
+ return null;
- if ((t == null) || !CheckAccessLevel (t))
- continue;
+ // Has any nested type
+ // Does not work, because base type can have
+ //if (PartialContainer.Types == null)
+ // return null;
- if (!TypeManager.IsGenericType (current_type))
- return t;
+ var container = PartialContainer.CurrentType;
- Type[] args = TypeManager.GetTypeArguments (current_type);
- Type[] targs = TypeManager.GetTypeArguments (t);
- for (int i = 0; i < args.Length; i++)
- targs [i] = TypeManager.TypeToCoreType (args [i]);
+ // Is not Root container
+ if (container == null)
+ return null;
-#if GMCS_SOURCE
- t = t.MakeGenericType (targs);
-#endif
+ var t = MemberCache.FindNestedType (container, name, arity);
+ if (t == null)
+ return null;
- return t;
- }
+ // FIXME: Breaks error reporting
+ if (!CheckAccessLevel (t))
+ return null;
- return null;
+ return t;
}
//
@@ -1218,72 +1353,50 @@ namespace Mono.CSharp {
//
// Returns: Type or null if they type can not be found.
//
- public override FullNamedExpression LookupNamespaceOrType (string name, Location loc, bool ignore_cs0104)
+ public override FullNamedExpression LookupNamespaceOrType (string name, int arity, Location loc, bool ignore_cs0104)
{
- if (Cache.Contains (name))
- return (FullNamedExpression) Cache [name];
+ FullNamedExpression e;
+ if (arity == 0 && Cache.TryGetValue (name, out e))
+ return e;
- FullNamedExpression e = null;
+ e = null;
int errors = Report.Errors;
- TypeParameter[] tp = CurrentTypeParameters;
- if (tp != null) {
- TypeParameter tparam = TypeParameter.FindTypeParameter (tp, name);
- if (tparam != null)
- e = new TypeParameterExpr (tparam, Location.Null);
+ if (arity == 0) {
+ TypeParameter[] tp = CurrentTypeParameters;
+ if (tp != null) {
+ TypeParameter tparam = TypeParameter.FindTypeParameter (tp, name);
+ if (tparam != null)
+ e = new TypeParameterExpr (tparam, Location.Null);
+ }
}
if (e == null) {
- Type t = LookupNestedTypeInHierarchy (name);
+ TypeSpec t = LookupNestedTypeInHierarchy (name, arity);
if (t != null)
e = new TypeExpression (t, Location.Null);
else if (Parent != null)
- e = Parent.LookupNamespaceOrType (name, loc, ignore_cs0104);
+ e = Parent.LookupNamespaceOrType (name, arity, loc, ignore_cs0104);
else
- e = NamespaceEntry.LookupNamespaceOrType (name, loc, ignore_cs0104);
+ e = NamespaceEntry.LookupNamespaceOrType (name, arity, loc, ignore_cs0104);
}
- if (errors == Report.Errors)
+ // TODO MemberCache: How to cache arity stuff ?
+ if (errors == Report.Errors && arity == 0)
Cache [name] = e;
return e;
}
- ///
- /// This function is broken and not what you're looking for. It should only
- /// be used while the type is still being created since it doesn't use the cache
- /// and relies on the filter doing the member name check.
- ///
- ///
- // [Obsolete ("Only MemberCache approach should be used")]
- public virtual MemberList FindMembers (MemberTypes mt, BindingFlags bf,
- MemberFilter filter, object criteria)
- {
- throw new NotSupportedException ();
- }
-
- ///
- /// If we have a MemberCache, return it. This property may return null if the
- /// class doesn't have a member cache or while it's still being created.
- ///
- public abstract MemberCache MemberCache {
- get;
+ public override Assembly Assembly {
+ get { return Module.Assembly; }
}
public virtual ModuleContainer Module {
get { return Parent.Module; }
}
- public override void ApplyAttributeBuilder (Attribute a, CustomAttributeBuilder cb, PredefinedAttributes pa)
- {
- if (a.Type == pa.Required) {
- Report.Error (1608, a.Location, "The RequiredAttribute attribute is not permitted on C# types");
- return;
- }
- TypeBuilder.SetCustomAttribute (cb);
- }
-
TypeParameter[] initialize_type_params ()
{
if (type_param_list != null)
@@ -1293,7 +1406,7 @@ namespace Mono.CSharp {
if (this is GenericMethod)
the_parent = null;
- ArrayList list = new ArrayList ();
+ var list = new List ();
if (the_parent != null && the_parent.IsGeneric) {
// FIXME: move generics info out of DeclSpace
TypeParameter[] parent_params = the_parent.TypeParameters;
@@ -1322,7 +1435,7 @@ namespace Mono.CSharp {
return type_param_list;
}
- public virtual void SetParameterInfo (ArrayList constraints_list)
+ public virtual void SetParameterInfo (List constraints_list)
{
if (!is_generic) {
if (constraints_list != null) {
@@ -1354,7 +1467,7 @@ namespace Mono.CSharp {
--total;
continue;
}
- if (constraints_at.TypeParameter == name.Name) {
+ if (constraints_at.TypeParameter.Value == name.Name) {
constraints = constraints_at;
constraints_list.RemoveAt(ii);
break;
@@ -1369,7 +1482,7 @@ namespace Mono.CSharp {
}
type_params [i] = new TypeParameter (
- Parent, this, name.Name, constraints, name.OptAttributes, variance, Location);
+ Parent, i, new MemberName (name.Name, Location), constraints, name.OptAttributes, variance);
AddToContainer (type_params [i], name.Name);
}
@@ -1377,12 +1490,12 @@ namespace Mono.CSharp {
if (constraints_list != null && constraints_list.Count > 0) {
foreach (Constraints constraint in constraints_list) {
Report.Error(699, constraint.Location, "`{0}': A constraint references nonexistent type parameter `{1}'",
- GetSignatureForError (), constraint.TypeParameter);
+ GetSignatureForError (), constraint.TypeParameter.Value);
}
}
}
- public TypeParameter[] TypeParameters {
+ protected TypeParameter[] TypeParameters {
get {
if (!IsGeneric)
throw new InvalidOperationException ();
@@ -1395,26 +1508,12 @@ namespace Mono.CSharp {
}
}
- public override Type CurrentType {
- get { return currentType != null ? currentType : TypeBuilder; }
- }
-
- public override TypeContainer CurrentTypeDefinition {
- get { return PartialContainer; }
- }
-
public int CountTypeParameters {
get {
return count_type_params;
}
}
- // Used for error reporting only
- public virtual Type LookupAnyGeneric (string typeName)
- {
- return NamespaceEntry.NS.LookForAnyGenericType (typeName);
- }
-
public override string[] ValidAttributeTargets {
get { return attribute_targets; }
}
@@ -1427,1402 +1526,9 @@ namespace Mono.CSharp {
if (type_params != null) {
foreach (TypeParameter tp in type_params) {
- if (tp.Constraints == null)
- continue;
-
- tp.Constraints.VerifyClsCompliance ();
- }
- }
-
- IDictionary cache = TypeManager.AllClsTopLevelTypes;
- if (cache == null)
- return true;
-
- string lcase = Name.ToLower (System.Globalization.CultureInfo.InvariantCulture);
- if (!cache.Contains (lcase)) {
- cache.Add (lcase, this);
- return true;
- }
-
- object val = cache [lcase];
- if (val == null) {
- Type t = AttributeTester.GetImportedIgnoreCaseClsType (lcase);
- if (t == null)
- return true;
- Report.SymbolRelatedToPreviousError (t);
- }
- else {
- Report.SymbolRelatedToPreviousError ((DeclSpace)val);
- }
-
- Report.Warning (3005, 1, Location, "Identifier `{0}' differing only in case is not CLS-compliant", GetSignatureForError ());
- return true;
- }
- }
-
- ///
- /// This is a readonly list of MemberInfo's.
- ///
- public class MemberList : IList {
- public readonly IList List;
- int count;
-
- ///
- /// Create a new MemberList from the given IList.
- ///
- public MemberList (IList list)
- {
- if (list != null)
- this.List = list;
- else
- this.List = new ArrayList ();
- count = List.Count;
- }
-
- ///
- /// Concatenate the ILists `first' and `second' to a new MemberList.
- ///
- public MemberList (IList first, IList second)
- {
- ArrayList list = new ArrayList ();
- list.AddRange (first);
- list.AddRange (second);
- count = list.Count;
- List = list;
- }
-
- public static readonly MemberList Empty = new MemberList (new ArrayList (0));
-
- ///
- /// Cast the MemberList into a MemberInfo[] array.
- ///
- ///
- /// This is an expensive operation, only use it if it's really necessary.
- ///
- public static explicit operator MemberInfo [] (MemberList list)
- {
- Timer.StartTimer (TimerType.MiscTimer);
- MemberInfo [] result = new MemberInfo [list.Count];
- list.CopyTo (result, 0);
- Timer.StopTimer (TimerType.MiscTimer);
- return result;
- }
-
- // ICollection
-
- public int Count {
- get {
- return count;
- }
- }
-
- public bool IsSynchronized {
- get {
- return List.IsSynchronized;
- }
- }
-
- public object SyncRoot {
- get {
- return List.SyncRoot;
- }
- }
-
- public void CopyTo (Array array, int index)
- {
- List.CopyTo (array, index);
- }
-
- // IEnumerable
-
- public IEnumerator GetEnumerator ()
- {
- return List.GetEnumerator ();
- }
-
- // IList
-
- public bool IsFixedSize {
- get {
- return true;
- }
- }
-
- public bool IsReadOnly {
- get {
- return true;
- }
- }
-
- object IList.this [int index] {
- get {
- return List [index];
- }
-
- set {
- throw new NotSupportedException ();
- }
- }
-
- // FIXME: try to find out whether we can avoid the cast in this indexer.
- public MemberInfo this [int index] {
- get {
- return (MemberInfo) List [index];
- }
- }
-
- public int Add (object value)
- {
- throw new NotSupportedException ();
- }
-
- public void Clear ()
- {
- throw new NotSupportedException ();
- }
-
- public bool Contains (object value)
- {
- return List.Contains (value);
- }
-
- public int IndexOf (object value)
- {
- return List.IndexOf (value);
- }
-
- public void Insert (int index, object value)
- {
- throw new NotSupportedException ();
- }
-
- public void Remove (object value)
- {
- throw new NotSupportedException ();
- }
-
- public void RemoveAt (int index)
- {
- throw new NotSupportedException ();
- }
- }
-
- ///
- /// This interface is used to get all members of a class when creating the
- /// member cache. It must be implemented by all DeclSpace derivatives which
- /// want to support the member cache and by TypeHandle to get caching of
- /// non-dynamic types.
- ///
- public interface IMemberContainer {
- ///
- /// The name of the IMemberContainer. This is only used for
- /// debugging purposes.
- ///
- string Name {
- get;
- }
-
- ///
- /// The type of this IMemberContainer.
- ///
- Type Type {
- get;
- }
-
- ///
- /// Returns the IMemberContainer of the base class or null if this
- /// is an interface or TypeManger.object_type.
- /// This is used when creating the member cache for a class to get all
- /// members from the base class.
- ///
- MemberCache BaseCache {
- get;
- }
-
- ///
- /// Whether this is an interface.
- ///
- bool IsInterface {
- get;
- }
-
- ///
- /// Returns all members of this class with the corresponding MemberTypes
- /// and BindingFlags.
- ///
- ///
- /// When implementing this method, make sure not to return any inherited
- /// members and check the MemberTypes and BindingFlags properly.
- /// Unfortunately, System.Reflection is lame and doesn't provide a way to
- /// get the BindingFlags (static/non-static,public/non-public) in the
- /// MemberInfo class, but the cache needs this information. That's why
- /// this method is called multiple times with different BindingFlags.
- ///
- MemberList GetMembers (MemberTypes mt, BindingFlags bf);
- }
-
- ///
- /// The MemberCache is used by dynamic and non-dynamic types to speed up
- /// member lookups. It has a member name based hash table; it maps each member
- /// name to a list of CacheEntry objects. Each CacheEntry contains a MemberInfo
- /// and the BindingFlags that were initially used to get it. The cache contains
- /// all members of the current class and all inherited members. If this cache is
- /// for an interface types, it also contains all inherited members.
- ///
- /// There are two ways to get a MemberCache:
- /// * if this is a dynamic type, lookup the corresponding DeclSpace and then
- /// use the DeclSpace.MemberCache property.
- /// * if this not a dynamic type, call TypeHandle.GetTypeHandle() to get a
- /// TypeHandle instance for the type and then use TypeHandle.MemberCache.
- ///
- public class MemberCache {
- public readonly IMemberContainer Container;
- protected Hashtable member_hash;
- protected Hashtable method_hash;
-
- ///
- /// Create a new MemberCache for the given IMemberContainer `container'.
- ///
- public MemberCache (IMemberContainer container)
- {
- this.Container = container;
-
- Timer.IncrementCounter (CounterType.MemberCache);
- Timer.StartTimer (TimerType.CacheInit);
-
- // If we have a base class (we have a base class unless we're
- // TypeManager.object_type), we deep-copy its MemberCache here.
- if (Container.BaseCache != null)
- member_hash = SetupCache (Container.BaseCache);
- else
- member_hash = new Hashtable ();
-
- // If this is neither a dynamic type nor an interface, create a special
- // method cache with all declared and inherited methods.
- Type type = container.Type;
- if (!(type is TypeBuilder) && !type.IsInterface &&
- // !(type.IsGenericType && (type.GetGenericTypeDefinition () is TypeBuilder)) &&
- !TypeManager.IsGenericType (type) && !TypeManager.IsGenericParameter (type) &&
- (Container.BaseCache == null || Container.BaseCache.method_hash != null)) {
- method_hash = new Hashtable ();
- AddMethods (type);
- }
-
- // Add all members from the current class.
- AddMembers (Container);
-
- Timer.StopTimer (TimerType.CacheInit);
- }
-
- public MemberCache (Type baseType, IMemberContainer container)
- {
- this.Container = container;
- if (baseType == null)
- this.member_hash = new Hashtable ();
- else
- this.member_hash = SetupCache (TypeManager.LookupMemberCache (baseType));
- }
-
- public MemberCache (Type[] ifaces)
- {
- //
- // The members of this cache all belong to other caches.
- // So, 'Container' will not be used.
- //
- this.Container = null;
-
- member_hash = new Hashtable ();
- if (ifaces == null)
- return;
-
- foreach (Type itype in ifaces)
- AddCacheContents (TypeManager.LookupMemberCache (itype));
- }
-
- public MemberCache (IMemberContainer container, Type base_class, Type[] ifaces)
- {
- this.Container = container;
-
- // If we have a base class (we have a base class unless we're
- // TypeManager.object_type), we deep-copy its MemberCache here.
- if (Container.BaseCache != null)
- member_hash = SetupCache (Container.BaseCache);
- else
- member_hash = new Hashtable ();
-
- if (base_class != null)
- AddCacheContents (TypeManager.LookupMemberCache (base_class));
- if (ifaces != null) {
- foreach (Type itype in ifaces) {
- MemberCache cache = TypeManager.LookupMemberCache (itype);
- if (cache != null)
- AddCacheContents (cache);
- }
- }
- }
-
- ///
- /// Bootstrap this member cache by doing a deep-copy of our base.
- ///
- static Hashtable SetupCache (MemberCache base_class)
- {
- if (base_class == null)
- return new Hashtable ();
-
- Hashtable hash = new Hashtable (base_class.member_hash.Count);
- IDictionaryEnumerator it = base_class.member_hash.GetEnumerator ();
- while (it.MoveNext ()) {
- hash.Add (it.Key, ((ArrayList) it.Value).Clone ());
- }
-
- return hash;
- }
-
- //
- // Converts ModFlags to BindingFlags
- //
- static BindingFlags GetBindingFlags (int modifiers)
- {
- BindingFlags bf;
- if ((modifiers & Modifiers.STATIC) != 0)
- bf = BindingFlags.Static;
- else
- bf = BindingFlags.Instance;
-
- if ((modifiers & Modifiers.PRIVATE) != 0)
- bf |= BindingFlags.NonPublic;
- else
- bf |= BindingFlags.Public;
-
- return bf;
- }
-
- ///
- /// Add the contents of `cache' to the member_hash.
- ///
- void AddCacheContents (MemberCache cache)
- {
- IDictionaryEnumerator it = cache.member_hash.GetEnumerator ();
- while (it.MoveNext ()) {
- ArrayList list = (ArrayList) member_hash [it.Key];
- if (list == null)
- member_hash [it.Key] = list = new ArrayList ();
-
- ArrayList entries = (ArrayList) it.Value;
- for (int i = entries.Count-1; i >= 0; i--) {
- CacheEntry entry = (CacheEntry) entries [i];
-
- if (entry.Container != cache.Container)
- break;
- list.Add (entry);
+ tp.VerifyClsCompliance ();
}
}
- }
-
- ///
- /// Add all members from class `container' to the cache.
- ///
- void AddMembers (IMemberContainer container)
- {
- // We need to call AddMembers() with a single member type at a time
- // to get the member type part of CacheEntry.EntryType right.
- if (!container.IsInterface) {
- AddMembers (MemberTypes.Constructor, container);
- AddMembers (MemberTypes.Field, container);
- }
- AddMembers (MemberTypes.Method, container);
- AddMembers (MemberTypes.Property, container);
- AddMembers (MemberTypes.Event, container);
- // Nested types are returned by both Static and Instance searches.
- AddMembers (MemberTypes.NestedType,
- BindingFlags.Static | BindingFlags.Public, container);
- AddMembers (MemberTypes.NestedType,
- BindingFlags.Static | BindingFlags.NonPublic, container);
- }
-
- void AddMembers (MemberTypes mt, IMemberContainer container)
- {
- AddMembers (mt, BindingFlags.Static | BindingFlags.Public, container);
- AddMembers (mt, BindingFlags.Static | BindingFlags.NonPublic, container);
- AddMembers (mt, BindingFlags.Instance | BindingFlags.Public, container);
- AddMembers (mt, BindingFlags.Instance | BindingFlags.NonPublic, container);
- }
-
- public void AddMember (MemberInfo mi, MemberCore mc)
- {
- AddMember (mi.MemberType, GetBindingFlags (mc.ModFlags), Container, mi.Name, mi);
- }
-
- public void AddGenericMember (MemberInfo mi, InterfaceMemberBase mc)
- {
- AddMember (mi.MemberType, GetBindingFlags (mc.ModFlags), Container,
- MemberName.MakeName (mc.GetFullName (mc.MemberName), mc.MemberName.TypeArguments), mi);
- }
-
- public void AddNestedType (DeclSpace type)
- {
- AddMember (MemberTypes.NestedType, GetBindingFlags (type.ModFlags), (IMemberContainer) type.Parent,
- type.TypeBuilder.Name, type.TypeBuilder);
- }
-
- public void AddInterface (MemberCache baseCache)
- {
- if (baseCache.member_hash.Count > 0)
- AddCacheContents (baseCache);
- }
-
- void AddMember (MemberTypes mt, BindingFlags bf, IMemberContainer container,
- string name, MemberInfo member)
- {
- // We use a name-based hash table of ArrayList's.
- ArrayList list = (ArrayList) member_hash [name];
- if (list == null) {
- list = new ArrayList (1);
- member_hash.Add (name, list);
- }
-
- // When this method is called for the current class, the list will
- // already contain all inherited members from our base classes.
- // We cannot add new members in front of the list since this'd be an
- // expensive operation, that's why the list is sorted in reverse order
- // (ie. members from the current class are coming last).
- list.Add (new CacheEntry (container, member, mt, bf));
- }
-
- ///
- /// Add all members from class `container' with the requested MemberTypes and
- /// BindingFlags to the cache. This method is called multiple times with different
- /// MemberTypes and BindingFlags.
- ///
- void AddMembers (MemberTypes mt, BindingFlags bf, IMemberContainer container)
- {
- MemberList members = container.GetMembers (mt, bf);
-
- foreach (MemberInfo member in members) {
- string name = member.Name;
-
- AddMember (mt, bf, container, name, member);
-
- if (member is MethodInfo) {
- string gname = TypeManager.GetMethodName ((MethodInfo) member);
- if (gname != name)
- AddMember (mt, bf, container, gname, member);
- }
- }
- }
-
- ///
- /// Add all declared and inherited methods from class `type' to the method cache.
- ///
- void AddMethods (Type type)
- {
- AddMethods (BindingFlags.Static | BindingFlags.Public |
- BindingFlags.FlattenHierarchy, type);
- AddMethods (BindingFlags.Static | BindingFlags.NonPublic |
- BindingFlags.FlattenHierarchy, type);
- AddMethods (BindingFlags.Instance | BindingFlags.Public, type);
- AddMethods (BindingFlags.Instance | BindingFlags.NonPublic, type);
- }
-
- static ArrayList overrides = new ArrayList ();
-
- void AddMethods (BindingFlags bf, Type type)
- {
- MethodBase [] members = type.GetMethods (bf);
-
- Array.Reverse (members);
-
- foreach (MethodBase member in members) {
- string name = member.Name;
-
- // We use a name-based hash table of ArrayList's.
- ArrayList list = (ArrayList) method_hash [name];
- if (list == null) {
- list = new ArrayList (1);
- method_hash.Add (name, list);
- }
-
- MethodInfo curr = (MethodInfo) member;
- while (curr.IsVirtual && (curr.Attributes & MethodAttributes.NewSlot) == 0) {
- MethodInfo base_method = curr.GetBaseDefinition ();
-
- if (base_method == curr)
- // Not every virtual function needs to have a NewSlot flag.
- break;
-
- overrides.Add (curr);
- list.Add (new CacheEntry (null, base_method, MemberTypes.Method, bf));
- curr = base_method;
- }
-
- if (overrides.Count > 0) {
- for (int i = 0; i < overrides.Count; ++i)
- TypeManager.RegisterOverride ((MethodBase) overrides [i], curr);
- overrides.Clear ();
- }
-
- // Unfortunately, the elements returned by Type.GetMethods() aren't
- // sorted so we need to do this check for every member.
- BindingFlags new_bf = bf;
- if (member.DeclaringType == type)
- new_bf |= BindingFlags.DeclaredOnly;
-
- list.Add (new CacheEntry (Container, member, MemberTypes.Method, new_bf));
- }
- }
-
- ///
- /// Compute and return a appropriate `EntryType' magic number for the given
- /// MemberTypes and BindingFlags.
- ///
- protected static EntryType GetEntryType (MemberTypes mt, BindingFlags bf)
- {
- EntryType type = EntryType.None;
-
- if ((mt & MemberTypes.Constructor) != 0)
- type |= EntryType.Constructor;
- if ((mt & MemberTypes.Event) != 0)
- type |= EntryType.Event;
- if ((mt & MemberTypes.Field) != 0)
- type |= EntryType.Field;
- if ((mt & MemberTypes.Method) != 0)
- type |= EntryType.Method;
- if ((mt & MemberTypes.Property) != 0)
- type |= EntryType.Property;
- // Nested types are returned by static and instance searches.
- if ((mt & MemberTypes.NestedType) != 0)
- type |= EntryType.NestedType | EntryType.Static | EntryType.Instance;
-
- if ((bf & BindingFlags.Instance) != 0)
- type |= EntryType.Instance;
- if ((bf & BindingFlags.Static) != 0)
- type |= EntryType.Static;
- if ((bf & BindingFlags.Public) != 0)
- type |= EntryType.Public;
- if ((bf & BindingFlags.NonPublic) != 0)
- type |= EntryType.NonPublic;
- if ((bf & BindingFlags.DeclaredOnly) != 0)
- type |= EntryType.Declared;
-
- return type;
- }
-
- ///
- /// The `MemberTypes' enumeration type is a [Flags] type which means that it may
- /// denote multiple member types. Returns true if the given flags value denotes a
- /// single member types.
- ///
- public static bool IsSingleMemberType (MemberTypes mt)
- {
- switch (mt) {
- case MemberTypes.Constructor:
- case MemberTypes.Event:
- case MemberTypes.Field:
- case MemberTypes.Method:
- case MemberTypes.Property:
- case MemberTypes.NestedType:
- return true;
-
- default:
- return false;
- }
- }
-
- ///
- /// We encode the MemberTypes and BindingFlags of each members in a "magic"
- /// number to speed up the searching process.
- ///
- [Flags]
- protected enum EntryType {
- None = 0x000,
-
- Instance = 0x001,
- Static = 0x002,
- MaskStatic = Instance|Static,
-
- Public = 0x004,
- NonPublic = 0x008,
- MaskProtection = Public|NonPublic,
-
- Declared = 0x010,
-
- Constructor = 0x020,
- Event = 0x040,
- Field = 0x080,
- Method = 0x100,
- Property = 0x200,
- NestedType = 0x400,
-
- NotExtensionMethod = 0x800,
-
- MaskType = Constructor|Event|Field|Method|Property|NestedType
- }
-
- protected class CacheEntry {
- public readonly IMemberContainer Container;
- public EntryType EntryType;
- public readonly MemberInfo Member;
-
- public CacheEntry (IMemberContainer container, MemberInfo member,
- MemberTypes mt, BindingFlags bf)
- {
- this.Container = container;
- this.Member = member;
- this.EntryType = GetEntryType (mt, bf);
- }
-
- public override string ToString ()
- {
- return String.Format ("CacheEntry ({0}:{1}:{2})", Container.Name,
- EntryType, Member);
- }
- }
-
- ///
- /// This is called each time we're walking up one level in the class hierarchy
- /// and checks whether we can abort the search since we've already found what
- /// we were looking for.
- ///
- protected bool DoneSearching (ArrayList list)
- {
- //
- // We've found exactly one member in the current class and it's not
- // a method or constructor.
- //
- if (list.Count == 1 && !(list [0] is MethodBase))
- return true;
-
- //
- // Multiple properties: we query those just to find out the indexer
- // name
- //
- if ((list.Count > 0) && (list [0] is PropertyInfo))
- return true;
-
- return false;
- }
-
- ///
- /// Looks up members with name `name'. If you provide an optional
- /// filter function, it'll only be called with members matching the
- /// requested member name.
- ///
- /// This method will try to use the cache to do the lookup if possible.
- ///
- /// Unlike other FindMembers implementations, this method will always
- /// check all inherited members - even when called on an interface type.
- ///
- /// If you know that you're only looking for methods, you should use
- /// MemberTypes.Method alone since this speeds up the lookup a bit.
- /// When doing a method-only search, it'll try to use a special method
- /// cache (unless it's a dynamic type or an interface) and the returned
- /// MemberInfo's will have the correct ReflectedType for inherited methods.
- /// The lookup process will automatically restart itself in method-only
- /// search mode if it discovers that it's about to return methods.
- ///
- ArrayList global = new ArrayList ();
- bool using_global = false;
-
- static MemberInfo [] emptyMemberInfo = new MemberInfo [0];
-
- public MemberInfo [] FindMembers (MemberTypes mt, BindingFlags bf, string name,
- MemberFilter filter, object criteria)
- {
- if (using_global)
- throw new Exception ();
-
- bool declared_only = (bf & BindingFlags.DeclaredOnly) != 0;
- bool method_search = mt == MemberTypes.Method;
- // If we have a method cache and we aren't already doing a method-only search,
- // then we restart a method search if the first match is a method.
- bool do_method_search = !method_search && (method_hash != null);
-
- ArrayList applicable;
-
- // If this is a method-only search, we try to use the method cache if
- // possible; a lookup in the method cache will return a MemberInfo with
- // the correct ReflectedType for inherited methods.
-
- if (method_search && (method_hash != null))
- applicable = (ArrayList) method_hash [name];
- else
- applicable = (ArrayList) member_hash [name];
-
- if (applicable == null)
- return emptyMemberInfo;
-
- //
- // 32 slots gives 53 rss/54 size
- // 2/4 slots gives 55 rss
- //
- // Strange: from 25,000 calls, only 1,800
- // are above 2. Why does this impact it?
- //
- global.Clear ();
- using_global = true;
-
- Timer.StartTimer (TimerType.CachedLookup);
-
- EntryType type = GetEntryType (mt, bf);
-
- IMemberContainer current = Container;
-
- bool do_interface_search = current.IsInterface;
-
- // `applicable' is a list of all members with the given member name `name'
- // in the current class and all its base classes. The list is sorted in
- // reverse order due to the way how the cache is initialy created (to speed
- // things up, we're doing a deep-copy of our base).
-
- for (int i = applicable.Count-1; i >= 0; i--) {
- CacheEntry entry = (CacheEntry) applicable [i];
-
- // This happens each time we're walking one level up in the class
- // hierarchy. If we're doing a DeclaredOnly search, we must abort
- // the first time this happens (this may already happen in the first
- // iteration of this loop if there are no members with the name we're
- // looking for in the current class).
- if (entry.Container != current) {
- if (declared_only)
- break;
-
- if (!do_interface_search && DoneSearching (global))
- break;
-
- current = entry.Container;
- }
-
- // Is the member of the correct type ?
- if ((entry.EntryType & type & EntryType.MaskType) == 0)
- continue;
-
- // Is the member static/non-static ?
- if ((entry.EntryType & type & EntryType.MaskStatic) == 0)
- continue;
-
- // Apply the filter to it.
- if (filter (entry.Member, criteria)) {
- if ((entry.EntryType & EntryType.MaskType) != EntryType.Method) {
- do_method_search = false;
- }
-
- // Because interfaces support multiple inheritance we have to be sure that
- // base member is from same interface, so only top level member will be returned
- if (do_interface_search && global.Count > 0) {
- bool member_already_exists = false;
-
- foreach (MemberInfo mi in global) {
- if (mi is MethodBase)
- continue;
-
- if (IsInterfaceBaseInterface (TypeManager.GetInterfaces (mi.DeclaringType), entry.Member.DeclaringType)) {
- member_already_exists = true;
- break;
- }
- }
- if (member_already_exists)
- continue;
- }
-
- global.Add (entry.Member);
- }
- }
-
- Timer.StopTimer (TimerType.CachedLookup);
-
- // If we have a method cache and we aren't already doing a method-only
- // search, we restart in method-only search mode if the first match is
- // a method. This ensures that we return a MemberInfo with the correct
- // ReflectedType for inherited methods.
- if (do_method_search && (global.Count > 0)){
- using_global = false;
-
- return FindMembers (MemberTypes.Method, bf, name, filter, criteria);
- }
-
- using_global = false;
- MemberInfo [] copy = new MemberInfo [global.Count];
- global.CopyTo (copy);
- return copy;
- }
-
- ///
- /// Returns true if iterface exists in any base interfaces (ifaces)
- ///
- static bool IsInterfaceBaseInterface (Type[] ifaces, Type ifaceToFind)
- {
- foreach (Type iface in ifaces) {
- if (iface == ifaceToFind)
- return true;
-
- Type[] base_ifaces = TypeManager.GetInterfaces (iface);
- if (base_ifaces.Length > 0 && IsInterfaceBaseInterface (base_ifaces, ifaceToFind))
- return true;
- }
- return false;
- }
-
- // find the nested type @name in @this.
- public Type FindNestedType (string name)
- {
- ArrayList applicable = (ArrayList) member_hash [name];
- if (applicable == null)
- return null;
-
- for (int i = applicable.Count-1; i >= 0; i--) {
- CacheEntry entry = (CacheEntry) applicable [i];
- if ((entry.EntryType & EntryType.NestedType & EntryType.MaskType) != 0)
- return (Type) entry.Member;
- }
-
- return null;
- }
-
- public MemberInfo FindBaseEvent (Type invocation_type, string name)
- {
- ArrayList applicable = (ArrayList) member_hash [name];
- if (applicable == null)
- return null;
-
- //
- // Walk the chain of events, starting from the top.
- //
- for (int i = applicable.Count - 1; i >= 0; i--)
- {
- CacheEntry entry = (CacheEntry) applicable [i];
- if ((entry.EntryType & EntryType.Event) == 0)
- continue;
-
- EventInfo ei = (EventInfo)entry.Member;
- return ei.GetAddMethod (true);
- }
-
- return null;
- }
-
- //
- // Looks for extension methods with defined name and extension type
- //
- public ArrayList FindExtensionMethods (Type extensionType, string name, bool publicOnly)
- {
- ArrayList entries;
- if (method_hash != null)
- entries = (ArrayList)method_hash [name];
- else
- entries = (ArrayList)member_hash [name];
-
- if (entries == null)
- return null;
-
- EntryType entry_type = EntryType.Static | EntryType.Method | EntryType.NotExtensionMethod;
- EntryType found_entry_type = entry_type & ~EntryType.NotExtensionMethod;
-
- ArrayList candidates = null;
- foreach (CacheEntry entry in entries) {
- if ((entry.EntryType & entry_type) == found_entry_type) {
- MethodBase mb = (MethodBase)entry.Member;
-
- // Simple accessibility check
- if ((entry.EntryType & EntryType.Public) == 0 && publicOnly) {
- MethodAttributes ma = mb.Attributes & MethodAttributes.MemberAccessMask;
- if (ma != MethodAttributes.Assembly && ma != MethodAttributes.FamORAssem)
- continue;
-
- if (!TypeManager.IsThisOrFriendAssembly (mb.DeclaringType.Assembly))
- continue;
- }
-
- IMethodData md = TypeManager.GetMethod (mb);
- AParametersCollection pd = md == null ?
- TypeManager.GetParameterData (mb) : md.ParameterInfo;
-
- Type ex_type = pd.ExtensionMethodType;
- if (ex_type == null) {
- entry.EntryType |= EntryType.NotExtensionMethod;
- continue;
- }
-
- //if (implicit conversion between ex_type and extensionType exist) {
- if (candidates == null)
- candidates = new ArrayList (2);
- candidates.Add (mb);
- //}
- }
- }
-
- return candidates;
- }
-
- //
- // This finds the method or property for us to override. invocation_type is the type where
- // the override is going to be declared, name is the name of the method/property, and
- // param_types is the parameters, if any to the method or property
- //
- // Because the MemberCache holds members from this class and all the base classes,
- // we can avoid tons of reflection stuff.
- //
- public MemberInfo FindMemberToOverride (Type invocation_type, string name, AParametersCollection parameters, GenericMethod generic_method, bool is_property)
- {
- ArrayList applicable;
- if (method_hash != null && !is_property)
- applicable = (ArrayList) method_hash [name];
- else
- applicable = (ArrayList) member_hash [name];
-
- if (applicable == null)
- return null;
- //
- // Walk the chain of methods, starting from the top.
- //
- for (int i = applicable.Count - 1; i >= 0; i--) {
- CacheEntry entry = (CacheEntry) applicable [i];
-
- if ((entry.EntryType & (is_property ? (EntryType.Property | EntryType.Field) : EntryType.Method)) == 0)
- continue;
-
- PropertyInfo pi = null;
- MethodInfo mi = null;
- FieldInfo fi = null;
- AParametersCollection cmp_attrs;
-
- if (is_property) {
- if ((entry.EntryType & EntryType.Field) != 0) {
- fi = (FieldInfo)entry.Member;
- cmp_attrs = ParametersCompiled.EmptyReadOnlyParameters;
- } else {
- pi = (PropertyInfo) entry.Member;
- cmp_attrs = TypeManager.GetParameterData (pi);
- }
- } else {
- mi = (MethodInfo) entry.Member;
- cmp_attrs = TypeManager.GetParameterData (mi);
- }
-
- if (fi != null) {
- // TODO: Almost duplicate !
- // Check visibility
- switch (fi.Attributes & FieldAttributes.FieldAccessMask) {
- case FieldAttributes.PrivateScope:
- continue;
- case FieldAttributes.Private:
- //
- // A private method is Ok if we are a nested subtype.
- // The spec actually is not very clear about this, see bug 52458.
- //
- if (!invocation_type.Equals (entry.Container.Type) &&
- !TypeManager.IsNestedChildOf (invocation_type, entry.Container.Type))
- continue;
- break;
- case FieldAttributes.FamANDAssem:
- case FieldAttributes.Assembly:
- //
- // Check for assembly methods
- //
- if (fi.DeclaringType.Assembly != CodeGen.Assembly.Builder)
- continue;
- break;
- }
- return entry.Member;
- }
-
- //
- // Check the arguments
- //
- if (cmp_attrs.Count != parameters.Count)
- continue;
-
- int j;
- for (j = 0; j < cmp_attrs.Count; ++j) {
- //
- // LAMESPEC: No idea why `params' modifier is ignored
- //
- if ((parameters.FixedParameters [j].ModFlags & ~Parameter.Modifier.PARAMS) !=
- (cmp_attrs.FixedParameters [j].ModFlags & ~Parameter.Modifier.PARAMS))
- break;
-
- if (!TypeManager.IsEqual (parameters.Types [j], cmp_attrs.Types [j]))
- break;
- }
-
- if (j < cmp_attrs.Count)
- continue;
-
- //
- // check generic arguments for methods
- //
- if (mi != null) {
- Type [] cmpGenArgs = TypeManager.GetGenericArguments (mi);
- if (generic_method == null && cmpGenArgs != null && cmpGenArgs.Length != 0)
- continue;
- if (generic_method != null && cmpGenArgs != null && cmpGenArgs.Length != generic_method.TypeParameters.Length)
- continue;
- }
-
- //
- // get one of the methods because this has the visibility info.
- //
- if (is_property) {
- mi = pi.GetGetMethod (true);
- if (mi == null)
- mi = pi.GetSetMethod (true);
- }
-
- //
- // Check visibility
- //
- switch (mi.Attributes & MethodAttributes.MemberAccessMask) {
- case MethodAttributes.PrivateScope:
- continue;
- case MethodAttributes.Private:
- //
- // A private method is Ok if we are a nested subtype.
- // The spec actually is not very clear about this, see bug 52458.
- //
- if (!invocation_type.Equals (entry.Container.Type) &&
- !TypeManager.IsNestedChildOf (invocation_type, entry.Container.Type))
- continue;
- break;
- case MethodAttributes.FamANDAssem:
- case MethodAttributes.Assembly:
- //
- // Check for assembly methods
- //
- if (!TypeManager.IsThisOrFriendAssembly (mi.DeclaringType.Assembly))
- continue;
- break;
- }
- return entry.Member;
- }
-
- return null;
- }
-
- ///
- /// The method is looking for conflict with inherited symbols (errors CS0108, CS0109).
- /// We handle two cases. The first is for types without parameters (events, field, properties).
- /// The second are methods, indexers and this is why ignore_complex_types is here.
- /// The latest param is temporary hack. See DoDefineMembers method for more info.
- ///
- public MemberInfo FindMemberWithSameName (string name, bool ignore_complex_types, MemberInfo ignore_member)
- {
- ArrayList applicable = null;
-
- if (method_hash != null)
- applicable = (ArrayList) method_hash [name];
-
- if (applicable != null) {
- for (int i = applicable.Count - 1; i >= 0; i--) {
- CacheEntry entry = (CacheEntry) applicable [i];
- if ((entry.EntryType & EntryType.Public) != 0)
- return entry.Member;
- }
- }
-
- if (member_hash == null)
- return null;
- applicable = (ArrayList) member_hash [name];
-
- if (applicable != null) {
- for (int i = applicable.Count - 1; i >= 0; i--) {
- CacheEntry entry = (CacheEntry) applicable [i];
- if ((entry.EntryType & EntryType.Public) != 0 & entry.Member != ignore_member) {
- if (ignore_complex_types) {
- if ((entry.EntryType & EntryType.Method) != 0)
- continue;
-
- // Does exist easier way how to detect indexer ?
- if ((entry.EntryType & EntryType.Property) != 0) {
- AParametersCollection arg_types = TypeManager.GetParameterData ((PropertyInfo)entry.Member);
- if (arg_types.Count > 0)
- continue;
- }
- }
- return entry.Member;
- }
- }
- }
- return null;
- }
-
- Hashtable locase_table;
-
- ///
- /// Builds low-case table for CLS Compliance test
- ///
- public Hashtable GetPublicMembers ()
- {
- if (locase_table != null)
- return locase_table;
-
- locase_table = new Hashtable ();
- foreach (DictionaryEntry entry in member_hash) {
- ArrayList members = (ArrayList)entry.Value;
- for (int ii = 0; ii < members.Count; ++ii) {
- CacheEntry member_entry = (CacheEntry) members [ii];
-
- if ((member_entry.EntryType & EntryType.Public) == 0)
- continue;
-
- // TODO: Does anyone know easier way how to detect that member is internal ?
- switch (member_entry.EntryType & EntryType.MaskType) {
- case EntryType.Constructor:
- continue;
-
- case EntryType.Field:
- if ((((FieldInfo)member_entry.Member).Attributes & (FieldAttributes.Assembly | FieldAttributes.Public)) == FieldAttributes.Assembly)
- continue;
- break;
-
- case EntryType.Method:
- if ((((MethodInfo)member_entry.Member).Attributes & (MethodAttributes.Assembly | MethodAttributes.Public)) == MethodAttributes.Assembly)
- continue;
- break;
-
- case EntryType.Property:
- PropertyInfo pi = (PropertyInfo)member_entry.Member;
- if (pi.GetSetMethod () == null && pi.GetGetMethod () == null)
- continue;
- break;
-
- case EntryType.Event:
- EventInfo ei = (EventInfo)member_entry.Member;
- MethodInfo mi = ei.GetAddMethod ();
- if ((mi.Attributes & (MethodAttributes.Assembly | MethodAttributes.Public)) == MethodAttributes.Assembly)
- continue;
- break;
- }
- string lcase = ((string)entry.Key).ToLower (System.Globalization.CultureInfo.InvariantCulture);
- locase_table [lcase] = member_entry.Member;
- break;
- }
- }
- return locase_table;
- }
-
- public Hashtable Members {
- get {
- return member_hash;
- }
- }
-
- ///
- /// Cls compliance check whether methods or constructors parameters differing only in ref or out, or in array rank
- ///
- ///
- // TODO: refactor as method is always 'this'
- public static void VerifyClsParameterConflict (ArrayList al, MethodCore method, MemberInfo this_builder)
- {
- EntryType tested_type = (method is Constructor ? EntryType.Constructor : EntryType.Method) | EntryType.Public;
-
- for (int i = 0; i < al.Count; ++i) {
- MemberCache.CacheEntry entry = (MemberCache.CacheEntry) al [i];
-
- // skip itself
- if (entry.Member == this_builder)
- continue;
-
- if ((entry.EntryType & tested_type) != tested_type)
- continue;
-
- MethodBase method_to_compare = (MethodBase)entry.Member;
- AttributeTester.Result result = AttributeTester.AreOverloadedMethodParamsClsCompliant (
- method.Parameters, TypeManager.GetParameterData (method_to_compare));
-
- if (result == AttributeTester.Result.Ok)
- continue;
-
- IMethodData md = TypeManager.GetMethod (method_to_compare);
-
- // TODO: now we are ignoring CLSCompliance(false) on method from other assembly which is buggy.
- // However it is exactly what csc does.
- if (md != null && !md.IsClsComplianceRequired ())
- continue;
-
- Report.SymbolRelatedToPreviousError (entry.Member);
- switch (result) {
- case AttributeTester.Result.RefOutArrayError:
- Report.Warning (3006, 1, method.Location,
- "Overloaded method `{0}' differing only in ref or out, or in array rank, is not CLS-compliant",
- method.GetSignatureForError ());
- continue;
- case AttributeTester.Result.ArrayArrayError:
- Report.Warning (3007, 1, method.Location,
- "Overloaded method `{0}' differing only by unnamed array types is not CLS-compliant",
- method.GetSignatureForError ());
- continue;
- }
-
- throw new NotImplementedException (result.ToString ());
- }
- }
-
- public bool CheckExistingMembersOverloads (MemberCore member, string name, ParametersCompiled parameters)
- {
- ArrayList entries = (ArrayList)member_hash [name];
- if (entries == null)
- return true;
-
- int method_param_count = parameters.Count;
- for (int i = entries.Count - 1; i >= 0; --i) {
- CacheEntry ce = (CacheEntry) entries [i];
-
- if (ce.Container != member.Parent.PartialContainer)
- return true;
-
- Type [] p_types;
- AParametersCollection pd;
- if ((ce.EntryType & EntryType.Property) != 0) {
- pd = TypeManager.GetParameterData ((PropertyInfo) ce.Member);
- p_types = pd.Types;
- } else {
- MethodBase mb = (MethodBase) ce.Member;
-
- // TODO: This is more like a hack, because we are adding generic methods
- // twice with and without arity name
- if (TypeManager.IsGenericMethod (mb) && !member.MemberName.IsGeneric)
- continue;
-
- pd = TypeManager.GetParameterData (mb);
- p_types = pd.Types;
- }
-
- if (p_types.Length != method_param_count)
- continue;
-
- if (method_param_count > 0) {
- int ii = method_param_count - 1;
- Type type_a, type_b;
- do {
- type_a = parameters.Types [ii];
- type_b = p_types [ii];
-
-#if GMCS_SOURCE
- if (TypeManager.IsGenericParameter (type_a) && type_a.DeclaringMethod != null)
- type_a = typeof (TypeParameter);
-
- if (TypeManager.IsGenericParameter (type_b) && type_b.DeclaringMethod != null)
- type_b = typeof (TypeParameter);
-#endif
- if ((pd.FixedParameters [ii].ModFlags & Parameter.Modifier.ISBYREF) !=
- (parameters.FixedParameters [ii].ModFlags & Parameter.Modifier.ISBYREF))
- type_a = null;
-
- } while (type_a == type_b && ii-- != 0);
-
- if (ii >= 0)
- continue;
-
- //
- // Operators can differ in return type only
- //
- if (member is Operator) {
- Operator op = TypeManager.GetMethod ((MethodBase) ce.Member) as Operator;
- if (op != null && op.ReturnType != ((Operator) member).ReturnType)
- continue;
- }
-
- //
- // Report difference in parameter modifiers only
- //
- if (pd != null && member is MethodCore) {
- ii = method_param_count;
- while (ii-- != 0 && parameters.FixedParameters [ii].ModFlags == pd.FixedParameters [ii].ModFlags &&
- parameters.ExtensionMethodType == pd.ExtensionMethodType);
-
- if (ii >= 0) {
- MethodCore mc = TypeManager.GetMethod ((MethodBase) ce.Member) as MethodCore;
- Report.SymbolRelatedToPreviousError (ce.Member);
- if ((member.ModFlags & Modifiers.PARTIAL) != 0 && (mc.ModFlags & Modifiers.PARTIAL) != 0) {
- if (parameters.HasParams || pd.HasParams) {
- Report.Error (758, member.Location,
- "A partial method declaration and partial method implementation cannot differ on use of `params' modifier");
- } else {
- Report.Error (755, member.Location,
- "A partial method declaration and partial method implementation must be both an extension method or neither");
- }
- } else {
- if (member is Constructor) {
- Report.Error (851, member.Location,
- "Overloaded contructor `{0}' cannot differ on use of parameter modifiers only",
- member.GetSignatureForError ());
- } else {
- Report.Error (663, member.Location,
- "Overloaded method `{0}' cannot differ on use of parameter modifiers only",
- member.GetSignatureForError ());
- }
- }
- return false;
- }
- }
- }
-
- if ((ce.EntryType & EntryType.Method) != 0) {
- Method method_a = member as Method;
- Method method_b = TypeManager.GetMethod ((MethodBase) ce.Member) as Method;
- if (method_a != null && method_b != null && (method_a.ModFlags & method_b.ModFlags & Modifiers.PARTIAL) != 0) {
- const int partial_modifiers = Modifiers.STATIC | Modifiers.UNSAFE;
- if (method_a.IsPartialDefinition == method_b.IsPartialImplementation) {
- if ((method_a.ModFlags & partial_modifiers) == (method_b.ModFlags & partial_modifiers) ||
- method_a.Parent.IsUnsafe && method_b.Parent.IsUnsafe) {
- if (method_a.IsPartialImplementation) {
- method_a.SetPartialDefinition (method_b);
- entries.RemoveAt (i);
- } else {
- method_b.SetPartialDefinition (method_a);
- }
- continue;
- }
-
- if ((method_a.ModFlags & Modifiers.STATIC) != (method_b.ModFlags & Modifiers.STATIC)) {
- Report.SymbolRelatedToPreviousError (ce.Member);
- Report.Error (763, member.Location,
- "A partial method declaration and partial method implementation must be both `static' or neither");
- }
-
- Report.SymbolRelatedToPreviousError (ce.Member);
- Report.Error (764, member.Location,
- "A partial method declaration and partial method implementation must be both `unsafe' or neither");
- return false;
- }
-
- Report.SymbolRelatedToPreviousError (ce.Member);
- if (method_a.IsPartialDefinition) {
- Report.Error (756, member.Location, "A partial method `{0}' declaration is already defined",
- member.GetSignatureForError ());
- }
-
- Report.Error (757, member.Location, "A partial method `{0}' implementation is already defined",
- member.GetSignatureForError ());
- return false;
- }
-
- Report.SymbolRelatedToPreviousError (ce.Member);
- IMethodData duplicate_member = TypeManager.GetMethod ((MethodBase) ce.Member);
- if (member is Operator && duplicate_member is Operator) {
- Report.Error (557, member.Location, "Duplicate user-defined conversion in type `{0}'",
- member.Parent.GetSignatureForError ());
- return false;
- }
-
- bool is_reserved_a = member is AbstractPropertyEventMethod || member is Operator;
- bool is_reserved_b = duplicate_member is AbstractPropertyEventMethod || duplicate_member is Operator;
-
- if (is_reserved_a || is_reserved_b) {
- Report.Error (82, member.Location, "A member `{0}' is already reserved",
- is_reserved_a ?
- TypeManager.GetFullNameSignature (ce.Member) :
- member.GetSignatureForError ());
- return false;
- }
- } else {
- Report.SymbolRelatedToPreviousError (ce.Member);
- }
-
- Report.Error (111, member.Location,
- "A member `{0}' is already defined. Rename this member or use different parameter types",
- member.GetSignatureForError ());
- return false;
- }
return true;
}