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

Reply via email to