Don't call CloseHandle() on the HMODULE returned from GetModuleHandle().
2003-09-02 Tor Lillqvist <tml@iki.fi> * gdk/win32/gdkdisplay-win32.c (gdk_display_open): Don't call CloseHandle() on the HMODULE returned from GetModuleHandle(). Didn't cause any harm, but didn't do anything useful either. When running a GTK+ program under MS's debugger, the CloseHandle() call would cause a "first-chance exception" in ntdll.dll to show up. (The exception is caught appropriately if you let it proceed, but it confuses the person using the debugger). Thanks to Bruce Hochstetler for noticing.
This commit is contained in:
committed by
Tor Lillqvist
parent
ed7926ac39
commit
6f1fb7d058
11
ChangeLog
11
ChangeLog
@@ -1,3 +1,14 @@
|
||||
2003-09-02 Tor Lillqvist <tml@iki.fi>
|
||||
|
||||
* gdk/win32/gdkdisplay-win32.c (gdk_display_open): Don't call
|
||||
CloseHandle() on the HMODULE returned from GetModuleHandle().
|
||||
Didn't cause any harm, but didn't do anything useful either. When
|
||||
running a GTK+ program under MS's debugger, the CloseHandle() call
|
||||
would cause a "first-chance exception" in ntdll.dll to show up.
|
||||
(The exception is caught appropriately if you let it proceed, but
|
||||
it confuses the person using the debugger). Thanks to Bruce
|
||||
Hochstetler for noticing.
|
||||
|
||||
Tue Sep 2 21:01:19 2003 Kristian Rietveld <kris@gtk.org>
|
||||
|
||||
* gtk/gtktreeview.c (gtk_tree_view_button_press): revert the change
|
||||
|
||||
@@ -1,3 +1,14 @@
|
||||
2003-09-02 Tor Lillqvist <tml@iki.fi>
|
||||
|
||||
* gdk/win32/gdkdisplay-win32.c (gdk_display_open): Don't call
|
||||
CloseHandle() on the HMODULE returned from GetModuleHandle().
|
||||
Didn't cause any harm, but didn't do anything useful either. When
|
||||
running a GTK+ program under MS's debugger, the CloseHandle() call
|
||||
would cause a "first-chance exception" in ntdll.dll to show up.
|
||||
(The exception is caught appropriately if you let it proceed, but
|
||||
it confuses the person using the debugger). Thanks to Bruce
|
||||
Hochstetler for noticing.
|
||||
|
||||
Tue Sep 2 21:01:19 2003 Kristian Rietveld <kris@gtk.org>
|
||||
|
||||
* gtk/gtktreeview.c (gtk_tree_view_button_press): revert the change
|
||||
|
||||
@@ -1,3 +1,14 @@
|
||||
2003-09-02 Tor Lillqvist <tml@iki.fi>
|
||||
|
||||
* gdk/win32/gdkdisplay-win32.c (gdk_display_open): Don't call
|
||||
CloseHandle() on the HMODULE returned from GetModuleHandle().
|
||||
Didn't cause any harm, but didn't do anything useful either. When
|
||||
running a GTK+ program under MS's debugger, the CloseHandle() call
|
||||
would cause a "first-chance exception" in ntdll.dll to show up.
|
||||
(The exception is caught appropriately if you let it proceed, but
|
||||
it confuses the person using the debugger). Thanks to Bruce
|
||||
Hochstetler for noticing.
|
||||
|
||||
Tue Sep 2 21:01:19 2003 Kristian Rietveld <kris@gtk.org>
|
||||
|
||||
* gtk/gtktreeview.c (gtk_tree_view_button_press): revert the change
|
||||
|
||||
@@ -1,3 +1,14 @@
|
||||
2003-09-02 Tor Lillqvist <tml@iki.fi>
|
||||
|
||||
* gdk/win32/gdkdisplay-win32.c (gdk_display_open): Don't call
|
||||
CloseHandle() on the HMODULE returned from GetModuleHandle().
|
||||
Didn't cause any harm, but didn't do anything useful either. When
|
||||
running a GTK+ program under MS's debugger, the CloseHandle() call
|
||||
would cause a "first-chance exception" in ntdll.dll to show up.
|
||||
(The exception is caught appropriately if you let it proceed, but
|
||||
it confuses the person using the debugger). Thanks to Bruce
|
||||
Hochstetler for noticing.
|
||||
|
||||
Tue Sep 2 21:01:19 2003 Kristian Rietveld <kris@gtk.org>
|
||||
|
||||
* gtk/gtktreeview.c (gtk_tree_view_button_press): revert the change
|
||||
|
||||
@@ -1,3 +1,14 @@
|
||||
2003-09-02 Tor Lillqvist <tml@iki.fi>
|
||||
|
||||
* gdk/win32/gdkdisplay-win32.c (gdk_display_open): Don't call
|
||||
CloseHandle() on the HMODULE returned from GetModuleHandle().
|
||||
Didn't cause any harm, but didn't do anything useful either. When
|
||||
running a GTK+ program under MS's debugger, the CloseHandle() call
|
||||
would cause a "first-chance exception" in ntdll.dll to show up.
|
||||
(The exception is caught appropriately if you let it proceed, but
|
||||
it confuses the person using the debugger). Thanks to Bruce
|
||||
Hochstetler for noticing.
|
||||
|
||||
Tue Sep 2 21:01:19 2003 Kristian Rietveld <kris@gtk.org>
|
||||
|
||||
* gtk/gtktreeview.c (gtk_tree_view_button_press): revert the change
|
||||
|
||||
@@ -119,8 +119,6 @@ gdk_display_open (const gchar *display_name)
|
||||
p_EnumDisplayMonitors = (t_EnumDisplayMonitors) GetProcAddress (user32, "EnumDisplayMonitors");
|
||||
p_GetMonitorInfoA = (t_GetMonitorInfoA) GetProcAddress (user32, "GetMonitorInfoA");
|
||||
|
||||
CloseHandle (user32);
|
||||
|
||||
if (p_EnumDisplayMonitors != NULL && p_GetMonitorInfoA != NULL)
|
||||
{
|
||||
gint i, index;
|
||||
|
||||
Reference in New Issue
Block a user