jgrulich added inline comments.

INLINE COMMENTS

> teamportsettingtest.cpp:103
> +        if (it.key() == QLatin1String(NM_SETTING_TEAM_PORT_LINK_WATCHERS)) {
> +            NMVariantMapList list = it_1.value().value<NMVariantMapList>();
> +            NMVariantMapList list1 = 
> map1.value(it_1.key()).value<NMVariantMapList>();

Why don't you skip this and just use NMVariantMapList list = 
map.value(QLatin1String(NM_SETTING_TEAM_PORT_LINK_WATCHERS).value<NMVariantMapList>()?
 same for the second one. You don't need to go through the map to get 
"link-watchers" property.

REVISION DETAIL
  https://phabricator.kde.org/D17342

To: pranavgade, jgrulich
Cc: kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to