Give connect to lambda a context object
ClosedPublic

Authored by gladhorn on Jul 11 2018, 8:26 PM.

Details

Summary

This is cleaner since it will track the life-time.

Diff Detail

Repository
R110 KScreen Library
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gladhorn created this revision.Jul 11 2018, 8:26 PM
Restricted Application added a project: Plasma. · View Herald TranscriptJul 11 2018, 8:26 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
gladhorn requested review of this revision.Jul 11 2018, 8:26 PM
broulik accepted this revision.Jul 11 2018, 8:28 PM
This revision is now accepted and ready to land.Jul 11 2018, 8:28 PM

Though you're connecting this as sender already, and loop is on the stack so this wouldn't really make a difference

This revision was automatically updated to reflect the committed changes.

Yes, this is partially to shut up clazy.