<table><tr><td style="">craigt updated this revision to Diff 19508.<br />craigt edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-e32brscyzeilzgk/" rel="noreferrer">(Show Details)</a><br />craigt 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/D7799" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Addressed milian's comments, modified unit tests to provide better coverage.</p></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">I ran the entire test suite before and after my changes and noticed no new breakage.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Modified extant testTypeAliasTemplate test in test_duchain-clang to verify this patch works as expected, namely, that it doesn't leak template parameters into the surrounding context, and that uses of the type alias template are correctly tracked in the DUChain. If you simply apply the changes to test_duchain.cpp on their own,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Maybe I should add a //gotcha// test-case for the leaked parameters?</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">the breakage of the current code model becomes apparent.</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D7799?vs=19477&id=19508" rel="noreferrer">https://phabricator.kde.org/D7799?vs=19477&id=19508</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7799" rel="noreferrer">https://phabricator.kde.org/D7799</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/clang/duchain/builder.cpp<br />
plugins/clang/tests/test_duchain.cpp</div></div></div><br /><div><strong>To: </strong>craigt, KDevelop, mwolff, kfunk<br /><strong>Cc: </strong>apol, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>