Review Request: port Sonnet to QSettings
Kevin Krammer
krammer at kde.org
Mon Dec 17 22:38:18 GMT 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107791/#review23643
-----------------------------------------------------------
IMHO this is wrong.
Not code wise but conceptual. As far as I understand QSettings is basically deprecated, it is just not official marked as such because there is no replacement. This would be porting away from a fully functional, way more advanced and maintained settings.
If the sole goal of this endavor is to remove the KConfig dependency than this ought to be done by either having an interface that can be implemented through KConfig or by passing values as QVariant maps or hashes.
- Kevin Krammer
On Dec. 17, 2012, 9:15 p.m., Martin Tobias Holmedahl Sandsmark wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107791/
> -----------------------------------------------------------
>
> (Updated Dec. 17, 2012, 9:15 p.m.)
>
>
> Review request for KDE Frameworks, kdelibs and David Faure.
>
>
> Description
> -------
>
> Ported everything away from KConfig to QSettings.
>
> I couldn't really find any users of the ::save function, so I think the source incompatible change would be worth it. Alternatively we could add a deprecated dummy function that takes in a KConfig object and just ignores it, and uses QSettings.
>
>
> Diffs
> -----
>
> kdeui/sonnet/configdialog.h 7c4993b
> kdeui/sonnet/configdialog.cpp 625441b
> kdeui/sonnet/configwidget.h 023b659
> kdeui/sonnet/configwidget.cpp 549d5af
> kdeui/sonnet/highlighter.cpp 6cbb14c
> kdeui/widgets/ktextedit.cpp 71d2a9f
> staging/sonnet/src/core/backgroundchecker.h f0da3a3
> staging/sonnet/src/core/backgroundchecker.cpp dc05b94
> staging/sonnet/src/core/globals.cpp bf4f504
> staging/sonnet/src/core/loader.cpp 887aee5
> staging/sonnet/src/core/settings.cpp 59cb593
> staging/sonnet/src/core/settings_p.h e14bad7
> staging/sonnet/src/core/speller.h 37dd82f
> staging/sonnet/src/core/speller.cpp f831f55
>
> Diff: http://git.reviewboard.kde.org/r/107791/diff/
>
>
> Testing
> -------
>
> it builds.
>
>
> Thanks,
>
> Martin Tobias Holmedahl Sandsmark
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20121217/5301bf07/attachment.htm>
More information about the kde-core-devel
mailing list