Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Do not actually reconnect PG event handlers when attempting to reconnect them to the same window.

Browse Source 
When in wxPropertyGridManager::ReconnectEventHandlers() new window ID is the same as old window ID then there is no need to do anything with handlers. An assertion warning is displayed in this case to notify about this unusual (and maybe unintended) situation.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@78321 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Artur Wieczorek
2014-12-27 11:30:45 +00:00
parent 6ecf4dcbfa
commit 1b0d09e5be
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/propgrid/manager.cpp
View File

@@ -1903,7 +1903,8 @@ void wxPropertyGridManager::SetPageSplitterLeft(int page, bool subProps)
void wxPropertyGridManager::ReconnectEventHandlers(wxWindowID oldId, wxWindowID newId) void wxPropertyGridManager::ReconnectEventHandlers(wxWindowID oldId, wxWindowID newId)
{ {
wxASSERT( oldId != newId ); wxCHECK_RET( oldId != newId,
wxT("Attempting to reconnect event handlers to the same window"));
if (oldId != wxID_NONE) if (oldId != wxID_NONE)
{ {