Review Request 129511: use QSettings::IniFormat everywhere

[Kåre Särs]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129511/#review100980
-----------------------------------------------------------



I think that the decision to go for "ini-format-always" is up to the Sonnet guys, and then we have to adapt here accordingly. We just read the global setting here and it has to be the same as Sonnet is using otherwise we only get the default value. 

The settings handling in Sonnet is not optimal. The applications can not have their own private Sonnet settings it is always global :(

At one point I was planning to create a patch to expose the Sonnet setting object so that an application could set it's own private settings, but I never came around to doing it...

- Kåre Särs


On Nov. 20, 2016, 8:10 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129511/
> -----------------------------------------------------------
> 
> (Updated Nov. 20, 2016, 8:10 p.m.)
> 
> 
> Review request for Kate, KDE Software on Mac OS X and Kåre Särs.
> 
> 
> Repository: ktexteditor
> 
> 
> Description
> -------
> 
> A while back already there was a consensus on the frameworks-devel ML (backed by a few related RRs) that it was best to standardise on the `QSettings::IniFormat` to reduce the number of unnecessary platform differences, and to avoid storing settings in the registry on MS Windows.
> 
> I've been running KTextEditor with this patch for almost a year; somehow I forgot to submit it for review until now.
> 
> 
> Diffs
> -----
> 
>   src/utils/kateconfig.cpp 30dd4fe 
> 
> Diff: https://git.reviewboard.kde.org/r/129511/diff/
> 
> 
> Testing
> -------
> 
> Tested since December 2015 on OS X 10.9.5 .
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20161120/4225303e/attachment.html>