On 23/05/10 21:20, Andres P wrote:
Signed-off-by: Andres P<[email protected]>
---

Without a function, this would look like:
        local fmt="= %s\n"
        [[ $license ]]&&  printf "license $fmt"   "${licen...@]}"
        [[ $replaces ]]&&  printf "replaces $fmt"  "${replac...@]}"
        [[ $groups ]]&&  printf "group $fmt"     "${grou...@]}"
        [[ $depends ]]&&  printf "depend $fmt"    "${depen...@]}"
        [[ $optdepend ]]&&  printf "optdepend $fmt" "${optdepen...@]}"
        [[ $conflict ]]&&  printf "conflict $fmt"  "${conflic...@]}"
        [[ $provides ]]&&  printf "provides $fmt"  "${provid...@]}"
        [[ $backup ]]&&  printf "backup $fmt"    "${back...@]}"


I'd prefer this version. In fact, removing the $fmt stuff would be better as it is only two characters longer and makes it a lot more readable.

Again...  commit messages...

Allan

Reply via email to