On Thu, Feb 21, 2013 at 11:31 PM, Niels Thykier <ni...@thykier.net> wrote: > > Thanks for the patches, these have been merged[1]. > > When reviewing your patch today, I realised that it was possible to > simplify the "inner if" by using an additional argument to $info->field. > I have attached a small patch showing the changes I did for comparison.
Ah yes I didn't think of using DEFAULT feature of $info->field. I noted that I missed to fix a test case for files-adobe-font, I guess since I didn't run entire test case (ran only fields) I didn't catch it. Thanks -- Vasudev Kamath http://copyninja.info copyninja@{frndk.de|vasudev.homelinux.net} -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org