Review Request 112590: Fix todo marker words and detailed completion settings not persistent across application restart.
Andreas Pakulat
apaku at gmx.de
Sun Sep 8 12:28:45 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112590/#review39580
-----------------------------------------------------------
I think you didn't understand the existing code. At least in master the todo marker words are written out explicitly, the kcmodule support for named widgets is intentionally not used because the string from the field should apparently be split according to shell-rules and should then be put into a stringlist. Using the support for named widgets would break this as it would use a simple string to store the text entry.
Same goes for the completion detail setting, IIRC the kcmodule support for named widgets uses the text of the entry but it seems the code wants to rather use the index. I might be wrong on this one though.
- Andreas Pakulat
On Sept. 8, 2013, 11:47 a.m., Vlas Puhov wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/112590/
> -----------------------------------------------------------
>
> (Updated Sept. 8, 2013, 11:47 a.m.)
>
>
> Review request for KDevelop.
>
>
> Description
> -------
>
> CCPreferences class is written with use of KCModule, so all setting can be saved/loaded automatically. For that widget's names should start with kcfg_ prefix, but neither todoMarkerWords nor completionDetail had one, so this feature didn't work.
>
>
> This addresses bug 324510.
> http://bugs.kde.org/show_bug.cgi?id=324510
>
>
> Diffs
> -----
>
> shell/completionsettings.h b3f28ce
> shell/completionsettings.cpp 6b6adf6
> shell/settings/ccconfig.kcfg 1bc89b1
> shell/settings/ccpreferences.h 4ecf623
> shell/settings/ccpreferences.cpp 0347fc6
> shell/settings/ccsettings.ui 64d24e2
>
> Diff: http://git.reviewboard.kde.org/r/112590/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Vlas Puhov
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20130908/17f36898/attachment.html>
More information about the KDevelop-devel
mailing list