This is the mail archive of the
mailing list for the Cygwin project.
Re: [PATCH] setup: add "Size" column to chooser
Igor Pechtchanski wrote:
As promised earlier, here's a patch against the current CVS that adds the
"Size" column to the setup chooser screen, showing the size of the package
tarball(s). It seems to work well for me. Two caveats: since the size
information is extracted from setup.ini, if a package is not in setup.ini,
the size will be wrong or nonexistent; and this doesn't work well for
source-only packages (mostly because currently only one version of the
source can be selected). Please test. Comments welcome.
It's really hard to make source packages fit well into the GUI design, isn't
It's a bit weird that the size is [binary package size always] plus [source
package only if selected], on the other hand I don't have a better
I've wondered sometimes if perhaps source handling should be removed
entirely from the main package picker.
2005-05-24 Igor Pechtchanski <email@example.com>
* PickView.h (PickView::size_col): New instance variable.
* PickView.cc (pkg_headers, cat_headers): Add size column.
(PickView::set_headers): Initialize size_col.
(PickView::init_headers): Include width of size column.
* PickPackageLine.cc (PickPackageLine::paint): Handle size_col.
* String++.cc (format_1000s): New functions.
* String++.h (format_1000s): Declare new functions.
This could just use a default argument for "sep", couldn't it, rather than a
wrapper written out in full?