Mailing-List:	contact cygwin-apps-help AT sourceware DOT cygnus DOT com; run by ezmlm
Sender:	cygwin-apps-owner AT sourceware DOT cygnus DOT com
List-Subscribe:	<mailto:cygwin-apps-subscribe AT sources DOT redhat DOT com>
List-Archive:	<http://sources.redhat.com/ml/cygwin-apps/>
List-Post:	<mailto:cygwin-apps AT sources DOT redhat DOT com>
List-Help:	<mailto:cygwin-apps-help AT sources DOT redhat DOT com>, <http://sources.redhat.com/lists.html#faqs>
Delivered-To:	mailing list cygwin-apps AT sources DOT redhat DOT com
Subject:	Re: new 'temp' directory in CVS cinstall contains dependency WIP
From:	Robert Collins <robert DOT collins AT itdomain DOT com DOT au>
To:	gp AT familiehaase DOT de
Cc:	cygwin-apps AT cygwin DOT com
In-Reply-To:	<3BB06510.18664.4B9F5BD6@localhost>
References:	<3BB06510 DOT 18664 DOT 4B9F5BD6 AT localhost>
X-Mailer:	Evolution/0.13 (Preview Release)
Date:	26 Sep 2001 08:40:48 +1000
Message-Id:	<1001457679.7862.9.camel@lifelesswks>
Mime-Version:	1.0
X-OriginalArrivalTime:	25 Sep 2001 22:48:57.0146 (UTC) FILETIME=[4276D9A0:01C14614]

On Tue, 2001-09-25 at 19:05, Gerrit P. Haase wrote:
> Something here is wrong (choose.cc, line 796):
> cat->name stays always at "Shells":
> 
>       for (Category *cat = pkg->category; cat; cat = cat->next)
> 	{
> 	  /* insert the package under this category in the list. If this category is not
> 	     visible, add it */
> 	  int n=0;
> 	  while (n < nlines)
> 	    {
> 	      /* this should be a generic call to list_sort_cmp */
> 	      if (lines[n].get_category ()
> 		  && cat->name == lines[n].get_category ()->name)
> 		  {
 		    insert_under (n, line);
strange. ok, well how about this then...

- 	        n++;
+		n = nlines;

 	      }
> 	      n++;
> 	    }
> 	  if (n == nlines)
> 	    {
> 	      /* the category wasn't visible - insert at the end */
> 	      insert_category (cat, CATEGORY_COLLAPSED);
> 	      insert_pkg (pkg);
> 	    }
> 	}
> 
> 

- Raw text -