Differential D22813

Don't give up if no results arrive after 500 ms, again.
ClosedPublic
Actions

Authored by broulik on Jul 29 2019, 7:03 PM.

Details

Reviewers
fvogt
Group Reviewers
Plasma
Commits
R112:82b1ed82848c: Don't give up if no results arrive after 500 ms, again.
Summary

Port the fix added by D21605 to the new model and add a comment so I don't end up cleaning up that thing again a year from now.

Test Plan

Context is no longer cleared for the dict runner

Diff Detail

Repository
R112 Milou
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Jul 29 2019, 7:03 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 29 2019, 7:03 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Jul 29 2019, 7:03 PM
fvogt requested changes to this revision.Jul 29 2019, 7:06 PM
fvogt added inline comments.
lib/runnerresultsmodel.cpp
56

Redundant

This revision now requires changes to proceed.Jul 29 2019, 7:06 PM
broulik updated this revision to Diff 62765.Jul 29 2019, 7:07 PM
  • doppelthältbesser--
fvogt accepted this revision.Jul 29 2019, 7:10 PM
This revision is now accepted and ready to land.Jul 29 2019, 7:10 PM
Closed by commit R112:82b1ed82848c: Don't give up if no results arrive after 500 ms, again. (authored by broulik). · Explain WhyJul 29 2019, 7:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mlib/runnerresultsmodel.cpp (9 lines)

Diff 62762

View Options

lib/runnerresultsmodel.cpp

Loading...