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>
Content-Type: text/plain
Content-Transfer-Encoding: 7bit
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);
> 	    }
> 	}
> 
>