Mercurial > dwindows
diff win/dw.c @ 1377:b6249d66404c
Cleanup signal handlers when calling dw_menu_delete_item() on OS/2 and Windows...
when the ID is auto-generated. It is unsafe to do it for specified IDs because there
may be more than one menu item with an ID... however autogenerated IDs should be unique.
author | bsmith@81767d24-ef19-dc11-ae90-00e081727c95 |
---|---|
date | Wed, 23 Nov 2011 00:49:12 +0000 |
parents | 896347a9be19 |
children | cc66b6d4b74e |
line wrap: on
line diff
--- a/win/dw.c Tue Nov 22 22:20:39 2011 +0000 +++ b/win/dw.c Wed Nov 23 00:49:12 2011 +0000 @@ -5215,9 +5215,14 @@ if ( IsWindow(menux) && !IsMenu(mymenu) ) mymenu = (HMENU)dw_window_get_data(menux, "_dw_menu"); - if ( DeleteMenu(mymenu, id, MF_BYCOMMAND) == 0 ) + if ( mymenu == 0 || DeleteMenu(mymenu, id, MF_BYCOMMAND) == 0 ) return DW_ERROR_UNKNOWN; + /* If the ID was autogenerated it is safe to remove it */ + if(id >= 30000) + dw_signal_disconnect_by_window((HWND)id); + + /* Make sure the menu is redrawn if needed */ if( (HMENU)menux != mymenu ) DrawMenuBar(menux); return DW_ERROR_NONE;