> On Feb. 15, 2013, 8:34 a.m., Frank Reininghaus wrote: > > Just found the link to this review request in bug 315061. I can't comment > > on whether this approach is suitable for Okular, but here is one idea: > > Couldn't one make it a bit simpler by using QScopedPointer? This would > > remove the need to delete manually. You would just have to use > > QScopedPointer::reset() to assign the pointers in > > SidebarDelegate::updateBrushCache().
Yep, sounds good. - Eike ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108958/#review27500 ----------------------------------------------------------- On Feb. 14, 2013, 1:20 p.m., Eike Hein wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/108958/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2013, 1:20 p.m.) > > > Review request for Okular. > > > Description > ------- > > Port the sidebar list delegate to using cached KStatefulBrushes. The cache is > invalidated on global color scheme changes. This avoids instanciating a > KColorScheme on every paint. > > > Diffs > ----- > > ui/sidebar.h f3f5ebd > ui/sidebar.cpp a73779d > > Diff: http://git.reviewboard.kde.org/r/108958/diff/ > > > Testing > ------- > > > Thanks, > > Eike Hein > >
_______________________________________________ Okular-devel mailing list Okular-devel@kde.org https://mail.kde.org/mailman/listinfo/okular-devel
