Hello Michael, On Tue, Jun 26, 2012 at 6:14 AM, Michael Gilbert <mgilb...@debian.org> wrote: > I was going to look at it again when I found the time, but its not > really at the top of my list.
Do you think there are many things to fix? Do you have an ETA for when you'll be able to give the patch a revamp? One thing I've noticed giving it a very quick look is the huge if section == .. part: I'd probably just prepare a dictionary section -> CC and then if section in d: header.append(d[section]) so in case you want to add new CC/sections you'd just have to change the dictionary, like it's a config section. Regards, -- Sandro Tosi (aka morph, morpheus, matrixhasu) My website: http://matrixhasu.altervista.org/ Me at Debian: http://wiki.debian.org/SandroTosi -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org