extcap: Fix required indicator

Fix required indicator on reset and change an assert to a simple
drop-out

Change-Id: I355980223f213fef8ee4c6ac7d6bcb0fce1a7913
Reviewed-on: https://code.wireshark.org/review/27416
Petri-Dish: Roland Knall <rknall@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Roland Knall <rknall@gmail.com>
This commit is contained in:
Roland Knall 2018-05-09 09:55:32 +02:00 committed by Roland Knall
parent bba0041bfd
commit a5e3414fc6
1 changed files with 9 additions and 1 deletions

View File

@ -315,7 +315,12 @@ void ExtcapOptionsDialog::updateWidgets()
if ( argument->group().length() > 0 )
groupKey = argument->group();
Q_ASSERT(layouts.keys().contains(groupKey));
/* Skip non-assigned group keys, this happens if the configuration of the extcap is faulty */
if ( ! layouts.keys().contains(groupKey) )
{
++iter;
continue;
}
QGridLayout * layout = ((QGridLayout *)layouts[groupKey]->layout());
lblWidget = argument->createLabel((QWidget *)this);
@ -531,7 +536,10 @@ void ExtcapOptionsDialog::resetValues()
/* this stores all values to the preferences */
if ( doStore )
{
storeValues();
anyValueChanged();
}
}
}