Open context for lambdas in ContextBuilder
ClosedPublic

Authored by flherne on Sep 12 2017, 11:55 PM.

Details

Reviewers
brauch
Summary

AbstractUseBuilder::newUse() relies on currentContext() to be correct, so we need to open the contexts here and not only in DeclarationBuilder.

This caused uses of lambda parameters to be created in the parent context, which broke rainbow-highlighting.

Test Plan

Checked that they're in the right context now. Highlighting works.

Diff Detail

Repository
R53 KDevelop: Python Support
Lint
Lint Skipped
Unit
Unit Tests Skipped
flherne created this revision.Sep 12 2017, 11:55 PM
Restricted Application added a subscriber: kdevelop-devel. ยท View Herald TranscriptSep 12 2017, 11:55 PM
brauch accepted this revision.Sep 13 2017, 12:01 AM

Looks sensible, thanks for investigating!

This revision is now accepted and ready to land.Sep 13 2017, 12:01 AM
flherne closed this revision.Sep 13 2017, 12:57 AM