diff options
author | Xavier Chantry <shiningxc@gmail.com> | 2008-07-25 02:02:36 +0200 |
---|---|---|
committer | Dan McGee <dan@archlinux.org> | 2008-07-24 22:16:29 -0500 |
commit | dfae7bdd52476673424f8020befef166cf95f3fc (patch) | |
tree | dfa54d7df22f17f29d27673cf416a81b46bf9acb /submitting-patches | |
parent | e6fb229534332396363f69bdb46081f45271bc53 (diff) | |
download | pacman-dfae7bdd52476673424f8020befef166cf95f3fc.tar.xz |
src/pacman : rework the display_targets function.
We had a lot of duplicated code here. The code handling the showsize option
needed to be there three times :
1) for install part of -S
2) for remove part of -S (conflict removal)
3) for -R
This patch introduce a new display_targets(pkglist, install) function which
can handle the 3 cases above. We pass install == 1 for case 1), and install
== 0 for case 2) and 3).
Now we can finally get the benefit of an old patch which handled the
ShowSize option consistently in the 3 cases above, without an awful lot of
duplicated code :
http://www.archlinux.org/pipermail/pacman-dev/2008-January/011029.html
Signed-off-by: Xavier Chantry <shiningxc@gmail.com>
Signed-off-by: Dan McGee <dan@archlinux.org>
Diffstat (limited to 'submitting-patches')
0 files changed, 0 insertions, 0 deletions