From kwin Thu Dec 27 14:59:19 2012 From: =?utf-8?q?Thomas_L=C3=BCbking?= Date: Thu, 27 Dec 2012 14:59:19 +0000 To: kwin Subject: Re: Review Request: Rewrite KillWindow on top of XCB Message-Id: <20121227145919.23848.63011 () vidsolbach ! de> X-MARC-Message: https://marc.info/?l=kwin&m=135662037423790 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============0221042265706920383==" --===============0221042265706920383== Content-Type: multipart/alternative; boundary="===============7368033558234193323==" --===============7368033558234193323== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107945/#review24057 ----------------------------------------------------------- Wasn't there some review to remove the slot (or was it just the dbus interf= ace?) - Thomas L=C3=BCbking On Dec. 27, 2012, 2:03 p.m., Martin Gr=C3=A4=C3=9Flin wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107945/ > ----------------------------------------------------------- > = > (Updated Dec. 27, 2012, 2:03 p.m.) > = > = > Review request for kwin and Fredrik H=C3=B6glund. > = > = > Description > ------- > = > Rewrite KillWindow on top of XCB > = > The code is rewritten in a way to replace the local event loop with event > filtering done in the normal way through events.cpp. Therefore instead of > creating a KillWindow whenever it's needed, there is one available in > Workspace which will be reused on next invocation. > = > The responsible events are passed from events.cpp to KillWindow for > processing. > = > In order to port the keycode to symbol to XCB, KWin now finds > xcb_keysyms and links it. > = > Currently there is one regression: the standard icon is used instead of > the user styled one. > = > = > Diffs > ----- > = > cmake/modules/FindXCB.cmake 7850c6831858ecd9f9d798d7a163115b2f0eeac9 = > kwin/CMakeLists.txt 62e9964a60783456dd11958f7a105e9d4df110c2 = > kwin/events.cpp 281f9e708d4488c504b8fc5eba3de5757a59c915 = > kwin/killwindow.h 585331ade43c052ff1e7de0e4976fb0fae54d7b0 = > kwin/killwindow.cpp 57e15e525a4428b80f5bc3c28d2934d7a4caa401 = > kwin/useractions.cpp 3617d2938cfd2483a0fb7591ffbce102db482b61 = > kwin/workspace.h 84163347d346d6381a4d825b560d30f04787e3c2 = > = > Diff: http://git.reviewboard.kde.org/r/107945/diff/ > = > = > Testing > ------- > = > = > Thanks, > = > Martin Gr=C3=A4=C3=9Flin > = > --===============7368033558234193323== Content-Type: text/html; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable

This is an automatically generated e-mail. To reply, visit: http://git.revie= wboard.kde.org/r/107945/

Wasn't=
 there some review to remove the slot (or was it just the dbus interface?)<=
/pre>
 








- Thomas




On December 27th, 2012, 2:03 p.m., Martin Gr=C3=A4=C3=9Flin wrote:







 
  

Review request for kwin and Fredrik H=C3=B6glund.
By Martin Gr=C3=A4=C3=9Flin.


Updated Dec. 27, 2012, 2:03 p.m.






Descripti=
on 
 
 
  
   Rewrite KillWindow on top of XCB

The code is rewritten in a way to replace the local event loop with event
filtering done in the normal way through events.cpp. Therefore instead of
creating a KillWindow whenever it's needed, there is one available in
Workspace which will be reused on next invocation.

The responsible events are passed from events.cpp to KillWindow for
processing.

In order to port the keycode to symbol to XCB, KWin now finds
xcb_keysyms and links it.

Currently there is one regression: the standard icon is used instead of
the user styled one.
  
 






Diffs=
 


 cmake/modules/FindXCB.cmake (7850c6831858e=
cd9f9d798d7a163115b2f0eeac9)

 kwin/CMakeLists.txt (62e9964a60783456dd119=
58f7a105e9d4df110c2)

 kwin/events.cpp (281f9e708d4488c504b8fc5eb=
a3de5757a59c915)

 kwin/killwindow.h (585331ade43c052ff1e7de0=
e4976fb0fae54d7b0)

 kwin/killwindow.cpp (57e15e525a4428b80f5bc=
3c28d2934d7a4caa401)

 kwin/useractions.cpp (3617d2938cfd2483a0fb=
7591ffbce102db482b61)

 kwin/workspace.h (84163347d346d6381a4d825b=
560d30f04787e3c2)



View Diff

--===============7368033558234193323==-- --===============0221042265706920383== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ kwin mailing list kwin@kde.org https://mail.kde.org/mailman/listinfo/kwin --===============0221042265706920383==--