From kde-panel-devel  Fri Dec 30 18:25:11 2016
From: =?utf-8?q?graesslin_=28Martin_Gr=C3=A4=C3=9Flin=29?= <noreply () phabricator ! kde ! or
Date: Fri, 30 Dec 2016 18:25:11 +0000
To: kde-panel-devel
Subject: [Differential] [Request, 4 lines] D3878: Use removeOne when uninstalling an InputEventFilter
Message-Id: <differential-rev-PHID-DREV-vlixtutbfyhzquammoke-req () phabricator ! kde ! org>
X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=148312232426964
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--1058e75e45aa47069ce2d76a1463a442"

--1058e75e45aa47069ce2d76a1463a442
Content-Type: text/plain; charset="ascii"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit

graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.

REVISION SUMMARY
  Adjusting to a difference from introducing the InputEventSpy. A filter
  may only be installed once. Thus removeOne is totally sufficient.
  
  Added a Q_ASSERT to ensure that it is not installed when installing.

REPOSITORY
  R108 KWin

BRANCH
  input-event-filter-remove-one

REVISION DETAIL
  https://phabricator.kde.org/D3878

AFFECTED FILES
  input.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas

--1058e75e45aa47069ce2d76a1463a442
Content-Type: text/html; charset="ascii"
Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable

<table><tr><td style=3D"">graesslin created this revision.<br />graesslin a=
dded reviewers: KWin, Plasma.<br />Restricted Application added a project: =
KWin.<br />Restricted Application added subscribers: kwin, plasma-devel.
</td><a style=3D"text-decoration: none; padding: 4px 8px; margin: 0 8px 8px=
; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; back=
ground-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1=
f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href=
=3D"https://phabricator.kde.org/D3878" rel=3D"noreferrer">View Revision</a>=
</tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Adjusting =
to a difference from introducing the InputEventSpy. A filter<br />
may only be installed once. Thus removeOne is totally sufficient.</p>

<p>Added a Q_ASSERT to ensure that it is not installed when installing.</p>=
</div></div><br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div>=
</div></div><br /><div><strong>BRANCH</strong><div><div>input-event-filter-=
remove-one</div></div></div><br /><div><strong>REVISION DETAIL</strong><div=
><a href=3D"https://phabricator.kde.org/D3878" rel=3D"noreferrer">https://p=
habricator.kde.org/D3878</a></div></div><br /><div><strong>AFFECTED FILES</=
strong><div><div>input.cpp</div></div></div><br /><div><strong>EMAIL PREFER=
ENCES</strong><div><a href=3D"https://phabricator.kde.org/settings/panel/em=
ailpreferences/" rel=3D"noreferrer">https://phabricator.kde.org/settings/pa=
nel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>graess=
lin, KWin, Plasma<br /><strong>Cc: </strong>plasma-devel, kwin, lesliezhai,=
 ali-mohamed, hardening, jensreuterberg, abetts, sebas<br /></div>
--1058e75e45aa47069ce2d76a1463a442--