Padraig - I would prefer if you put a stub "-1" into the text["size"] when local_info.size == 0 and test for this string as opposed to catching the key error in set_package_details_text(). Think its a bit cleaner and clearer.

JR

Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-12105-v1/, fixes
12105 Package with dependencies only should not show size in details

We no longer display the size for packages whose size is 0.

Padraig
_______________________________________________
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

Reply via email to