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
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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 (7 lines)

Diff 62766

View Options

lib/runnerresultsmodel.cpp

Loading...