Padraig - as this is post 2009.06, I'd like us to review all the Tab
panels and decide what they should look like. As we are introducing a
Tree View to get around l10n layout issues, I think we could use the
installed icon in the Dependencies tab if we also change it to use the
Tree View. We should review all this with Jenya.
Whatever we do its good to know the performance is not an issue here for
fetch and display this info.
JR
Padraig O'Briain wrote:
The cost of the api_o.info call is about 0.05 seconds. This is in the
noise as there is a delay of 600 milliseconds before __show_info is
called to avoid getting info for packages being scrolled.
I have respun the webrev,
http://cr.opensolaris.org/~padraig/ips-5321-v2/, so that the
api_o.info call is not made in the GUI thread.
I do not understand your other comments. I was not able to get the
api_o.info call to work without using frozenset.
Padraig
John Rice wrote:
Padraig - looks good. What's the performance impact of this when
fetching the state info to display the details, compared to the old
way? Does using the frozenset on STATE avoid the manifest access in
the api.info() call? If it does then we can use this to replace other
image access calls. I thought Michal had said he tried this but was
still finding it slower by x4 or maybe it was another issue entirely.
JR
Padraig O'Briain wrote:
The webrev, http://cr.opensolaris.org/~padraig/ips-5321-v1/,
partially fixes
5321 package manager dependencies tab could show installation status
This change identifies the dependencies which are installed.
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
