LGTM Michal
Padraig O'Briain wrote: > I have reworked the webrev http://cr.opensolaris.org/~padraig/ips-5526-v3/. > > It now justs addresses the issue in bug 5526. > > I have logged bug 5866 on the issue of unnecessary refiltering during > package manager startup as it is not as simple as I first thought.. > > Padraig > > On 12/16/08 15:26, Michal Pryc wrote: >> Padraig, >> Looks way better :) >> >> best >> Michal >> >> Padraig O'Briain wrote: >>> I have updated the webrev >>> http://cr.opensolaris.org/~padraig/ips-5316-v2/. >>> >>> On 12/16/08 14:45, Michal Pryc wrote: >>>> Padraig, >>>> >>>> This looks fine for me, except that I would put everything in the >>>> __application_refilter() function. >>>> >>>> best >>>> Michal >>>> >>>> Padraig O'Briain wrote: >>>>> The webrev http://cr.opensolaris.org/~padraig/ips-5526-v1/ fixes >>>>> Bug 5526 - Package Manager _very_ slow after UA "Screen R&M" turned >>>>> on. >>>>> >>>>> This is a rework of a patch provided by [email protected] to >>>>> fix performance issue with Packagemanager when accessibility is >>>>> enabled. >>>>> >>>>> A change is made to avoid refiltering the list twice during startup, >>>>> when the repository is changed and when category selection is changed. >>>>> >>>>> _______________________________________________ >>>>> pkg-discuss mailing list >>>>> [email protected] >>>>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss > _______________________________________________ > pkg-discuss mailing list > [email protected] > http://mail.opensolaris.org/mailman/listinfo/pkg-discuss _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
