Differential D12444

Don't try to open application menu when client doesn't have one
ClosedPublic
Actions

Authored by broulik on Apr 22 2018, 12:47 PM.

Details

Reviewers
graesslin
Group Reviewers
KWin
Commits
R108:27c65aade3ba: Don't try to open application menu when client doesn't have one
Summary

There's no point in sending a DBus message to an invalid (empty) address

Test Plan

No more warning on console when trying to invoke application menu on client that doesn't have one

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.
broulik created this revision.Apr 22 2018, 12:47 PM
Restricted Application added a project: KWin. · View Herald TranscriptApr 22 2018, 12:47 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
broulik requested review of this revision.Apr 22 2018, 12:47 PM
graesslin accepted this revision.Apr 22 2018, 1:40 PM
This revision is now accepted and ready to land.Apr 22 2018, 1:40 PM
Closed by commit R108:27c65aade3ba: Don't try to open application menu when client doesn't have one (authored by broulik). · Explain WhyApr 23 2018, 1:43 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mappmenu.cpp (3 lines)

Diff 32884

View Options

appmenu.cpp

Loading...