<table><tr><td style="">cfeck added a reviewer: KDE Applications.<br />cfeck 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/D24219">View Revision</a></tr></table><br /><div><div><p>I feared that it would result in many more changes, because KCalc code is 20+ years old, and maintainers didn't have guidelines for coding style back then.</p>

<p>We usually do not fix the coding style on old code because it makes it harder to see who contributed which changes ("git blame"). Style conventions make more sense for new code, or when the git history is rewritten anyway (e.g. for the KF5 split).</p>

<p>I don't know if Evan still has time to review changes; adding more reviewers to check if anyone objects.</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/D24219#inline-137102">View Inline</a><span style="color: #4b4d51; font-weight: bold;">kcalc_core.cpp:175</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; "><span style="color: #74777d">// build precedence list</span>
</div><div style="padding: 0 8px; margin: 0 4px; "><span style="color: #aa4000">const</span> <span style="color: #aa4000">struct</span> <span class="n">operator_data</span> <span class="n">Operator</span><span class="p">[]</span> <span style="color: #aa2211">=</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(251, 175, 175, .7);">    <span class="p">{<span class="bright"></span></span><span class="bright"> </span><span style="color: #601200"><span class="bright">0</span></span><span class="bright"></span><span class="p"><span class="bright">,</span></span><span class="bright"> </span><span style="color: #aa4000"><span class="bright">nullptr</span></span><span class="bright"></span><span class="p"><span class="bright">,</span></span><span class="bright">         </span><span style="color: #aa4000"><span class="bright">nullptr</span></span><span class="bright"></span><span class="p"><span class="bright">},</span></span><span class="bright">          </span><span style="color: #74777d"><span class="bright">// FUNC_EQUAL</span></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;">You might want to revert this part manually. The new table is much harder to read.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R353 KCalc</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24219">https://phabricator.kde.org/D24219</a></div></div><br /><div><strong>To: </strong>umanovskis, cfeck, teran, KDE Applications<br /><strong>Cc: </strong>cfeck, kde-utils-devel<br /></div>