moparisthebest/pacman
1
0
Fork 0
mirror of https://github.com/moparisthebest/pacman synced 2024-11-12 04:15:06 -05:00
Code Issues Releases Wiki Activity

Revert "libalpm: compare pkgname with strcoll"

Browse Source 
This commit was made with the intent of displaying "correctly" sorted
package lists to users. Here are some reasons I think this is incorrect:

* It is done in the wrong place. If a frontend application wants to show
  a different order of packages dependent on locale, it should do that
  on its own.
* Even if one wants a locale-specific order, almost all package names
  are all ASCII and language agnostic, so this different comparison
  makes little sense and may serve only to confuse people.
* _alpm_pkg_cmp was unlike any other comparator function. None of the
  rest had any dependency on anything but the content of the structs
  being compared (e.g., they only used strcmp() or other basic
  comparison operators).

This reverts commit 3e4d2c3aa6.

Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
Dan McGee 2011-09-27 20:24:04 -05:00
parent 69962184bb
commit ea3c47825e
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/libalpm/package.c
View File

@ -641,7 +641,7 @@ int _alpm_pkg_cmp(const void *p1, const void *p2)
{ {
const alpm_pkg_t *pkg1 = p1; const alpm_pkg_t *pkg1 = p1;
const alpm_pkg_t *pkg2 = p2; const alpm_pkg_t *pkg2 = p2;
return strcoll(pkg1->name, pkg2->name); return strcmp(pkg1->name, pkg2->name);
} }
/* Test for existence of a package in a alpm_list_t* /* Test for existence of a package in a alpm_list_t*