Resume background parser when replacing TestCore's project controller
ClosedPublic

Authored by mwolff on Jan 8 2018, 12:01 AM.

Details

Summary

This fixes the clang duchain unit tests that set a custom project
controller. Without this patch, the background parser would be
suspended and none of the unit tests would produce a DUChain in time.

I wonder why I haven't spotted this before when introducing the
background parser suspension on startup... Was I not running the
tests? Also note that tons of these tests are actually broken, seems
like I'm not the only one who's not running the tests :(

Diff Detail

Repository
R32 KDevelop
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mwolff created this revision.Jan 8 2018, 12:01 AM
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptJan 8 2018, 12:01 AM
mwolff requested review of this revision.Jan 8 2018, 12:01 AM
kfunk accepted this revision.Jan 9 2018, 7:21 AM
This revision is now accepted and ready to land.Jan 9 2018, 7:21 AM
This revision was automatically updated to reflect the committed changes.