[Differential] [Commented On] D1145: DUChain: Prioritize active documents

[kfunk (Kevin Funk)]

kfunk added a comment.


  In https://phabricator.kde.org/D1145#21781, @mwolff wrote:
  
  > cool, thanks for this! small nitpicks, otherwise good to go.
  >
  > one question though: when we startup kdevelop with ~20 documents open, are they activated one after the other, leading to ~20 high priority parse jobs? that would need to be work-arounded via a timer then I think... If this is the case (and I actually fear that's the case), then I'd leave the function you touched as-is and only change the background parser to increase priority of activated documents after some time
  
  
  It's not the case here. If I start with those 20 documents, then the last-opened gets high prio + the one I'm switching to. Just those.
  
  > could we get a unit test for this as well? would be nice to have I think, but not too important
  
  Will check.

REPOSITORY
  rKDEVPLATFORM KDevPlatform

BRANCH
  assistantpopup-ng

REVISION DETAIL
  https://phabricator.kde.org/D1145

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: kfunk, mwolff
Cc: mwolff, kdevelop-devel, arrowdodger