Differential D28032

[StorageDevices] Fix order of notification connect and initialization
ClosedPublic
Actions

Authored by bruns on Mar 13 2020, 5:25 PM.

Details

Reviewers
ngraham
davidedmundson
Group Reviewers
Baloo
Commits
R293:d771f908cb44: [StorageDevices] Fix order of notification connect and initialization
Summary

There is a small timeframe between cache initialization and connection
of device notifiers where events can go amiss. Swapping the order fixes
the problem.

Diff Detail

Repository
R293 Baloo
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bruns created this revision.Mar 13 2020, 5:25 PM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptMar 13 2020, 5:25 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Mar 13 2020, 5:25 PM
davidedmundson accepted this revision.Mar 13 2020, 5:27 PM
This revision is now accepted and ready to land.Mar 13 2020, 5:27 PM
Closed by commit R293:d771f908cb44: [StorageDevices] Fix order of notification connect and initialization (authored by bruns). · Explain WhyMar 13 2020, 5:28 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/file/storagedevices.cpp (4 lines)

Diff 77587

View Options

src/file/storagedevices.cpp

Loading...