Don't close external open window

You discovered a bug in the engine, and you are sure that it is not a problem of your code? Just post it in here. Please read the bug posting guidelines first.
Post Reply
ldrrn
Posts: 4
Joined: Sun Jul 20, 2008 9:14 am

Don't close external open window

Post by ldrrn »

possible to change

Code: Select all

//! notifies the device that it should close itself
void CIrrDeviceWin32::closeDevice()
{
	MSG msg;
	PeekMessage(&msg, NULL, WM_QUIT, WM_QUIT, PM_REMOVE);
	PostQuitMessage(0);
	PeekMessage(&msg, NULL, WM_QUIT, WM_QUIT, PM_REMOVE);
	DestroyWindow(HWnd);
}
to

Code: Select all

//! notifies the device that it should close itself
void CIrrDeviceWin32::closeDevice()
{
	MSG msg;
	PeekMessage(&msg, NULL, WM_QUIT, WM_QUIT, PM_REMOVE);
	PostQuitMessage(0);
	PeekMessage(&msg, NULL, WM_QUIT, WM_QUIT, PM_REMOVE);
	if (!ExternalWindow) DestroyWindow(HWnd);
}
i think it is more logical to let external module to manage it own window handle. Thanks
[/code]
hybrid
Admin
Posts: 14143
Joined: Wed Apr 19, 2006 9:20 pm
Location: Oldenburg(Oldb), Germany
Contact:

Post by hybrid »

Oh, we have a similar request for the Linux driver. I thought that the win32 driver already does handle this situation. I guess we need an extended testcase here... Thanks for reporting, we're investigating this issue. Maybe you should add your information to the bug tracker: https://sourceforge.net/tracker2/?func= ... tid=540676
ldrrn
Posts: 4
Joined: Sun Jul 20, 2008 9:14 am

Post by ldrrn »

comment added :)
Post Reply