<table><tr><td style="">amhndu updated this revision to Diff 42772.<br />amhndu 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/D15902">View Revision</a></tr></table><br /><div><div><p>Added a test.<br />
But, as I've mentioned before, the tests for code completion crash,<br />
So I haven't been able to run all tests, though you can run a single test case<br />
before it crashes.</p>

<p>There's another thing, the visitor will keep scanning even past the invocation<br />
point, even though function declarations (and using-directives) after the<br />
invocation point shouldn't be considered while making suggestions. Would it<br />
better to create a new patch and test the file/line to break the visitor ?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D15902?vs=42744&id=42772">https://phabricator.kde.org/D15902?vs=42744&id=42772</a></div></div><br /><div><strong>BRANCH</strong><div><div>implements-fix</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15902">https://phabricator.kde.org/D15902</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/clang/codecompletion/completionhelper.cpp<br />
plugins/clang/tests/test_codecompletion.cpp<br />
plugins/clang/util/clangutils.cpp<br />
plugins/clang/util/clangutils.h</div></div></div><br /><div><strong>To: </strong>amhndu, KDevelop<br /><strong>Cc: </strong>apol, kdevelop-devel, glebaccon, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>