[PATCH] Fix iterator bug in choose.cc

Robert Collins rbcollins@cygwin.com
Thu Mar 13 20:52:00 GMT 2003


On Fri, 2003-03-14 at 07:47, Igor Pechtchanski wrote:
> I noticed there is a bug in an iterator in choose.cc that causes "All" to
> be printed twice in the list of categories in setup.log.full.  The
> attached patch fixes that.  Note that, like the original, the patch will
> not work if a package is not in any category (but that shouldn't happen
> anyway, right?).
> 	Igor
> ==============================================================================
> 2003-03-13  Igor Pechtchanski <pechtcha@cs.nyu.edu>
> 
> 	* choose.cc (ChooserPage::OnNext): Fix iterator bug.

Nice catch. Wouldn't
            = pkg.categories.begin ();
->          String all_categories = *(i++);

Be an easier fix?
-- 
GPG key available at: <http://users.bigpond.net.au/robertc/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://cygwin.com/pipermail/cygwin-apps/attachments/20030313/a326f6fc/attachment.sig>


More information about the Cygwin-apps mailing list