<table><tr><td style="">brauch accepted this revision.<br />brauch added a comment.<br />This revision is now accepted and ready to land.
</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/D3512" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Looks good to me. Feel free to rename things, esp. lots of functions could really have better names. Using what is now called functionDeclarationForCalledDeclaration in more places sounds very sensible as well.</p>

<p>As a regression test by the way, I recommend downloading a large python project (e.g. pypy) and just running duchainify on it (built in debug). That is quite good in triggering obscure crashes.</p>

<p>Very nice of you to tidy this up!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKDEVPYTHON KDev-python</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3512" rel="noreferrer">https://phabricator.kde.org/D3512</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>flherne, brauch<br /><strong>Cc: </strong>kdevelop-devel<br /></div>