D24490: Make kconfig_compiler generate ctors with the optional parent arg
Aleix Pol Gonzalez
noreply at phabricator.kde.org
Tue Oct 8 13:29:52 BST 2019
apol added a comment.
+1 in spirit.
INLINE COMMENTS
> test_notifiers.cpp.ref:7
> +TestNotifiers::TestNotifiers( int Number, QObject *parent )
> : KConfigSkeleton( QStringLiteral( "test7rc" ) )
> , mParamNumber(Number)
Maybe it would be better to use the parent argument in KConfigSkeleton?
explicit KConfigSkeleton(const QString &configname = QString(), QObject *parent = nullptr);
REPOSITORY
R237 KConfig
REVISION DETAIL
https://phabricator.kde.org/D24490
To: ervin, #plasma, #frameworks, dfaure, mart
Cc: apol, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191008/78fd1cb4/attachment.html>
More information about the Kde-frameworks-devel
mailing list