// Licensed under the terms of the GNU GPL
//
// (C) 2001 Ximian, Inc (http://www.ximian.com)
+// (C) 2004 Novell, Inc
//
// TODO: Move the method verification stuff from the class.cs and interface.cs here
//
using System.Reflection.Emit;
using System.Reflection;
+#if BOOTSTRAP_WITH_OLDLIB
+using XmlElement = System.Object;
+#else
+using System.Xml;
+#endif
+
namespace Mono.CSharp {
+ public class MemberName {
+ public readonly string Name;
+ public readonly MemberName Left;
+ public readonly Location Location;
+
+ bool is_double_colon;
+
+ public static readonly MemberName Null = new MemberName ("");
+
+ private MemberName (MemberName left, string name, bool is_double_colon, Location loc)
+ {
+ this.Name = name;
+ this.Location = loc;
+ this.is_double_colon = is_double_colon;
+ this.Left = left;
+ }
+
+ public MemberName (string name)
+ : this (null, name, false, Location.Null)
+ {
+ }
+
+ public MemberName (MemberName left, string name)
+ : this (left, name, false, left != null ? left.Location : Location.Null)
+ {
+ }
+
+ public MemberName (string name, Location loc)
+ : this (null, name, false, loc)
+ {
+ }
+
+ public MemberName (MemberName left, string name, Location loc)
+ : this (left, name, false, loc)
+ {
+ }
+
+ public MemberName (string alias, string name, Location loc)
+ : this (new MemberName (alias), name, true, loc)
+ {
+ }
+
+ public MemberName (MemberName left, MemberName right)
+ : this (left, right, right.Location)
+ {
+ }
+
+ public MemberName (MemberName left, MemberName right, Location loc)
+ : this (null, right.Name, false, loc)
+ {
+ if (right.is_double_colon)
+ throw new InternalErrorException ("Cannot append double_colon member name");
+ this.Left = (right.Left == null) ? left : new MemberName (left, right.Left);
+ }
+
+ static readonly char [] dot_array = { '.' };
+
+ public static MemberName FromDotted (string name)
+ {
+ string [] elements = name.Split (dot_array);
+ int count = elements.Length;
+ int i = 0;
+ MemberName n = new MemberName (elements [i++]);
+ while (i < count)
+ n = new MemberName (n, elements [i++]);
+ return n;
+ }
+
+ public string GetName ()
+ {
+ return GetName (false);
+ }
+
+ public string GetName (bool is_generic)
+ {
+ string name = is_generic ? Basename : Name;
+ string connect = is_double_colon ? "::" : ".";
+ if (Left != null)
+ return Left.GetName (is_generic) + connect + name;
+ else
+ return name;
+ }
+
+ ///
+ /// This returns exclusively the name as seen on the source code
+ /// it is not the fully qualifed type after resolution
+ ///
+ public string GetPartialName ()
+ {
+ string connect = is_double_colon ? "::" : ".";
+ if (Left != null)
+ return Left.GetPartialName () + connect + Name;
+ else
+ return Name;
+ }
+
+ public string GetTypeName ()
+ {
+ string connect = is_double_colon ? "::" : ".";
+ if (Left != null)
+ return Left.GetTypeName () + connect + Name;
+ else
+ return Name;
+ }
+
+ public Expression GetTypeExpression ()
+ {
+ if (Left == null)
+ return new SimpleName (Name, Location);
+ if (is_double_colon) {
+ if (Left.Left != null)
+ throw new InternalErrorException ("The left side of a :: should be an identifier");
+ return new QualifiedAliasMember (Left.Name, Name, Location);
+ }
+
+ Expression lexpr = Left.GetTypeExpression ();
+ return new MemberAccess (lexpr, Name, Location);
+ }
+
+ public MemberName Clone ()
+ {
+ MemberName left_clone = Left == null ? null : Left.Clone ();
+ return new MemberName (left_clone, Name, is_double_colon, Location);
+ }
+
+ public string Basename {
+ get { return Name; }
+ }
+
+ public override string ToString ()
+ {
+ string connect = is_double_colon ? "::" : ".";
+ if (Left != null)
+ return Left + connect + Name;
+ else
+ return Name;
+ }
+
+ public override bool Equals (object other)
+ {
+ return Equals (other as MemberName);
+ }
+
+ public bool Equals (MemberName other)
+ {
+ if (this == other)
+ return true;
+ if (other == null || Name != other.Name)
+ return false;
+ if (is_double_colon != other.is_double_colon)
+ return false;
+#if NET_2_0
+ if (TypeArguments == null)
+ return other.TypeArguments == null;
+
+ if (other.TypeArguments == null || TypeArguments.Count != other.TypeArguments.Count)
+ return false;
+#endif
+ if (Left == null)
+ return other.Left == null;
+
+ return Left.Equals (other.Left);
+ }
+
+ public override int GetHashCode ()
+ {
+ int hash = Name.GetHashCode ();
+ for (MemberName n = Left; n != null; n = n.Left)
+ hash ^= n.Name.GetHashCode ();
+ if (is_double_colon)
+ hash ^= 0xbadc01d;
+#if NET_2_0
+ if (TypeArguments != null)
+ hash ^= TypeArguments.Count << 5;
+#endif
+
+ return hash & 0x7FFFFFFF;
+ }
+ }
+
/// <summary>
/// Base representation for members. This is used to keep track
/// of Name, Location and Modifier flags, and handling Attributes.
/// <summary>
/// Public name
/// </summary>
- public string Name;
+
+ protected string cached_name;
+ public string Name {
+ get {
+ if (cached_name == null)
+ cached_name = MemberName.GetName (false);
+ return cached_name;
+ }
+ }
+
+ // Is not readonly because of IndexerName attribute
+ private MemberName member_name;
+ public MemberName MemberName {
+ get { return member_name; }
+ }
/// <summary>
/// Modifier flags that the user specified in the source code
/// </summary>
public int ModFlags;
+ public /*readonly*/ TypeContainer Parent;
+
/// <summary>
/// Location where this declaration happens
/// </summary>
- public readonly Location Location;
+ public Location Location {
+ get { return member_name.Location; }
+ }
+
+ /// <summary>
+ /// XML documentation comment
+ /// </summary>
+ public string DocComment;
+
+ /// <summary>
+ /// Represents header string for documentation comment
+ /// for each member types.
+ /// </summary>
+ public abstract string DocCommentHeader { get; }
[Flags]
public enum Flags {
HasClsCompliantAttribute = 1 << 6, // Type has CLSCompliantAttribute
ClsCompliantAttributeTrue = 1 << 7, // Type has CLSCompliant (true)
Excluded_Undetected = 1 << 8, // Conditional attribute has not been detected yet
- Excluded = 1 << 9 // Method is conditional
-
+ Excluded = 1 << 9, // Method is conditional
+ TestMethodDuplication = 1 << 10, // Test for duplication must be performed
+ IsUsed = 1 << 11,
+ IsAssigned = 1 << 12 // Field is assigned
}
/// <summary>
/// MemberCore flags at first detected then cached
/// </summary>
- protected Flags caching_flags;
+ internal Flags caching_flags;
- public MemberCore (string name, Attributes attrs, Location loc)
+ public MemberCore (TypeContainer parent, MemberName name, Attributes attrs)
: base (attrs)
{
- Name = name;
- Location = loc;
+ if (parent is PartialContainer && !(this is PartialContainer))
+ throw new InternalErrorException ("A PartialContainer cannot be the direct parent of a member");
+
+ Parent = parent;
+ member_name = name;
caching_flags = Flags.Obsolete_Undetected | Flags.ClsCompliance_Undetected | Flags.HasCompliantAttribute_Undetected | Flags.Excluded_Undetected;
}
- /// <summary>
- /// Tests presence of ObsoleteAttribute and report proper error
- /// </summary>
- protected void CheckUsageOfObsoleteAttribute (Type type)
+ protected virtual void SetMemberName (MemberName new_name)
{
- if (type == null)
- return;
-
- ObsoleteAttribute obsolete_attr = AttributeTester.GetObsoleteAttribute (type);
- if (obsolete_attr == null)
- return;
-
- AttributeTester.Report_ObsoleteMessage (obsolete_attr, type.FullName, Location);
+ member_name = new_name;
+ cached_name = null;
}
- public abstract bool Define (TypeContainer parent);
+ public abstract bool Define ();
//
// Returns full member name for error message
//
public virtual string GetSignatureForError ()
{
- return Name;
+ if (Parent == null || Parent.Parent == null)
+ return Name;
+
+ return String.Concat (Parent.GetSignatureForError (), '.', Name);
}
/// <summary>
/// Base Emit method. This is also entry point for CLS-Compliant verification.
/// </summary>
- public virtual void Emit (TypeContainer container)
+ public virtual void Emit ()
{
- VerifyObsoleteAttribute ();
-
if (!RootContext.VerifyClsCompliance)
return;
- VerifyClsCompliance (container);
+ VerifyClsCompliance (Parent);
+ }
+
+ public virtual EmitContext EmitContext
+ {
+ get {
+ return Parent.EmitContext;
+ }
+ }
+
+ public bool InUnsafe {
+ get {
+ return ((ModFlags & Modifiers.UNSAFE) != 0) || Parent.UnsafeContext;
+ }
+ }
+
+ public virtual bool IsUsed {
+ get {
+ return (caching_flags & Flags.IsUsed) != 0;
+ }
+ }
+
+ public void SetMemberIsUsed ()
+ {
+ caching_flags |= Flags.IsUsed;
}
//
/// <summary>
/// Returns instance of ObsoleteAttribute for this MemberCore
/// </summary>
- public ObsoleteAttribute GetObsoleteAttribute (DeclSpace ds)
+ public virtual ObsoleteAttribute GetObsoleteAttribute ()
{
// ((flags & (Flags.Obsolete_Undetected | Flags.Obsolete)) == 0) is slower, but why ?
if ((caching_flags & Flags.Obsolete_Undetected) == 0 && (caching_flags & Flags.Obsolete) == 0) {
if (OptAttributes == null)
return null;
- // TODO: remove this allocation
- EmitContext ec = new EmitContext (ds.Parent, ds, ds.Location,
- null, null, ds.ModFlags, false);
-
- Attribute obsolete_attr = OptAttributes.Search (TypeManager.obsolete_attribute_type, ec);
+ Attribute obsolete_attr = OptAttributes.Search (
+ TypeManager.obsolete_attribute_type, EmitContext);
if (obsolete_attr == null)
return null;
- ObsoleteAttribute obsolete = obsolete_attr.GetObsoleteAttribute (ds);
+ ObsoleteAttribute obsolete = obsolete_attr.GetObsoleteAttribute (EmitContext);
if (obsolete == null)
return null;
return obsolete;
}
+ /// <summary>
+ /// Checks for ObsoleteAttribute presence. It's used for testing of all non-types elements
+ /// </summary>
+ public virtual void CheckObsoleteness (Location loc)
+ {
+ if (Parent != null)
+ Parent.CheckObsoleteness (loc);
+
+ ObsoleteAttribute oa = GetObsoleteAttribute ();
+ if (oa == null) {
+ return;
+ }
+
+ AttributeTester.Report_ObsoleteMessage (oa, GetSignatureForError (), loc);
+ }
+
+ protected void CheckObsoleteType (Expression type)
+ {
+ ObsoleteAttribute obsolete_attr = AttributeTester.GetObsoleteAttribute (type.Type);
+ if (obsolete_attr == null)
+ return;
+
+ if (GetObsoleteAttribute () != null || Parent.GetObsoleteAttribute () != null)
+ return;
+
+ AttributeTester.Report_ObsoleteMessage (obsolete_attr, TypeManager.CSharpName (type.Type), type.Location);
+ }
+
/// <summary>
/// Analyze whether CLS-Compliant verification must be execute for this MemberCore.
/// </summary>
/// <summary>
/// Returns true when MemberCore is exposed from assembly.
/// </summary>
- protected bool IsExposedFromAssembly (DeclSpace ds)
+ public bool IsExposedFromAssembly (DeclSpace ds)
{
if ((ModFlags & (Modifiers.PUBLIC | Modifiers.PROTECTED)) == 0)
return false;
bool GetClsCompliantAttributeValue (DeclSpace ds)
{
if (OptAttributes != null) {
- EmitContext ec = new EmitContext (ds.Parent, ds, ds.Location,
- null, null, ds.ModFlags, false);
- Attribute cls_attribute = OptAttributes.GetClsCompliantAttribute (ec);
+ Attribute cls_attribute = OptAttributes.Search (
+ TypeManager.cls_compliant_attribute_type, ds.EmitContext);
if (cls_attribute != null) {
caching_flags |= Flags.HasClsCompliantAttribute;
- return cls_attribute.GetClsCompliantAttributeValue (ds);
+ return cls_attribute.GetClsCompliantAttributeValue (ds.EmitContext);
}
}
return ds.GetClsCompliantAttributeValue ();
}
/// <summary>
- /// This method is used to testing error 3005 (Method or parameter name collision).
- /// </summary>
- protected abstract bool IsIdentifierClsCompliant (DeclSpace ds);
-
- /// <summary>
- /// Common helper method for identifier and parameters CLS-Compliant testing.
- /// When return false error 3005 is reported. True means no violation.
- /// And error 3006 tests are peformed here because of speed.
+ /// It helps to handle error 102 & 111 detection
/// </summary>
- protected bool IsIdentifierAndParamClsCompliant (DeclSpace ds, string name, MemberInfo methodBuilder, Type[] paramTypes)
+ public virtual bool MarkForDuplicationCheck ()
{
- MemberList ml = ds.FindMembers (MemberTypes.Event | MemberTypes.Field | MemberTypes.Method | MemberTypes.Property,
- BindingFlags.Public | BindingFlags.Static | BindingFlags.Instance, System.Type.FilterNameIgnoreCase, name);
-
- if (ml.Count < 2)
- return true;
-
- bool error3006 = false;
- for (int i = 0; i < ml.Count; ++i) {
- MemberInfo mi = ml [i];
- if (name == mi.Name) {
- MethodBase method = mi as MethodBase;
- if (method == null || method == methodBuilder || paramTypes == null || paramTypes.Length == 0)
- continue;
-
- if (AttributeTester.AreOverloadedMethodParamsClsCompliant (paramTypes, TypeManager.GetArgumentTypes (method))) {
- error3006 = false;
- continue;
- }
-
- error3006 = true;
- }
-
- // We need to test if member is not marked as CLSCompliant (false) and if type is not only internal
- // because BindingFlags.Public returns internal types too
- DeclSpace temp_ds = TypeManager.LookupDeclSpace (mi.DeclaringType);
-
- // Type is external, we can get attribute directly
- if (temp_ds == null) {
- object[] cls_attribute = mi.GetCustomAttributes (TypeManager.cls_compliant_attribute_type, false);
- if (cls_attribute.Length == 1 && (!((CLSCompliantAttribute)cls_attribute[0]).IsCompliant))
- continue;
- } else {
- string tmp_name = String.Concat (temp_ds.Name, '.', mi.Name);
-
- MemberCore mc = temp_ds.GetDefinition (tmp_name) as MemberCore;
- if (!mc.IsClsCompliaceRequired (ds))
- continue;
- }
-
- for (int ii = 0; ii < ml.Count; ++ii) {
- mi = ml [ii];
- if (name == mi.Name)
- continue;
- Report.SymbolRelatedToPreviousError (mi);
- }
-
- if (error3006)
- Report.Error_T (3006, Location, GetSignatureForError ());
-
- return error3006;
-
- }
- return true;
+ return false;
}
/// <summary>
protected virtual bool VerifyClsCompliance (DeclSpace ds)
{
if (!IsClsCompliaceRequired (ds)) {
- if (HasClsCompliantAttribute && !IsExposedFromAssembly (ds)) {
- Report.Warning_T (3019, Location, GetSignatureForError ());
+ if (HasClsCompliantAttribute && RootContext.WarningLevel >= 2) {
+ if (!IsExposedFromAssembly (ds))
+ Report.Warning (3019, Location, "CLS compliance checking will not be performed on `{0}' because it is not visible from outside this assembly", GetSignatureForError ());
+ if (!CodeGen.Assembly.IsClsCompliant)
+ Report.Warning (3021, Location, "`{0}' does not need a CLSCompliant attribute because the assembly is not marked as CLS-compliant", GetSignatureForError ());
}
return false;
}
if (!CodeGen.Assembly.IsClsCompliant) {
if (HasClsCompliantAttribute) {
- Report.Error_T (3014, Location, GetSignatureForError ());
+ Report.Error (3014, Location,
+ "`{0}' cannot be marked as CLS-compliant because the assembly is not marked as CLS-compliant",
+ GetSignatureForError ());
}
+ return false;
}
- int index = Name.LastIndexOf ('.');
- if (Name [index > 0 ? index + 1 : 0] == '_') {
- Report.Error_T (3008, Location, GetSignatureForError () );
- }
-
- if (!IsIdentifierClsCompliant (ds)) {
- Report.Error_T (3005, Location, GetSignatureForError ());
+ if (member_name.Name [0] == '_') {
+ Report.Error (3008, Location, "Identifier `{0}' is not CLS-compliant", GetSignatureForError () );
}
-
return true;
}
- protected abstract void VerifyObsoleteAttribute ();
+ //
+ // Raised (and passed an XmlElement that contains the comment)
+ // when GenerateDocComment is writing documentation expectedly.
+ //
+ internal virtual void OnGenerateDocComment (DeclSpace ds, XmlElement intermediateNode)
+ {
+ }
+ //
+ // Returns a string that represents the signature for this
+ // member which should be used in XML documentation.
+ //
+ public virtual string GetDocCommentName (DeclSpace ds)
+ {
+ if (ds == null || this is DeclSpace)
+ return DocCommentHeader + Name;
+ else
+ return String.Concat (DocCommentHeader, ds.Name, ".", Name);
+ }
+
+ //
+ // Generates xml doc comments (if any), and if required,
+ // handle warning report.
+ //
+ internal virtual void GenerateDocComment (DeclSpace ds)
+ {
+ DocUtil.GenerateDocComment (this, ds);
+ }
}
/// <summary>
/// </remarks>
public abstract class DeclSpace : MemberCore {
/// <summary>
- /// this points to the actual definition that is being
+ /// This points to the actual definition that is being
/// created with System.Reflection.Emit
/// </summary>
public TypeBuilder TypeBuilder;
//
public NamespaceEntry NamespaceEntry;
- public Hashtable Cache = new Hashtable ();
+ private Hashtable Cache = new Hashtable ();
public string Basename;
- /// <summary>
- /// defined_names is used for toplevel objects
- /// </summary>
protected Hashtable defined_names;
- TypeContainer parent;
+ // The emit context for toplevel objects.
+ protected EmitContext ec;
+
+ public override EmitContext EmitContext {
+ get {
+ return ec;
+ }
+ }
static string[] attribute_targets = new string [] { "type" };
- public DeclSpace (NamespaceEntry ns, TypeContainer parent, string name, Attributes attrs, Location l)
- : base (name, attrs, l)
+ public DeclSpace (NamespaceEntry ns, TypeContainer parent, MemberName name,
+ Attributes attrs)
+ : base (parent, name, attrs)
{
NamespaceEntry = ns;
- Basename = name.Substring (1 + name.LastIndexOf ('.'));
+ Basename = name.Name;
defined_names = new Hashtable ();
- this.parent = parent;
- }
-
- public void RecordDecl ()
- {
- if ((NamespaceEntry != null) && (parent == RootContext.Tree.Types))
- NamespaceEntry.DefineName (Basename, this);
- }
-
- /// <summary>
- /// The result value from adding an declaration into
- /// a struct or a class
- /// </summary>
- public enum AdditionResult {
- /// <summary>
- /// The declaration has been successfully
- /// added to the declation space.
- /// </summary>
- Success,
-
- /// <summary>
- /// The symbol has already been defined.
- /// </summary>
- NameExists,
-
- /// <summary>
- /// Returned if the declation being added to the
- /// name space clashes with its container name.
- ///
- /// The only exceptions for this are constructors
- /// and static constructors
- /// </summary>
- EnclosingClash,
-
- /// <summary>
- /// Returned if a constructor was created (because syntactically
- /// it looked like a constructor) but was not (because the name
- /// of the method is not the same as the container class
- /// </summary>
- NotAConstructor,
-
- /// <summary>
- /// This is only used by static constructors to emit the
- /// error 111, but this error for other things really
- /// happens at another level for other functions.
- /// </summary>
- MethodExists,
-
- /// <summary>
- /// Some other error.
- /// </summary>
- Error
}
/// <summary>
- /// Returns a status code based purely on the name
- /// of the member being added
+ /// Adds the member to defined_names table. It tests for duplications and enclosing name conflicts
/// </summary>
- protected AdditionResult IsValid (string basename, string name)
+ protected bool AddToContainer (MemberCore symbol, string name)
{
- if (basename == Basename)
- return AdditionResult.EnclosingClash;
-
- if (defined_names.Contains (name))
- return AdditionResult.NameExists;
-
- return AdditionResult.Success;
- }
-
- public static int length;
- public static int small;
-
- /// <summary>
- /// Introduce @name into this declaration space and
- /// associates it with the object @o. Note that for
- /// methods this will just point to the first method. o
- /// </summary>
- public void DefineName (string name, object o)
- {
- defined_names.Add (name, o);
+ if (name == Basename && !(this is Interface) && !(this is Enum)) {
+ Report.SymbolRelatedToPreviousError (this);
+ Report.Error (542, symbol.Location, "`{0}': member names cannot be the same as their enclosing type", symbol.GetSignatureForError ());
+ return false;
+ }
-#if DEBUGME
- int p = name.LastIndexOf ('.');
- int l = name.Length;
- length += l;
- small += l -p;
-#endif
- }
+ MemberCore mc = (MemberCore) defined_names [name];
- /// <summary>
- /// Returns the object associated with a given name in the declaration
- /// space. This is the inverse operation of `DefineName'
- /// </summary>
- public object GetDefinition (string name)
- {
- return defined_names [name];
- }
-
- bool in_transit = false;
-
- /// <summary>
- /// This function is used to catch recursive definitions
- /// in declarations.
- /// </summary>
- public bool InTransit {
- get {
- return in_transit;
+ if (mc == null) {
+ defined_names.Add (name, symbol);
+ return true;
}
- set {
- in_transit = value;
+ if (symbol.MarkForDuplicationCheck () && mc.MarkForDuplicationCheck ())
+ return true;
+
+ Report.SymbolRelatedToPreviousError (mc);
+ if (symbol is PartialContainer || mc is PartialContainer) {
+ Report.Error (260, symbol.Location,
+ "Missing partial modifier on declaration of type `{0}'. Another partial declaration of this type exists",
+ name);
+ return false;
}
- }
- public TypeContainer Parent {
- get {
- return parent;
+ if (this is RootTypes) {
+ Report.Error (101, symbol.Location,
+ "The namespace `{0}' already contains a definition for `{1}'",
+ ((DeclSpace)symbol).NamespaceEntry.GetSignatureForError (), symbol.MemberName.Name);
+ } else {
+ Report.Error (102, symbol.Location, "The type `{0}' already contains a definition for `{1}'",
+ GetSignatureForError (), symbol.MemberName.Name);
}
+ return false;
}
/// <summary>
- /// Looks up the alias for the name
+ /// Returns the MemberCore associated with a given name in the declaration
+ /// space. It doesn't return method based symbols !!
/// </summary>
- public string LookupAlias (string name)
+ ///
+ public MemberCore GetDefinition (string name)
{
- if (NamespaceEntry != null)
- return NamespaceEntry.LookupAlias (name);
- else
- return null;
+ return (MemberCore)defined_names [name];
}
//
// why there is a non-obvious test down here.
//
public bool IsTopLevel {
- get {
- if (parent != null){
- if (parent.parent == null)
- return true;
- }
- return false;
- }
+ get { return (Parent != null && Parent.Parent == null); }
}
public virtual void CloseType ()
}
}
+ protected virtual TypeAttributes TypeAttr {
+ get { return CodeGen.Module.DefaultCharSetType; }
+ }
+
/// <remarks>
/// Should be overriten by the appropriate declaration space
/// </remarks>
/// Define all members, but don't apply any attributes or do anything which may
/// access not-yet-defined classes. This method also creates the MemberCache.
/// </summary>
- public abstract bool DefineMembers (TypeContainer parent);
+ public virtual bool DefineMembers (TypeContainer parent)
+ {
+ if (((ModFlags & Modifiers.NEW) != 0) && IsTopLevel) {
+ Report.Error (1530, Location, "Keyword `new' is not allowed on namespace elements");
+ return false;
+ }
+ return true;
+ }
+
+ public override string GetSignatureForError ()
+ {
+ // Parent.GetSignatureForError
+ return Name;
+ }
//
// Whether this is an `unsafe context'
get {
if ((ModFlags & Modifiers.UNSAFE) != 0)
return true;
- if (parent != null)
- return parent.UnsafeContext;
+ if (Parent != null)
+ return Parent.UnsafeContext;
return false;
}
}
- public static string MakeFQN (string nsn, string name)
- {
- if (nsn == "")
- return name;
- return String.Concat (nsn, ".", name);
- }
-
EmitContext type_resolve_ec;
- EmitContext GetTypeResolveEmitContext (TypeContainer parent, Location loc)
- {
- type_resolve_ec = new EmitContext (parent, this, loc, null, null, ModFlags, false);
- type_resolve_ec.ResolvingTypeTree = true;
-
- return type_resolve_ec;
- }
-
- // <summary>
- // Looks up the type, as parsed into the expression `e'
- // </summary>
- public Type ResolveType (Expression e, bool silent, Location loc)
- {
- if (type_resolve_ec == null)
- type_resolve_ec = GetTypeResolveEmitContext (parent, loc);
- type_resolve_ec.loc = loc;
- type_resolve_ec.ContainerType = TypeBuilder;
-
- int errors = Report.Errors;
- TypeExpr d = e.ResolveAsTypeTerminal (type_resolve_ec);
-
- if (d == null || d.eclass != ExprClass.Type){
- if (!silent && errors == Report.Errors){
- Report.Error (246, loc, "Cannot find type `"+ e.ToString () +"'");
+ protected EmitContext TypeResolveEmitContext {
+ get {
+ if (type_resolve_ec == null) {
+ // FIXME: I think this should really be one of:
+ //
+ // a. type_resolve_ec = Parent.EmitContext;
+ // b. type_resolve_ec = new EmitContext (Parent, Parent, loc, null, null, ModFlags, false);
+ //
+ // However, if Parent == RootContext.Tree.Types, its NamespaceEntry will be null.
+ //
+ type_resolve_ec = new EmitContext (Parent, this, Location.Null, null, null, ModFlags, false);
}
- return null;
+ return type_resolve_ec;
}
-
- if (!d.CheckAccessLevel (this)) {
- Report.Error_T (122, loc, d.Name);
- return null;
- }
-
- return d.Type;
}
// <summary>
// Resolves the expression `e' for a type, and will recursively define
- // types.
+ // types. This should only be used for resolving base types.
// </summary>
- public TypeExpr ResolveTypeExpr (Expression e, bool silent, Location loc)
- {
- if (type_resolve_ec == null)
- type_resolve_ec = GetTypeResolveEmitContext (parent, loc);
- type_resolve_ec.loc = loc;
- type_resolve_ec.ContainerType = TypeBuilder;
-
- TypeExpr d = e.ResolveAsTypeTerminal (type_resolve_ec);
-
- if (d == null || d.eclass != ExprClass.Type){
- if (!silent){
- Report.Error (246, loc, "Cannot find type `"+ e +"'");
- }
- return null;
- }
+ public TypeExpr ResolveBaseTypeExpr (Expression e, bool silent, Location loc)
+ {
+ TypeResolveEmitContext.loc = loc;
+ TypeResolveEmitContext.ContainerType = TypeBuilder;
- return d;
+ return e.ResolveAsTypeTerminal (TypeResolveEmitContext, silent);
}
- public bool CheckAccessLevel (Type check_type)
+ public bool CheckAccessLevel (Type check_type)
{
if (check_type == TypeBuilder)
return true;
TypeAttributes check_attr = check_type.Attributes & TypeAttributes.VisibilityMask;
-
+
//
// Broken Microsoft runtime, return public for arrays, no matter what
// the accessibility is for their underlying class, and they return
if (check_type.IsArray || check_type.IsPointer)
return CheckAccessLevel (TypeManager.GetElementType (check_type));
+ if (TypeBuilder == null)
+ // FIXME: TypeBuilder will be null when invoked by Class.GetNormalBases().
+ // However, this is invoked again later -- so safe to return true.
+ // May also be null when resolving top-level attributes.
+ return true;
+
switch (check_attr){
case TypeAttributes.Public:
return true;
case TypeAttributes.NotPublic:
-
- // In same cases is null.
- if (TypeBuilder == null)
- return true;
-
//
// This test should probably use the declaringtype.
//
return true;
case TypeAttributes.NestedPrivate:
- string check_type_name = check_type.FullName;
- string type_name = TypeBuilder.FullName;
-
- int cio = check_type_name.LastIndexOf ('+');
- string container = check_type_name.Substring (0, cio);
-
- //
- // Check if the check_type is a nested class of the current type
- //
- if (check_type_name.StartsWith (type_name + "+")){
- return true;
- }
-
- if (type_name.StartsWith (container)){
- return true;
- }
-
- return false;
+ return NestedAccessible (check_type);
case TypeAttributes.NestedFamily:
- //
- // Only accessible to methods in current type or any subtypes
- //
return FamilyAccessible (check_type);
case TypeAttributes.NestedFamANDAssem:
}
- protected bool FamilyAccessible (Type check_type)
+ protected bool NestedAccessible (Type check_type)
{
Type declaring = check_type.DeclaringType;
- if (TypeBuilder.IsSubclassOf (declaring))
- return true;
-
- string check_type_name = check_type.FullName;
-
- int cio = check_type_name.LastIndexOf ('+');
- string container = check_type_name.Substring (0, cio);
-
- //
- // Check if the check_type is a nested class of the current type
- //
- if (check_type_name.StartsWith (container + "+"))
- return true;
+ return TypeBuilder == declaring ||
+ TypeManager.IsNestedChildOf (TypeBuilder, declaring);
+ }
- return false;
+ protected bool FamilyAccessible (Type check_type)
+ {
+ Type declaring = check_type.DeclaringType;
+ return TypeManager.IsNestedFamilyAccessible (TypeBuilder, declaring);
}
// Access level of a type.
return ~ (~ mAccess | pAccess) == 0;
}
-
- static DoubleHash dh = new DoubleHash (1000);
-
- Type DefineTypeAndParents (DeclSpace tc)
- {
- DeclSpace container = tc.Parent;
-
- if (container.TypeBuilder == null && container.Name != "")
- DefineTypeAndParents (container);
-
- return tc.DefineType ();
- }
-
- Type LookupInterfaceOrClass (string ns, string name, out bool error)
- {
- DeclSpace parent;
- Type t;
- object r;
-
- error = false;
- int p = name.LastIndexOf ('.');
-
- if (dh.Lookup (ns, name, out r))
- return (Type) r;
- else {
- //
- // If the type is not a nested type, we do not need `LookupType's processing.
- // If the @name does not have a `.' in it, this cant be a nested type.
- //
- if (ns != ""){
- if (Namespace.IsNamespace (ns)) {
- if (p != -1)
- t = TypeManager.LookupType (ns + "." + name);
- else
- t = TypeManager.LookupTypeDirect (ns + "." + name);
- } else
- t = null;
- } else if (p != -1)
- t = TypeManager.LookupType (name);
- else
- t = TypeManager.LookupTypeDirect (name);
- }
-
- if (t != null) {
- dh.Insert (ns, name, t);
- return t;
- }
-
- //
- // In case we are fed a composite name, normalize it.
- //
-
- if (p != -1){
- ns = MakeFQN (ns, name.Substring (0, p));
- name = name.Substring (p+1);
- }
-
- parent = RootContext.Tree.LookupByNamespace (ns, name);
- if (parent == null) {
- dh.Insert (ns, name, null);
- return null;
- }
-
- t = DefineTypeAndParents (parent);
- if (t == null){
- error = true;
- return null;
- }
-
- dh.Insert (ns, name, t);
- return t;
- }
- public static void Error_AmbiguousTypeReference (Location loc, string name, Type t1, Type t2)
+ //
+ // Return the nested type with name @name. Ensures that the nested type
+ // is defined if necessary. Do _not_ use this when you have a MemberCache handy.
+ //
+ public virtual Type FindNestedType (string name)
{
- Report.Error (104, loc,
- String.Format ("`{0}' is an ambiguous reference ({1} or {2}) ", name,
- t1.FullName, t2.FullName));
+ return null;
}
- /// <summary>
- /// GetType is used to resolve type names at the DeclSpace level.
- /// Use this to lookup class/struct bases, interface bases or
- /// delegate type references
- /// </summary>
- ///
- /// <remarks>
- /// Contrast this to LookupType which is used inside method bodies to
- /// lookup types that have already been defined. GetType is used
- /// during the tree resolution process and potentially define
- /// recursively the type
- /// </remarks>
- public Type FindType (Location loc, string name)
+ private Type LookupNestedTypeInHierarchy (string name)
{
- Type t;
- bool error;
-
- //
- // For the case the type we are looking for is nested within this one
- // or is in any base class
- //
- DeclSpace containing_ds = this;
-
- while (containing_ds != null){
- Type container_type = containing_ds.TypeBuilder;
- Type current_type = container_type;
-
- while (current_type != null && current_type != TypeManager.object_type) {
- string pre = current_type.FullName;
-
- t = LookupInterfaceOrClass (pre, name, out error);
- if (error)
- return null;
-
- if ((t != null) && containing_ds.CheckAccessLevel (t))
- return t;
-
- current_type = current_type.BaseType;
+ // if the member cache has been created, lets use it.
+ // the member cache is MUCH faster.
+ if (MemberCache != null)
+ return MemberCache.FindNestedType (name);
+
+ // no member cache. Do it the hard way -- reflection
+ Type t = null;
+ for (Type current_type = TypeBuilder;
+ current_type != null && current_type != TypeManager.object_type;
+ current_type = current_type.BaseType) {
+ if (current_type is TypeBuilder) {
+ DeclSpace decl = this;
+ if (current_type != TypeBuilder)
+ decl = TypeManager.LookupDeclSpace (current_type);
+ t = decl.FindNestedType (name);
+ } else {
+ t = TypeManager.GetNestedType (current_type, name);
}
- containing_ds = containing_ds.Parent;
- }
- //
- // Attempt to lookup the class on our namespace and all it's implicit parents
- //
- for (NamespaceEntry ns = NamespaceEntry; ns != null; ns = ns.ImplicitParent) {
- t = LookupInterfaceOrClass (ns.FullName, name, out error);
- if (error)
- return null;
-
- if (t != null)
+ if (t != null && CheckAccessLevel (t))
return t;
}
-
- //
- // Attempt to do a direct unqualified lookup
- //
- t = LookupInterfaceOrClass ("", name, out error);
- if (error)
- return null;
-
- if (t != null)
- return t;
-
- //
- // Attempt to lookup the class on any of the `using'
- // namespaces
- //
- for (NamespaceEntry ns = NamespaceEntry; ns != null; ns = ns.Parent){
-
- t = LookupInterfaceOrClass (ns.FullName, name, out error);
- if (error)
- return null;
-
- if (t != null)
- return t;
-
- if (name.IndexOf ('.') > 0)
- continue;
+ return null;
+ }
- string alias_value = ns.LookupAlias (name);
- if (alias_value != null) {
- t = LookupInterfaceOrClass ("", alias_value, out error);
- if (error)
- return null;
+ //
+ // Public function used to locate types.
+ //
+ // Set 'ignore_cs0104' to true if you want to ignore cs0104 errors.
+ //
+ // Returns: Type or null if they type can not be found.
+ //
+ public FullNamedExpression LookupType (string name, Location loc, bool ignore_cs0104)
+ {
+ if (this is PartialContainer)
+ throw new InternalErrorException ("Should not get here");
- if (t != null)
- return t;
- }
+ if (Cache.Contains (name))
+ return (FullNamedExpression) Cache [name];
- //
- // Now check the using clause list
- //
- Type match = null;
- foreach (Namespace using_ns in ns.GetUsingTable ()) {
- match = LookupInterfaceOrClass (using_ns.Name, name, out error);
- if (error)
- return null;
-
- if (match != null){
- if (t != null){
- if (CheckAccessLevel (match)) {
- Error_AmbiguousTypeReference (loc, name, t, match);
- return null;
- }
- continue;
- }
-
- t = match;
- }
- }
- if (t != null)
- return t;
- }
+ FullNamedExpression e;
+ Type t = LookupNestedTypeInHierarchy (name);
+ if (t != null)
+ e = new TypeExpression (t, Location.Null);
+ else if (Parent != null && Parent != RootContext.Tree.Types)
+ e = Parent.LookupType (name, loc, ignore_cs0104);
+ else
+ e = NamespaceEntry.LookupNamespaceOrType (this, name, loc, ignore_cs0104);
- //Report.Error (246, Location, "Can not find type `"+name+"'");
- return null;
+ Cache [name] = e;
+ return e;
}
/// <remarks>
public override void ApplyAttributeBuilder (Attribute a, CustomAttributeBuilder cb)
{
- try {
- TypeBuilder.SetCustomAttribute (cb);
- } catch (System.ArgumentException e) {
- Report.Warning (-21, a.Location,
- "The CharSet named property on StructLayout\n"+
- "\tdoes not work correctly on Microsoft.NET\n"+
- "\tYou might want to remove the CharSet declaration\n"+
- "\tor compile using the Mono runtime instead of the\n"+
- "\tMicrosoft .NET runtime\n"+
- "\tThe runtime gave the error: " + e);
+ if (a.Type == TypeManager.required_attr_type) {
+ Report.Error (1608, a.Location, "The RequiredAttribute attribute is not permitted on C# types");
+ return;
}
+ TypeBuilder.SetCustomAttribute (cb);
}
/// <summary>
caching_flags &= ~Flags.HasCompliantAttribute_Undetected;
if (OptAttributes != null) {
- EmitContext ec = new EmitContext (parent, this, Location,
- null, null, ModFlags, false);
- Attribute cls_attribute = OptAttributes.GetClsCompliantAttribute (ec);
+ Attribute cls_attribute = OptAttributes.Search (TypeManager.cls_compliant_attribute_type, ec);
if (cls_attribute != null) {
caching_flags |= Flags.HasClsCompliantAttribute;
- if (cls_attribute.GetClsCompliantAttributeValue (this)) {
+ if (cls_attribute.GetClsCompliantAttributeValue (ec)) {
caching_flags |= Flags.ClsCompliantAttributeTrue;
return true;
}
}
}
- if (parent == null) {
+ if (Parent == null) {
if (CodeGen.Assembly.IsClsCompliant) {
caching_flags |= Flags.ClsCompliantAttributeTrue;
return true;
return false;
}
- if (parent.GetClsCompliantAttributeValue ()) {
+ if (Parent.GetClsCompliantAttributeValue ()) {
caching_flags |= Flags.ClsCompliantAttributeTrue;
return true;
}
return false;
}
+ public override string[] ValidAttributeTargets {
+ get { return attribute_targets; }
+ }
- // Tests container name for CLS-Compliant name (differing only in case)
- // Possible optimalization: search in same namespace only
- protected override bool IsIdentifierClsCompliant (DeclSpace ds)
+ protected override bool VerifyClsCompliance (DeclSpace ds)
{
- int l = Name.Length;
-
- if (Namespace.LookupNamespace (NamespaceEntry.FullName, false) != null) {
- // Seek through all imported types
- foreach (string type_name in TypeManager.all_imported_types.Keys)
- {
- if (l != type_name.Length)
- continue;
-
- if (String.Compare (Name, type_name, true, CultureInfo.InvariantCulture) == 0 &&
- AttributeTester.IsClsCompliant (TypeManager.all_imported_types [type_name] as Type)) {
- Report.SymbolRelatedToPreviousError ((Type)TypeManager.all_imported_types [type_name]);
- return false;
- }
- }
+ if (!base.VerifyClsCompliance (ds)) {
+ return false;
}
- // Seek through generated types
- foreach (string name in RootContext.Tree.Decls.Keys) {
- if (l != name.Length)
- continue;
-
- if (String.Compare (Name, name, true, CultureInfo.InvariantCulture) == 0) {
-
- if (Name == name)
- continue;
-
- DeclSpace found_ds = RootContext.Tree.Decls[name] as DeclSpace;
- if (found_ds.IsClsCompliaceRequired (found_ds.Parent)) {
- Report.SymbolRelatedToPreviousError (found_ds.Location, found_ds.GetSignatureForError ());
- return false;
- }
- }
+ IDictionary cache = TypeManager.AllClsTopLevelTypes;
+ string lcase = Name.ToLower (System.Globalization.CultureInfo.InvariantCulture);
+ if (!cache.Contains (lcase)) {
+ cache.Add (lcase, this);
+ return true;
}
- return true;
- }
+ object val = cache [lcase];
+ if (val == null) {
+ Type t = AttributeTester.GetImportedIgnoreCaseClsType (lcase);
+ if (t == null)
+ return true;
+ Report.SymbolRelatedToPreviousError (t);
+ }
+ else {
+ if (val is PartialContainer)
+ return true;
- protected override string[] ValidAttributeTargets {
- get {
- return attribute_targets;
+ Report.SymbolRelatedToPreviousError ((DeclSpace)val);
}
+ Report.Error (3005, Location, "Identifier `{0}' differing only in case is not CLS-compliant", GetSignatureForError ());
+ return true;
}
}
}
/// <summary>
- /// Returns the IMemberContainer of the parent class or null if this
+ /// 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 parent class.
+ /// members from the base class.
/// </summary>
- IMemberContainer Parent {
+ MemberCache BaseCache {
get;
}
public readonly IMemberContainer Container;
protected Hashtable member_hash;
protected Hashtable method_hash;
-
+
/// <summary>
/// Create a new MemberCache for the given IMemberContainer `container'.
/// </summary>
Timer.IncrementCounter (CounterType.MemberCache);
Timer.StartTimer (TimerType.CacheInit);
-
-
- // If we have a parent class (we have a parent class unless we're
+ // 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.IsInterface) {
- MemberCache parent;
-
- if (Container.Parent != null)
- parent = Container.Parent.MemberCache;
- else
- parent = TypeHandle.ObjectType.MemberCache;
- member_hash = SetupCacheForInterface (parent);
- } else if (Container.Parent != null)
- member_hash = SetupCache (Container.Parent.MemberCache);
+ 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) {
+ if (!(type is TypeBuilder) && !type.IsInterface &&
+ (Container.BaseCache == null || Container.BaseCache.method_hash != null)) {
method_hash = new Hashtable ();
AddMethods (type);
}
Timer.StopTimer (TimerType.CacheInit);
}
+ 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));
+ }
+
/// <summary>
- /// Bootstrap this member cache by doing a deep-copy of our parent.
+ /// Bootstrap this member cache by doing a deep-copy of our base.
/// </summary>
- Hashtable SetupCache (MemberCache parent)
+ Hashtable SetupCache (MemberCache base_class)
{
Hashtable hash = new Hashtable ();
- IDictionaryEnumerator it = parent.member_hash.GetEnumerator ();
+ if (base_class == null)
+ return hash;
+
+ IDictionaryEnumerator it = base_class.member_hash.GetEnumerator ();
while (it.MoveNext ()) {
hash [it.Key] = ((ArrayList) it.Value).Clone ();
- }
+ }
return hash;
}
-
/// <summary>
- /// Add the contents of `new_hash' to `hash'.
+ /// Add the contents of `cache' to the member_hash.
/// </summary>
- void AddHashtable (Hashtable hash, MemberCache cache)
+ void AddCacheContents (MemberCache cache)
{
- Hashtable new_hash = cache.member_hash;
- IDictionaryEnumerator it = new_hash.GetEnumerator ();
+ IDictionaryEnumerator it = cache.member_hash.GetEnumerator ();
while (it.MoveNext ()) {
- ArrayList list = (ArrayList) hash [it.Key];
+ ArrayList list = (ArrayList) member_hash [it.Key];
if (list == null)
- hash [it.Key] = list = new ArrayList ();
+ 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];
- foreach (CacheEntry entry in (ArrayList) it.Value) {
if (entry.Container != cache.Container)
break;
list.Add (entry);
}
}
- /// <summary>
- /// Bootstrap the member cache for an interface type.
- /// Type.GetMembers() won't return any inherited members for interface types,
- /// so we need to do this manually. Interfaces also inherit from System.Object.
- /// </summary>
- Hashtable SetupCacheForInterface (MemberCache parent)
- {
- Hashtable hash = SetupCache (parent);
- TypeExpr [] ifaces = TypeManager.GetInterfaces (Container.Type);
-
- foreach (TypeExpr iface in ifaces) {
- Type itype = iface.Type;
-
- IMemberContainer iface_container =
- TypeManager.LookupMemberContainer (itype);
-
- MemberCache iface_cache = iface_container.MemberCache;
-
- AddHashtable (hash, iface_cache);
- }
-
- return hash;
- }
-
/// <summary>
/// Add all members from class `container' to the cache.
/// </summary>
{
// We need to call AddMembers() with a single member type at a time
// to get the member type part of CacheEntry.EntryType right.
- AddMembers (MemberTypes.Constructor, container);
- AddMembers (MemberTypes.Field, container);
+ if (!container.IsInterface) {
+ AddMembers (MemberTypes.Constructor, container);
+ AddMembers (MemberTypes.Field, container);
+ }
AddMembers (MemberTypes.Method, container);
AddMembers (MemberTypes.Property, container);
AddMembers (MemberTypes.Event, container);
}
// When this method is called for the current class, the list will
- // already contain all inherited members from our parent classes.
+ // 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).
AddMethods (BindingFlags.Instance | BindingFlags.NonPublic, type);
}
+ static ArrayList overrides = new ArrayList ();
+
void AddMethods (BindingFlags bf, Type type)
{
- MemberInfo [] members = type.GetMethods (bf);
+ MethodBase [] members = type.GetMethods (bf);
Array.Reverse (members);
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;
list.Add (new CacheEntry (Container, member, MemberTypes.Method, new_bf));
}
-
-
}
/// <summary>
MaskType = Constructor|Event|Field|Method|Property|NestedType
}
- protected struct CacheEntry {
+ protected class CacheEntry {
public readonly IMemberContainer Container;
public readonly EntryType EntryType;
public readonly MemberInfo Member;
this.Member = member;
this.EntryType = GetEntryType (mt, bf);
}
+
+ public override string ToString ()
+ {
+ return String.Format ("CacheEntry ({0}:{1}:{2})", Container.Name,
+ EntryType, Member);
+ }
}
/// <summary>
// `applicable' is a list of all members with the given member name `name'
- // in the current class and all its parent classes. The list is sorted in
+ // 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 parent).
+ // 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];
return null;
}
+
+ /// <summary>
+ /// 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.
+ /// </summary>
+ 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) {
+ Type[] arg_types = TypeManager.GetArgumentTypes ((PropertyInfo)entry.Member);
+ if (arg_types.Length > 0)
+ continue;
+ }
+ }
+ return entry.Member;
+ }
+ }
+ }
+ return null;
+ }
+
+ Hashtable locase_table;
+
+ /// <summary>
+ /// Builds low-case table for CLS Compliance test
+ /// </summary>
+ 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;
+ }
+ }
+
+ /// <summary>
+ /// Cls compliance check whether methods or constructors parameters differing only in ref or out, or in array rank
+ /// </summary>
+ public 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.ParameterTypes, TypeManager.GetArgumentTypes (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.IsClsCompliaceRequired (method.Parent))
+ continue;
+
+ Report.SymbolRelatedToPreviousError (entry.Member);
+ switch (result) {
+ case AttributeTester.Result.RefOutArrayError:
+ Report.Error (3006, 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.Error (3007, method.Location, "Overloaded method `{0}' differing only by unnamed array types is not CLS-compliant", method.GetSignatureForError ());
+ continue;
+ }
+
+ throw new NotImplementedException (result.ToString ());
+ }
+ }
}
}
projects / mono.git / blobdiff