jmr wrote:
Ok LGTM - a nice cleanup, especially with the change to the Applications as the default category, we really needed to split the Category and Search views so Search applies to the whole list.

I assume this is an optimization, which will cut down on the row updating, which is annoying when descriptions are being loaded.

2632 2659          def update_desc(description, pkg):
2633 - pkg[enumerations.DESCRIPTION_COLUMN] = description 2660 + if pkg[enumerations.DESCRIPTION_COLUMN] != description: 2661 + pkg[enumerations.DESCRIPTION_COLUMN] = description

Yep.

Michal



JR

Michal Pryc wrote:
Hello,
Webrev:
http://cr.opensolaris.org/~migi/search_all_11_03_09/

Bug:
http://defect.opensolaris.org/bz/show_bug.cgi?id=7290

The fix is as discussed with xDesign. So search operates not only on the visible list but for all packages in the currently selected repository.

best
Michal




_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to