diff options
author | Dave Reisner <dreisner@archlinux.org> | 2014-11-18 09:02:06 -0500 |
---|---|---|
committer | Allan McRae <allan@archlinux.org> | 2014-11-20 14:40:25 +1000 |
commit | 04bc3a24ebedc2529d814164fb356a5a33489e6d (patch) | |
tree | 988c4da28ce463cb782cd925792f60650f004030 /src/pacman/pacman.c | |
parent | c23ff87893de4f27c898b164c8586ce691fc370f (diff) |
pacsort: parse inputs up front into control struct
This moves most of the parsing work out of the sorting path. The explode
and splitfile functions now call input_new and append input_t structs
to the list of sort candidates instead of raw strings. This lets us
make smarter and easier decisions in the sorting callbacks, which are
now also split into the version and file comparison methods for clarity.
This fixes two bugs:
1) Incorrect ordering with filenames containing epoch in the pkgver
2) Incorrect ordering with package names which are substrings of
each other (e.g. "systemd" and "systemd-sysvcompat").
Performance of the --files mode degrades slightly as a result of this
change, but not unreasonably. Sorting with small inputs (5-10) doubles
in runtime, but larger inputs (4000+) only increase by 20%.
ref: https://bugs.archlinux.org/task/37631
Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'src/pacman/pacman.c')
0 files changed, 0 insertions, 0 deletions