<table><tr><td style="">loh.tar added a comment.
</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><div><p>Contrary to my announcement to use an enum as key I chosed the string for maximum simplicity. <br />
Yes, this way you notice a typo only at runtime, but in any case you have to know how some stuff has to be written. An enum key offer not only complile time checks but also to use QVector instead of QHash which may be more perfomand but need some more effort when add new stuff and probably longer longer keys, prefixed by namespace.</p>

<p>My only goal is to get rid of all these tiny functions. But I understand you so, that you don't see a need for that.</p></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>To: </strong>loh.tar, cullmann, dhaumann<br /><strong>Cc: </strong>mwolff, kwrite-devel, kde-frameworks-devel, KTextEditor, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>