Fix build (with clang at least)
ClosedPublic

Authored by apol on Dec 16 2016, 12:08 PM.

Details

Summary

No need to capture the variable as it's static. Also apparently wrong.
Used to get the following error message:
/home/apol/devel/frameworks/kwin/libinput/connection.cpp:103:56: error: 's_self' cannot be captured because it does not have automatic storage duration

connect(s_self, &Connection::deviceAdded, s_self, [s_self](Device* device) {
                                                   ^

/home/apol/devel/frameworks/kwin/libinput/connection.cpp:45:25: note: 's_self' declared here
Connection *Connection::s_self = nullptr;

^

/home/apol/devel/frameworks/kwin/libinput/connection.cpp:106:58: error: 's_self' cannot be captured because it does not have automatic storage duration

connect(s_self, &Connection::deviceRemoved, s_self, [s_self](Device* device) {
                                                     ^

/home/apol/devel/frameworks/kwin/libinput/connection.cpp:45:25: note: 's_self' declared here
Connection *Connection::s_self = nullptr;

^

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
apol updated this revision to Diff 9076.Dec 16 2016, 12:08 PM
apol retitled this revision from to Fix build (with clang at least).
apol updated this object.
apol edited the test plan for this revision. (Show Details)
apol added a reviewer: Plasma.
Restricted Application added a project: KWin. · View Herald TranscriptDec 16 2016, 12:08 PM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
davidedmundson accepted this revision.Dec 16 2016, 12:10 PM
davidedmundson added a reviewer: davidedmundson.
This revision is now accepted and ready to land.Dec 16 2016, 12:10 PM
This revision was automatically updated to reflect the committed changes.