[Win32] Fix leaks in Win32 Cursor property
authorFraser Waters <frassle@gmail.com>
Mon, 10 Aug 2015 20:57:05 +0000 (21:57 +0100)
committerFraser Waters <frassle@gmail.com>
Mon, 10 Aug 2015 20:57:05 +0000 (21:57 +0100)
commit33c06d4b35de7e49d07f2b2d60c26fec337a894e
tree34281cc9280490fa33cea6febf8a629bb5478e19
parentbf3267c4a90d43731159abcc25282a0b7cb9bb37
[Win32] Fix leaks in Win32 Cursor property

GetIconInfo
(https://msdn.microsoft.com/en-us/library/windows/desktop/ms648070(v=vs.85).aspx)
creates bitmaps that must be deleted after the call to
CreateIconIndirect, which copies the bitmaps to the icon it creates.

bmpIcon created by Bitmap.GetHicon is now destroyed after being used.

The return value of SetCursor was used to retrieve the last cursor, as
it could have been set by another library (Some UI libraries change the
cursor using the .net Cursor) this could of leaked the cursor we created
and now lost track of. We now delete the handle we had set, not the one
returned by SetCursor.
Source/OpenTK/Platform/Windows/API.cs
Source/OpenTK/Platform/Windows/WinGLNative.cs