Review Request 112583: Steps to make KConfigWidgets compile Standalone

[Aurélien Gâteau]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112583/#review39649
-----------------------------------------------------------


Instead of adding methods to get buttons and combo boxes from KSelectAction, which is an implementation detail to me, what about this:

- Add KSelectAction::insertAction(QAction* before, QAction*) (because what KRecentFilesAction::addAction() really does is insert(firstAction, action)) 

- Implement KRecentFilesAction::addAction(QAction* action, const QUrl& url, const QString& name) like this:

  KSelectAction::insertAction(actions().first(), action);
  d->m_shortNames.insert(action, name);
  d->m_urls.insert(action, url);

Could this work?

- Aurélien Gâteau


On Sept. 7, 2013, 9:45 p.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112583/
> -----------------------------------------------------------
> 
> (Updated Sept. 7, 2013, 9:45 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 b388df94baa042e8230f2795f35dd378117c6ab6 
> 
> 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/20130909/c32c6c2e/attachment.html>