logind: Add a function that detects the logind seat.
ClosedPublic

Authored by nerdopolist on Dec 29 2017, 5:16 PM.

Details

Summary

This will allow other parts of kwin to get the current logind seat that this instance is running under

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
nerdopolist created this revision.Dec 29 2017, 5:16 PM
Restricted Application added a subscriber: kwin. · View Herald TranscriptDec 29 2017, 5:16 PM
nerdopolist requested review of this revision.Dec 29 2017, 5:16 PM
anthonyfieroni added inline comments.
logind.cpp
441

You can't compare content of char pointers like that

logind.h
98

Make it QString like others.

graesslin requested changes to this revision.Dec 30 2017, 9:49 AM
graesslin added a subscriber: graesslin.

Yes, please store as QString. Also please default init the m_seat with e.g. QStringLiteral("seat0")

This revision now requires changes to proceed.Dec 30 2017, 9:49 AM

Adjust for requested changes

graesslin accepted this revision.Dec 31 2017, 9:32 AM
This revision is now accepted and ready to land.Dec 31 2017, 9:32 AM
aacid closed this revision.Jan 5 2018, 9:46 PM