Review Request: Replicate the existing KConfigDialog constructor and change one argument type

[Commit Hook]

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


This review has been submitted with commit e249c59263cd3a9bf59c01f3884d6eabc29df3ef by Laszlo Papp to branch KDE/4.8.

- Commit Hook


On Feb. 21, 2012, 3:56 p.m., Laszlo Papp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103716/
> -----------------------------------------------------------
> 
> (Updated Feb. 21, 2012, 3:56 p.m.)
> 
> 
> Review request for kdelibs and David Faure.
> 
> 
> Description
> -------
> 
> Use case: there are applications like kanagram which would be nice to have
> running on several platforms, like handsets; for instance Harmattan on N9. It
> would be nice to use the same settings code generation in certain cases for all
> the platforms since the additions of KConfigSkeleton on the top of
> KCoreConfigSkeleton are the font and color settings. These are currently not
> used in many KDE applications. Hence, it should not be mandatory. The kdeui
> module is unlikely welcome on mobile platforms, especially in appstores with
> its sizes and complexity for no real need.
> 
> KConfigDialogManager has apparently already two constructors; one with
> KConfigSkeleton argument type, and yet another with KCoreConfigSkeleton. It
> looks like a situation where the KCoreConfigSkeleton version was added later.
> 
> KConfigDialog does not have a constructor yet with KCoreConfigSkeleton argument
> type yet; it has probably somehow been missed so far. Changing the current
> constructor to KCoreConfigSkeleton usage is not possible in the 4.X major
> version because of the consequences (ABI breakage). Thereby, the freshly
> replacated constructor. The proper fix can be filed against frameworks where
> there is only one, and properly working constructor.
> 
> 
> Diffs
> -----
> 
>   kdeui/dialogs/kconfigdialog.h 2ac0eda 
>   kdeui/dialogs/kconfigdialog.cpp e815e54 
> 
> Diff: http://git.reviewboard.kde.org/r/103716/diff/
> 
> 
> Testing
> -------
> 
> On Archlinux (build test only)
> 
> 
> Thanks,
> 
> Laszlo Papp
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20120221/60a8b80a/attachment.htm>