Author: atsushi
Date: 2005-12-09 05:55:25 -0500 (Fri, 09 Dec 2005)
New Revision: 54154
Added:
trunk/mcs/tests/test-xml-050-ref.xml
trunk/mcs/tests/test-xml-050.cs
Modified:
trunk/mcs/mcs/ChangeLog
trunk/mcs/mcs/doc.cs
trunk/mcs/tests/ChangeLog
trunk/mcs/tests/known-issues-gmcs
Log:
2005-12-09 Atsushi Enomoto <[EMAIL PROTECTED]>
* doc.cs : so it turned out that we cannot skip override check for
interface members. Fixed bug #76954.
* test-xml-050-ref.xml, test-xml-050.cs : new test for bug #76954.
* known-issues-gmcs : fixed filename and removed test-60.cs.
Modified: trunk/mcs/mcs/ChangeLog
===================================================================
--- trunk/mcs/mcs/ChangeLog 2005-12-09 10:24:45 UTC (rev 54153)
+++ trunk/mcs/mcs/ChangeLog 2005-12-09 10:55:25 UTC (rev 54154)
@@ -1,5 +1,10 @@
2005-12-09 Atsushi Enomoto <[EMAIL PROTECTED]>
+ * doc.cs : so it turned out that we cannot skip override check for
+ interface members. Fixed bug #76954.
+
+2005-12-09 Atsushi Enomoto <[EMAIL PROTECTED]>
+
* cs-tokenizer.cs : fixed bug #75984:
- #warning and #error should not be handled when the source line
is disabled.
Modified: trunk/mcs/mcs/doc.cs
===================================================================
--- trunk/mcs/mcs/doc.cs 2005-12-09 10:24:45 UTC (rev 54153)
+++ trunk/mcs/mcs/doc.cs 2005-12-09 10:55:25 UTC (rev 54154)
@@ -383,22 +383,9 @@
{
if (ml == null)
return empty_member_infos;
- if (type.IsInterface)
- return ml;
ArrayList al = new ArrayList (ml.Length);
for (int i = 0; i < ml.Length; i++) {
- // Interface methods which are returned
- // from the filter must exist in the
- // target type (if there is only a
- // private implementation, then the
- // filter should not return it.)
- // This filtering is required to
- // deambiguate results.
- //
- // It is common to properties, so check it here.
- if (ml [i].DeclaringType.IsInterface)
- continue;
MethodBase mx = ml [i] as MethodBase;
PropertyInfo px = ml [i] as PropertyInfo;
if (mx != null || px != null) {
Modified: trunk/mcs/tests/ChangeLog
===================================================================
--- trunk/mcs/tests/ChangeLog 2005-12-09 10:24:45 UTC (rev 54153)
+++ trunk/mcs/tests/ChangeLog 2005-12-09 10:55:25 UTC (rev 54154)
@@ -1,5 +1,10 @@
2005-12-09 Atsushi Enomoto <[EMAIL PROTECTED]>
+ * test-xml-050-ref.xml, test-xml-050.cs : new test for bug #76954.
+ * known-issues-gmcs : fixed filename and removed test-60.cs.
+
+2005-12-09 Atsushi Enomoto <[EMAIL PROTECTED]>
+
* test-476.cs : test for bug #75984.
* test-60.cs : unrecognized directive should be handled as an error
even in a disable line (the behavior has changed in 2.0).
Modified: trunk/mcs/tests/known-issues-gmcs
===================================================================
--- trunk/mcs/tests/known-issues-gmcs 2005-12-09 10:24:45 UTC (rev 54153)
+++ trunk/mcs/tests/known-issues-gmcs 2005-12-09 10:55:25 UTC (rev 54154)
@@ -10,5 +10,5 @@
test-anon-27.cs
test-xml-027.cs
test-465.cs IGNORE # need to fix the path separator to work both on Unix
and Windows
-test-60.cs
-tset-476.cs
+test-476.cs
+
Added: trunk/mcs/tests/test-xml-050-ref.xml
===================================================================
--- trunk/mcs/tests/test-xml-050-ref.xml 2005-12-09 10:24:45 UTC (rev
54153)
+++ trunk/mcs/tests/test-xml-050-ref.xml 2005-12-09 10:55:25 UTC (rev
54154)
@@ -0,0 +1,52 @@
+<?xml version="1.0"?>
+<doc>
+ <assembly>
+ <name>test-xml-050</name>
+ </assembly>
+ <members>
+ <member name="T:EntryPoint">
+ <summary>
+ <see cref="M:IB.Execute"/>
+ <see cref="P:IB.Name"/>
+ <see cref="M:B.Execute"/>
+ <see cref="P:B.Name"/>
+ </summary>
+ </member>
+ <member name="T:IA">
+ <summary />
+ </member>
+ <member name="M:IA.Execute">
+ <summary />
+ </member>
+ <member name="P:IA.Name">
+ <summary />
+ </member>
+ <member name="T:IB">
+ <summary />
+ </member>
+ <member name="M:IB.Execute">
+ <summary />
+ </member>
+ <member name="P:IB.Name">
+ <summary />
+ </member>
+ <member name="T:A">
+ <summary />
+ </member>
+ <member name="M:A.Execute">
+ <summary />
+ </member>
+ <member name="P:A.Name">
+ <summary />
+ </member>
+ <member name="T:B">
+ <summary />
+ </member>
+ <member name="M:B.Execute">
+ <summary />
+ </member>
+ <member name="P:B.Name">
+ <summary />
+ </member>
+ </members>
+</doc>
Property changes on: trunk/mcs/tests/test-xml-050-ref.xml
___________________________________________________________________
Name: svn:executable
+ *
Added: trunk/mcs/tests/test-xml-050.cs
===================================================================
--- trunk/mcs/tests/test-xml-050.cs 2005-12-09 10:24:45 UTC (rev 54153)
+++ trunk/mcs/tests/test-xml-050.cs 2005-12-09 10:55:25 UTC (rev 54154)
@@ -0,0 +1,61 @@
+// Compiler options: -doc:xml-050.cs -warnaserror
+// see bug #76954.
+// NOTE: It might got broken after some /doc related merge.
+/// <summary>
+/// <see cref="IB.Execute ()" />
+/// <see cref="IB.Name" />
+/// <see cref="B.Execute ()" />
+/// <see cref="B.Name" />
+/// </summary>
+public class EntryPoint {
+ static void Main () {
+ }
+}
+
+/// <summary />
+public interface IA {
+ /// <summary />
+ string Name {
+ get;
+ }
+
+ /// <summary />
+ string Execute ();
+}
+
+/// <summary />
+public interface IB : IA {
+ /// <summary />
+ new int Name {
+ get;
+ }
+
+ /// <summary />
+ new int Execute ();
+}
+
+/// <summary />
+public class A {
+ /// <summary />
+ public string Name {
+ get { return null; }
+ }
+
+ /// <summary />
+ public string Execute () {
+ return null;
+ }
+}
+
+/// <summary />
+public class B : A {
+ /// <summary />
+ public new int Name {
+ get { return 0; }
+ }
+
+ /// <summary />
+ public new int Execute () {
+ return 0;
+ }
+}
_______________________________________________
Mono-patches maillist - [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches
