<table><tr><td style="">kossebau 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/D15140">View Revision</a></tr></table><br /><div><div><p>I was about to say Ship-It, but first went to ask for adding a note to the code comment in MainWindowPrivate::addPlugin() that this is a hack behind KXmlGui's back, but should be fine, and in the future this should be a functionality of KXmlGui itself.<br />
Typing those words though, I found myself in conflict with my personal principle of "if-not-a-data-loss-bug-fix-upstream-instead-of-fragile-workaround".</p>

<p><a href="https://phabricator.kde.org/p/antonanikin/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@antonanikin</a>  Given this is not a grave bug, but just some slightly annoying inconsistency. would you agree we should rather live with the current situation instead of adding potentially fragile code (which needs maintenance by future people), but instead spend efforts on adding support to KXmlGui to allow menu sorting mechanisms?<br />
When it comes to the Analyzers menus, I have a personal TODO item for a KXmlGui feature to solve in any case, as I would like to get rid of the repeated "Analyze * With" in the actual analyze action menu entries, if listed in that submenu.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D15140#inline-83150">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mainwindow_p.cpp:88</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #aa4000">void</span> <span class="n">sortMenuAlphabetically</span><span class="p">(</span><span class="n">QMenu</span><span style="color: #aa2211">*</span> <span class="n">menu</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span class="p">{</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Add also <tt style="background: #ebebeb; font-size: 13px;">static</tt>, to keep the funtion symbol private to this compilation unit, which would be better behaviour.<br />
IIUC, as we are in the <tt style="background: #ebebeb; font-size: 13px;">KDevelop</tt> namespace here, not the anonymous one, so the symbol is exported from the compilation unit, and might result in clashes with similar util methods from other compilation units on creating the linkage unit (or whatever the term is).</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15140">https://phabricator.kde.org/D15140</a></div></div><br /><div><strong>To: </strong>antonanikin, KDevelop<br /><strong>Cc: </strong>kossebau, kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>