matthew/ostinato
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

During stream configuration - when deleting a protocol, delete its associated widget also, otherwise if a newly allocated protocol gets the same heap address as the old protocol, the old protocol's widget gets associated with the new protocol.

Browse Source 
Fixes issue 131
This commit is contained in:
Srivats P. 2014-11-07 20:21:02 +05:30
parent 943eb49b66
commit 50f20a5452
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
client/streamconfigdialog.cpp
View File

@ -516,6 +516,8 @@ void StreamConfigDialog::on_tbDelete_clicked()
Q_CHECK_PTR(p);
_iter->remove();
// Free both protocol and associated widget
delete _protocolWidgets.take(p);
delete p;
updateSelectProtocolsAdvancedWidget();
@ -835,6 +837,8 @@ void StreamConfigDialog::__updateProtocol(int level, int newId)
newId, mpStream));
else
_iter->remove();
// Free both protocol and associated widget
delete _protocolWidgets.take(p);
delete p;
if (level == ProtoPayload)
{
@ -842,6 +846,8 @@ void StreamConfigDialog::__updateProtocol(int level, int newId)
{
p = _iter->next();
_iter->remove();
// Free both protocol and associated widget
delete _protocolWidgets.take(p);
delete p;
}
}