> On Nov. 28, 2011, 12:40 a.m., Rick W. Chen wrote: > > I've not seen this problem on my setup. Just tried the patch and it doesn't > > cause any issues here. Ship it.
I gave the "Ship it!" prematurely. There are some layout problems appearing with certain applets. I'll investigate some more. - Rick W. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103271/#review8553 ----------------------------------------------------------- On Nov. 27, 2011, 2:30 p.m., Matěj Laitl wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/103271/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2011, 2:30 p.m.) > > > Review request for Amarok. > > > Description > ------- > > Fix infinite loop in context view eating CPU until main window is shown > > We used to use _scene_ sceneRect in > VerticalToolbarContainment::updateGeometry() to update applets and > geometry, but that leaded to infinite loop (across mainloop) - > m_applets->setGeometry(), refresh() would enlarge _scene_ sceneRect by > a few pixels which would trigger updateGeometry() and so on... > > We now use _view_ sceneRect to update geometry. > > BUG: 278897 > FIXED-IN: 2.5 > > > Diffs > ----- > > ChangeLog a99a4de75b1941ecdc7cb3b41cce9ad3c996df27 > src/context/containments/verticallayout/VerticalToolbarContainment.cpp > 99fe147ab7b4d5ba8ca6f156a544d4066b1dc3da > > Diff: http://git.reviewboard.kde.org/r/103271/diff/diff > > > Testing > ------- > > Amarok doesn't eat CPU on start anymore. > > > Thanks, > > Matěj Laitl > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
