This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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?
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 suggestion.
I've wondered sometimes if perhaps source handling should be removed entirely from the main package picker.


ChangeLog below:
==============================================================================
2005-05-24  Igor Pechtchanski  <pechtcha@cs.nyu.edu>

* 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?


Max.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]