[Mono-bugs] [Bug 665171] AvoidVisibleFieldsRule only looks at [SuppressMessage] metadata at get / set or add / remove

[bugzilla_noreply at novell.com]

https://bugzilla.novell.com/show_bug.cgi?id=665171

https://bugzilla.novell.com/show_bug.cgi?id=665171#c3


Sebastien Pouliot <spouliot at novell.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEEDINFO
       InfoProvider|                            |ethan_j_brown at hotmail.com

--- Comment #3 from Sebastien Pouliot <spouliot at novell.com> 2011-01-20 00:55:09 UTC ---
> the same issue must also exists on properties.
.. and I should have read the bug's summary :)

Now this looks like bug #665179. AFAICT there's no event here, only a field -
so no add/remove on which I can apply the [SuppressMessage]. Now this
field/event mixup could maybe could maybe become a rule ;-)

    [SuppressMessage("Microsoft.Usage",
"CA2211:NonConstantFieldsShouldNotBeVisible", Justification="Add/remove is
thread-safe for events in .NET.")]
    public static EventHandler<SimpleAuthenticationFailureEventArgs>
AuthenticationFailure;


p.s. that the code in GIT did not handle properties/events correctly (to copy
the ignore data). I'll be pushing my fix (to master) later tonight.

-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.