[Mono-list] stubs for Microsoft.Win32

[Gaurav Vaish]

Hello Guenther,

    I have not gone through the other classes, but a peek at "SystemEvents"
shows that it is pretty incomplete and quite wrong.

    With all the event handlers, you should have "add/remove", something
like:

    public static event EventHandler DisplaySettingsChanged
    {
        add
        {
            // do something
        }
        remove
        {
            // do some other thing
        }
    }

    A simple
        public static event EventHandler DisplaySettingsChanged;

    will not function.
    Or else, if you plan to add it at a later stage (as most of us do -
first stub the methods), please do add a "MonoTODO" attribute and also make
the methods throw "NotImplementedException".

    I hope you get my point. If not, "grep NotImplementedException
<some-class-dir>/*.cs". ;-)

    Otherwise, I think the delegates and the argument classes look pretty
decent.

Happy hacking,
Gaurav
http://mastergaurav.virtualave.net/iitk
----------------------------
----- Original Message -----
From: "Guenther Roith" <groith@tcrz.net>
To: "mono-list" <mono-list@ximian.com>
Sent: Sunday, September 15, 2002 20:14
Subject: [Mono-list] stubs for Microsoft.Win32


> Hello!
>
> I've dstarted Microsoft.Win32 with some stubs for the part of the
namespace
> Microsoft.Win32,that lives in System.dll.
> Can I commit?
>
> Johannes
>