--===============2581375818215906770== Content-Type: multipart/alternative; boundary="===============2051116477146313282==" --===============2051116477146313282== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable > On Jan. 21, 2012, 5:19 p.m., Philipp Knechtges wrote: > > Although I agree with you that the effects shouldn't be bothered with a= window that isn't shown once, > > I would prefer to make the emitted signals consistent. Your current app= roach has the shortcoming that > > it might not emit the windowAdded signal, but on the other hand it woul= d always emit windowClosed > > and windowDeleted. i hope not. those are connected in setupClientConnections()now being invoke= d by slotClientShown() which is not called unless the window is or got read= yForPainting() ie. the connection is cut off on the effectshandler - Thomas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103742/#review9984 ----------------------------------------------------------- On Jan. 20, 2012, 2:21 a.m., Thomas L=C3=BCbking wrote: > = > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103742/ > ----------------------------------------------------------- > = > (Updated Jan. 20, 2012, 2:21 a.m.) > = > = > Review request for kwin, Martin Gr=C3=A4=C3=9Flin and Philipp Knechtges. > = > = > Description > ------- > = > see summary. Makes animations smoother and would have prevented RR http:/= /git.reviewboard.kde.org/r/103733 ;-) > = > I'll keep it in my personal 4.8 branch and therefore won't care about whe= ther it's applied to 4.8.n or 4.9 :-P > = > = > Diffs > ----- > = > kwin/client.cpp 520ed85 = > kwin/composite.cpp 55a7144 = > kwin/effects.h f4ad2fb = > kwin/effects.cpp 6c252c9 = > kwin/events.cpp 26e0139 = > kwin/toplevel.h 693e6f3 = > kwin/toplevel.cpp ffe7f0c = > = > Diff: http://git.reviewboard.kde.org/r/103742/diff/diff > = > = > Testing > ------- > = > generic animations still work - better ;-) > = > = > Thanks, > = > Thomas L=C3=BCbking > = > --===============2051116477146313282== 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/103742/

On January 21st, 2012, 5:19 p.m., Philipp K= nechtges wrote:

Although =
I agree with you that the effects shouldn't be bothered with a window t=
hat isn't shown once,
I would prefer to make the emitted signals consistent. Your current approac=
h has the shortcoming that
it might not emit the windowAdded signal, but on the other hand it would al=
ways emit windowClosed
and windowDeleted.
i hope not.=
 those are connected in setupClientConnections()now being invoked by slotCl=
ientShown() which is not called unless the window is or got readyForPaintin=
g()

ie. the connection is cut off on the effectshandler

- Thomas


On January 20th, 2012, 2:21 a.m., Thomas L=C3=BCbking wrote:

Review request for kwin, Martin Gr=C3=A4=C3=9Flin and Philipp Knechtge= s.
By Thomas L=C3=BCbking.

Updated Jan. 20, 2012, 2:21 a.m.

Descripti= on

see summary. Makes animations smoother and would have preven=
ted RR http://git.reviewboard.kde.org/r/103733 ;-)

I'll keep it in my personal 4.8 branch and therefore won't care abo=
ut whether it's applied to 4.8.n or 4.9 :-P

Testing <= /h1>
generic animations still work - better ;-)

Diffs=

  • kwin/client.cpp (520ed85)
  • kwin/composite.cpp (55a7144)
  • kwin/effects.h (f4ad2fb)
  • kwin/effects.cpp (6c252c9)
  • kwin/events.cpp (26e0139)
  • kwin/toplevel.h (693e6f3)
  • kwin/toplevel.cpp (ffe7f0c)

View Diff

--===============2051116477146313282==-- --===============2581375818215906770== 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 --===============2581375818215906770==--