Author: marek
Date: 2007-11-07 10:17:31 -0500 (Wed, 07 Nov 2007)
New Revision: 89108

Modified:
   trunk/mcs/mcs/ChangeLog
   trunk/mcs/mcs/class.cs
Log:
2007-11-07  Marek Safar  <[EMAIL PROTECTED]>

        A fix for bug #324222
        * class.cs: Report all non-used event fields.
        


Modified: trunk/mcs/mcs/ChangeLog
===================================================================
--- trunk/mcs/mcs/ChangeLog     2007-11-07 15:15:30 UTC (rev 89107)
+++ trunk/mcs/mcs/ChangeLog     2007-11-07 15:17:31 UTC (rev 89108)
@@ -1,5 +1,10 @@
 2007-11-07  Marek Safar  <[EMAIL PROTECTED]>
 
+       A fix for bug #324222
+       * class.cs: Report all non-used event fields.
+       
+2007-11-07  Marek Safar  <[EMAIL PROTECTED]>
+
        A fix for bug #325161
        * cs-parser.jay, gcs-parser.jay, decl.cs: Implemented namespace alias
        qualifier for generic types.

Modified: trunk/mcs/mcs/class.cs
===================================================================
--- trunk/mcs/mcs/class.cs      2007-11-07 15:15:30 UTC (rev 89107)
+++ trunk/mcs/mcs/class.cs      2007-11-07 15:17:31 UTC (rev 89108)
@@ -2842,9 +2842,8 @@
 
                        if ((events != null) && Report.WarningLevel >= 3) {
                                foreach (Event e in events){
-                                       if ((e.ModFlags & 
Modifiers.Accessibility) != Modifiers.PRIVATE)
-                                               continue;
-
+                                       // Note: The event can be assigned from 
same class only, so we can report
+                                       // this warning for all accessibility 
modes
                                        if ((e.caching_flags & Flags.IsUsed) == 
0)
                                                Report.Warning (67, 3, 
e.Location, "The event `{0}' is never used", e.GetSignatureForError ());
                                }
@@ -7246,9 +7245,7 @@
                        if (!base.Define ())
                                return false;
 
-                       if (IsExplicitImpl)
-                               SetMemberIsUsed ();
-
+                       SetMemberIsUsed ();
                        return true;
                }
 

_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches

Reply via email to