Review Request 112583: Steps to make KConfigWidgets compile Standalone
Aurélien Gâteau
agateau at kde.org
Wed Sep 11 16:13:50 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112583/#review39840
-----------------------------------------------------------
A few nitpicks, but I like this version better.
tier1/kwidgetsaddons/src/kselectaction.h
<http://git.reviewboard.kde.org/r/112583/#comment29377>
Missing @since 5.0
tier1/kwidgetsaddons/src/kselectaction.cpp
<http://git.reviewboard.kde.org/r/112583/#comment29376>
To avoid duplication you could reimplement KSelectAction::addAction(act) as insertAction(0, act)
You might also want to make KSelectAction::addAction() not virtual, so that there is only one way to change the class behavior when actions are added/inserted.
- Aurélien Gâteau
On Sept. 11, 2013, 4:55 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112583/
> -----------------------------------------------------------
>
> (Updated Sept. 11, 2013, 4:55 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Description
> -------
>
> KConfigWidgets relies on a private header in KWidgetAddons. This prevents KConfigWidgets building on it's own
>
> By exposing the two of these private members in protected methods, we can solve this problem.
>
>
> Commit log below:
> --------------
>
> change target link libraries to ${lib_LIBRARIES} format
> --
>
> Remove inheritance on KSelectionActionPrivate in KRecentFileActions
>
> This is in a different framework so cannot be used when building
> standalone.
>
> --
> Expose associated comboboxes and toolbuttons in KSelectAction
>
>
> Diffs
> -----
>
> staging/kconfigwidgets/src/CMakeLists.txt dad6bcc140cb4fe56380bd44ea007fe1f06edace
> staging/kconfigwidgets/src/krecentfilesaction.h edf77a24866018134d9c5a200c6e35213742c555
> staging/kconfigwidgets/src/krecentfilesaction.cpp 9aa5f8bd83d064402dd6333c7cd89a388d373fe1
> staging/kconfigwidgets/src/krecentfilesaction_p.h 8b3cc8f82b248cc9ed8a1f1be69cf4160f97378c
> tier1/kwidgetsaddons/src/kselectaction.h 07afaa960d2be4877aeef0ff074d0fd20c7b7f40
> tier1/kwidgetsaddons/src/kselectaction.cpp 3aa079731cd19ccad4f6a6344ebe5d6921d0bd7e
>
> Diff: http://git.reviewboard.kde.org/r/112583/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130911/df058a28/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list