Find generator variables without fake early declarations

Authored by flherne on Aug 28 2017, 9:49 PM.



This causes the real point of declaration to be bolded/highlighted. It also means all declarations have the same range as their NameAst node, which should allow simplification of some other code (not in this patch).

Test Plan

Doesn't break the tests. TODO: add some more, including some already-broken cases I discovered, and do some duchainify runs.

Diff Detail

R53 KDevelop: Python Support
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
flherne created this revision.Aug 28 2017, 9:49 PM
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptAug 28 2017, 9:49 PM
brauch accepted this revision.Aug 28 2017, 9:52 PM

Good solution I think, better than the hack from before (because it actually does the right thing, in a way). Feel free to put it into 5.2, we still have a few weeks to catch issues it causes. Thanks!

This revision is now accepted and ready to land.Aug 28 2017, 9:52 PM
This revision was automatically updated to reflect the committed changes.