jgrulich added inline comments.
INLINE COMMENTS
> dcbsetting.cpp:491
> +
> + if (priorityFlowControl(7) > 0) {
> + setting.insert(QLatin1String(NM_SETTING_DCB_PRIORITY_FLOW_CONTROL),
> QVariant::fromValue(priorityFlowControl()));
I think all those should be added all the time, checking whether the last one
is higher can sometime avoid adding them to the map when one of previous ones
is set to non-default value.
> dcbsetting.cpp:527
> + dbg.nospace() << NM_SETTING_DCB_PRIORITY_FLOW_CONTROL_FLAGS << ": " <<
> setting.priorityFlowControlFlags() << '\n';
> +
> + return dbg.maybeSpace();
You don't print priorityBandwith, priororityGroupBandwidth and others.
> dcbsetting.h:43
> +
> + enum DcbFlagsType {
> + None = 0,
Rename to DcbFlagType
> dcbsetting.h:124
> +};
> +
> +NETWORKMANAGERQT_EXPORT QDebug operator<<(QDebug dbg, const DcbSetting
> &setting);
Please also add Q_DECLARE_OPERATORS_FOR_FLAGS(DcbSetting::DcbFlags)
> setting.h:97
> Q_DECLARE_FLAGS(SecretFlags, SecretFlagType)
> -
> +
> enum MacAddressRandomization {
Remove trailing spaces. They are not visible here, but once I apply this patch,
I see them in KDevelop.
REPOSITORY
R282 NetworkManagerQt
REVISION DETAIL
https://phabricator.kde.org/D17425
To: pranavgade, jgrulich
Cc: cfeck, kde-frameworks-devel, michaelh, ngraham, bruns
