<table><tr><td style="">loh.tar created this revision.<br />loh.tar added reviewers: cullmann, dhaumann.<br />Herald added projects: Kate, Frameworks.<br />Herald added subscribers: kde-frameworks-devel, kwrite-devel.<br />loh.tar requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18894">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div>

<ul class="remarkup-list">
<li class="remarkup-list-item">Remove unneeded bool configIsRunning, add m_ prefix to configSessionNumber</li>
<li class="remarkup-list-item">Remove unneeded inline keyword</li>
</ul>

<p>Pro</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Much easier to add new options/settings</li>
<li class="remarkup-list-item">Less ever the same boring code</li>
</ul>

<p>Cons</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Slower access</li>
<li class="remarkup-list-item">Presumably more memory consumption</li>
</ul>

<p>Notes</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Only three example settings in use, "Word Wrap"/bool, "Word Wrap Column"/int and "Indentation Mode"/string</li>
<li class="remarkup-list-item">No use of enum as key, but const char *, to keep it simple</li>
<li class="remarkup-list-item">Use Q_ASSERT to avoid wrong written keys</li>
</ul>

<p>TODO/NeedHelp</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">See code comments</li>
<li class="remarkup-list-item">FIX constness of getter functions</li>
<li class="remarkup-list-item">Apply in a general way to<ul class="remarkup-list">
<li class="remarkup-list-item">DocumentPrivate::readVariableLine</li>
<li class="remarkup-list-item">KateCommands::CoreCommands</li>
<li class="remarkup-list-item">VariableLineEdit::addKateItems</li>
</ul></li>
<li class="remarkup-list-item">Finish/improve documentation</li>
</ul>

<p>Questions/Hints</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">See code comments</li>
<li class="remarkup-list-item">Is setting "BackupFlags" really needed?</li>
<li class="remarkup-list-item">Are all these SomeClass:: get/set functions really needed? e.g: DocumentPrivate::wordWrapAt() seems to be unused; DocumentPrivate::setWordWrapAt(uint col) effective only 4 times used; Instead is often accessed directly by doc->config()->foo(); So perhaps could the way over fooClass()->config() be done in general</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18894">https://phabricator.kde.org/D18894</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/src/katedocument_test.cpp<br />
src/completion/katecompletionconfig.h<br />
src/dialogs/katedialogs.cpp<br />
src/document/editorconfig.cpp<br />
src/document/katebuffer.cpp<br />
src/document/katedocument.cpp<br />
src/render/katerenderer.cpp<br />
src/utils/kateautoindent.cpp<br />
src/utils/katecmds.cpp<br />
src/utils/kateconfig.cpp<br />
src/utils/kateconfig.h<br />
src/variableeditor/variablelineedit.cpp<br />
src/view/kateviewinternal.cpp</div></div></div><br /><div><strong>To: </strong>loh.tar, cullmann, dhaumann<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, KTextEditor, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>