Re: [PATCH v3 3/3] gitweb: Optional grouping of projects by category

At Fri, 12 Dec 2008 01:13:45 +0100, Jakub Narebski wrote:

I just tried, it works but we first need to sort @projects by
category.

I don't understand.
[...]
I propose to change it to:

Well in my previous iteration of the patch (v3), the printing of
projects with categories is done using:

  foreach my $cat (sort keys %categories) {

So everything was already sorted by category (and then by whichever
property you picked inside each category).  You seemed okay with it,
but requested that I documented that behaviour in the commit log.

To maintain the same result with your proposed change (which is what I
submitted in my patch), we need to sort by categories first (AFAIK
Perl sort retains the original order inside equivalence classes of
comparison key?), otherwise splice(projlist, from, to) doesn't return
the expected subset.

-- 
Sébastien Cevey / inso.cc