Split colored conflict resolution to helper function, unify callers
authorPanu Matilainen <pmatilai@redhat.com>
Wed, 29 Aug 2012 12:04:59 +0000 (15:04 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Wed, 29 Aug 2012 12:04:59 +0000 (15:04 +0300)
commitc906140266c2f17ba442b354079117a75b1899d5
tree6e12f88697cdfd6f04d8ea6309686cecb461865b
parent161494b973641175dd5fc9f460e587e0c5d50cbc
Split colored conflict resolution to helper function, unify callers

- The cases in handleOverlappedFiles() and handleInstInstalledFile() only
  differ by presence of "other file state", unify the copy-paste code
  to helper function and comment a bit more.
- Push all the color fiddling into the helper, eliminating loads of
  unnecessary rpmfiFColorIndex() calls in the common case. Also gives
  us a single point of control for the color-conflict behavior.
- This is not supposed to change any behavior at all, just refactoring...
lib/transaction.c