<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/D9063" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Code generation with file templates sadly has regressed a little again with 5.2, so results are not yet perfect::<br />
a) Abstract virtual methods are not generated as overriding, but still abstract (since 5.2)<br />
b) Some template code copied around is assuming properties which got removed from codegen code.  Snippets like this currently are dead code, fix yet to be done (so template code kept for consistency in templates)</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">{% if method.type %}
return {{ method.default_return_value }};
{% endif %}</pre></div>

<p><tt style="background: #ebebeb; font-size: 13px;">type</tt> should be <tt style="background: #ebebeb; font-size: 13px;">returnType</tt> these days, and <tt style="background: #ebebeb; font-size: 13px;">default_return_value</tt> once got removed without replacement, might need to be done via the grantlee filter similar to filter <tt style="background: #ebebeb; font-size: 13px;">arg_type</tt>.</p>

<p>But the templates themselves in the patch here should be fine also once the regressions are fixed.</p></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/D9063" rel="noreferrer">https://phabricator.kde.org/D9063</a></div></div><br /><div><strong>To: </strong>kossebau, KDevelop<br /><strong>Cc: </strong>kdevelop-devel<br /></div>