[Folder View] Don't spawn multiple stat jobs for the same folder
ClosedPublic
Actions

Authored by broulik on Jul 12 2018, 12:01 PM.

Details

Reviewers

Group Reviewers

Commits

Summary

Check for one already running before spawning a new one.
Also move the check for the isDirCache in one.

Test Plan

My .desktop files pointing to folders are still treated as folders
Doesn't spawn a gazillion of identical jobs on startup anymore

Repository

R119 Plasma Desktop

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

broulik requested review of this revision.Jul 12 2018, 12:01 PM

This revision is now accepted and ready to land.Jul 12 2018, 12:35 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			containments/desktop/plugins/folder/foldermodel.h (2 lines)
M			containments/desktop/plugins/folder/foldermodel.cpp (47 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	37641			Jul 12 2018, 11:59 AM	★	★
Diff 2	37644			Jul 12 2018, 12:17 PM	★	★
Diff 3	37652	6f6a002	R119:e03217edfc1a002a3ea654c58a28c1e1f689f3bf	Jul 12 2018, 12:52 PM	★	★