[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#701061: Please include the check to warn about font package not marked as Multi-Arch foreign [PATCH]



On Thu, Feb 21, 2013 at 11:31 PM, Niels Thykier <niels@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}


Reply to: