[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [ft] FT_New_Face and PCF fonts
From: |
suzuki toshiya |
Subject: |
Re: [ft] FT_New_Face and PCF fonts |
Date: |
Wed, 25 Sep 2013 11:04:42 +0900 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130518 Icedove/17.0.5 |
Hi,
Just I've committed, now PCF and BDF drivers return invalid
argument error when non-zero face index is given.
BTW, the argument checks are done AFTER the opening of the
font file by each driver. Although I was going to change the
drivers to refuse non-zero face index immediately without
any allocation and any reading, Type1 and Type42 drivers are
designed to check invalid face index AFTER the opening of
the font file, so I followed it.
If anybody thinks the immediate refusal is better, please post!
Regards,
mpsuzuki
On 09/24/2013 01:46 PM, Werner LEMBERG wrote:
>
>> I could not find any record changing to allow such, from git commit
>> log, so I guess it is unexpected/overlooked behaviour. I want to
>> change it to return an error (invalid argument or something else?).
>> Werner, more consideration is expected?
>
> Certainly a bug. Of course it should return `invalid argument'.
>
> Thanks for the report and the forthcoming fix :-)
>
>
> Werner
>