Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread suzuki toshiya
Dear Dakai,

Thanks! I was showing all characters with ftview and
wondering why I could not receive any errors :-),

Regards,
mpsuzuki

Dakai Liu wrote:
> Hi Werner and Mr. Suzuki,
> 
> Sorry, This IS a false alarm. The problem does happen in version 2.1.4, which 
> is what we are using. And the 2.4.10 reference are out of hasted peruse. 
> My apology.
> 
> 
> Regards,
> Dakai
> 
> From: Werner LEMBERG [[email protected]]
> Sent: Monday, 17 September 2012 3:55 PM
> To: Dakai Liu
> Cc: [email protected]
> Subject: Re: [ft] NPUSHW in skipcode().
> 
>> I have encountered a byte interpreter exception when rendering a
>> Japanese font. I think I may have found a bug in SkipCode() function
>> in ttinterp.c.  On version 2.4.10, in ttinterp.c, at line#4477,
>>
>> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
>>
>> The NPUSHW needs to double the offset:
>>
>> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;
> 
> Thanks for the report, however, I don't think this is the right fix:
> `CUR.length' is -1 for NPUSHB, and -2 for NPUSHW, so the offset is
> already doubled.
> 
> Can you send me the offending font offline, together with instructions
> (ideally for ftview or a similar demo program) how to reproduce your
> problem?
> 
> 
> Werner
> 
> ___
> Freetype mailing list
> [email protected]
> https://lists.nongnu.org/mailman/listinfo/freetype


___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype


Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread Dakai Liu
Hi Werner and Mr. Suzuki,

Sorry, This IS a false alarm. The problem does happen in version 2.1.4, which 
is what we are using. And the 2.4.10 reference are out of hasted peruse. 
My apology.


Regards,
Dakai

From: Werner LEMBERG [[email protected]]
Sent: Monday, 17 September 2012 3:55 PM
To: Dakai Liu
Cc: [email protected]
Subject: Re: [ft] NPUSHW in skipcode().

> I have encountered a byte interpreter exception when rendering a
> Japanese font. I think I may have found a bug in SkipCode() function
> in ttinterp.c.  On version 2.4.10, in ttinterp.c, at line#4477,
>
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
>
> The NPUSHW needs to double the offset:
>
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;

Thanks for the report, however, I don't think this is the right fix:
`CUR.length' is -1 for NPUSHB, and -2 for NPUSHW, so the offset is
already doubled.

Can you send me the offending font offline, together with instructions
(ideally for ftview or a similar demo program) how to reproduce your
problem?


Werner

___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype


Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread Werner LEMBERG

> I have encountered a byte interpreter exception when rendering a
> Japanese font. I think I may have found a bug in SkipCode() function
> in ttinterp.c.  On version 2.4.10, in ttinterp.c, at line#4477,
> 
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
> 
> The NPUSHW needs to double the offset:
> 
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;

Thanks for the report, however, I don't think this is the right fix:
`CUR.length' is -1 for NPUSHB, and -2 for NPUSHW, so the offset is
already doubled.

Can you send me the offending font offline, together with instructions
(ideally for ftview or a similar demo program) how to reproduce your
problem?


Werner

___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype


Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread Dakai Liu
Hi Mr. Suzuki,

>Thank you! I have FA P Mincho ver 1.02 (1995-1996), packed in fanmin3.ttc.

Yes, that is the one, and it happens in "prep" section with 8880 bytes of code.

Regards,
Dakai

From: suzuki toshiya [[email protected]]
Sent: Monday, 17 September 2012 3:07 PM
To: Dakai Liu
Cc: [email protected]
Subject: Re: [ft] NPUSHW in skipcode().

Dear Dakai,

Thank you! I have FA P Mincho ver 1.02 (1995-1996), packed in fanmin3.ttc.
I will check.

Regards,
mpsuzuki

Dakai Liu wrote:
> Hi Mr. Suzuki,
>
>
> It's called "FA PMinchou", and it's a sub font in a composite one.
>
> Regards,
> Dakai
> 
> From: suzuki toshiya [[email protected]]
> Sent: Monday, 17 September 2012 2:53 PM
> To: Dakai Liu
> Cc: [email protected]
> Subject: Re: [ft] NPUSHW in skipcode().
>
> Dear Dakai,
>
> Thank you for the information. If possible, please let me know
> which Japanese font hits this issue...
>
> Regards,
> mpsuzuki
>
> Dakai Liu wrote:
>> Dear List,
>>
>> I have encountered a byte interpreter exception when rendering a Japanese 
>> font. I think I may have found a bug in SkipCode() function in ttinterp.c.
>> On version 2.4.10, in ttinterp.c, at line#4477,
>>
>> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
>>
>> The NPUSHW needs to double the offset:
>>
>>
>> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;
>>
>>
>> It works for our product, anyone can confirm it?
>>
>>
>> Best Regards,
>> Dakai
>> ___
>> Freetype mailing list
>> [email protected]
>> https://lists.nongnu.org/mailman/listinfo/freetype
>


___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype


Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread suzuki toshiya
Dear Dakai,

Thank you! I have FA P Mincho ver 1.02 (1995-1996), packed in fanmin3.ttc.
I will check.

Regards,
mpsuzuki

Dakai Liu wrote:
> Hi Mr. Suzuki,
> 
> 
> It's called "FA PMinchou", and it's a sub font in a composite one.
> 
> Regards,
> Dakai
> 
> From: suzuki toshiya [[email protected]]
> Sent: Monday, 17 September 2012 2:53 PM
> To: Dakai Liu
> Cc: [email protected]
> Subject: Re: [ft] NPUSHW in skipcode().
> 
> Dear Dakai,
> 
> Thank you for the information. If possible, please let me know
> which Japanese font hits this issue...
> 
> Regards,
> mpsuzuki
> 
> Dakai Liu wrote:
>> Dear List,
>>
>> I have encountered a byte interpreter exception when rendering a Japanese 
>> font. I think I may have found a bug in SkipCode() function in ttinterp.c.
>> On version 2.4.10, in ttinterp.c, at line#4477,
>>
>> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
>>
>> The NPUSHW needs to double the offset:
>>
>>
>> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;
>>
>>
>> It works for our product, anyone can confirm it?
>>
>>
>> Best Regards,
>> Dakai
>> ___
>> Freetype mailing list
>> [email protected]
>> https://lists.nongnu.org/mailman/listinfo/freetype
> 


___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype


Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread Dakai Liu
Hi Mr. Suzuki,


It's called "FA PMinchou", and it's a sub font in a composite one.

Regards,
Dakai

From: suzuki toshiya [[email protected]]
Sent: Monday, 17 September 2012 2:53 PM
To: Dakai Liu
Cc: [email protected]
Subject: Re: [ft] NPUSHW in skipcode().

Dear Dakai,

Thank you for the information. If possible, please let me know
which Japanese font hits this issue...

Regards,
mpsuzuki

Dakai Liu wrote:
> Dear List,
>
> I have encountered a byte interpreter exception when rendering a Japanese 
> font. I think I may have found a bug in SkipCode() function in ttinterp.c.
> On version 2.4.10, in ttinterp.c, at line#4477,
>
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
>
> The NPUSHW needs to double the offset:
>
>
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;
>
>
> It works for our product, anyone can confirm it?
>
>
> Best Regards,
> Dakai
> ___
> Freetype mailing list
> [email protected]
> https://lists.nongnu.org/mailman/listinfo/freetype


___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype


Re: [ft] NPUSHW in skipcode().

2012-09-16 Thread suzuki toshiya
Dear Dakai,

Thank you for the information. If possible, please let me know
which Japanese font hits this issue...

Regards,
mpsuzuki

Dakai Liu wrote:
> Dear List,
> 
> I have encountered a byte interpreter exception when rendering a Japanese 
> font. I think I may have found a bug in SkipCode() function in ttinterp.c.
> On version 2.4.10, in ttinterp.c, at line#4477,
> 
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1];
> 
> The NPUSHW needs to double the offset:
> 
> 
> CUR.length = 2 - CUR.length * CUR.code[CUR.IP + 1]*2;
> 
> 
> It works for our product, anyone can confirm it?
> 
> 
> Best Regards,
> Dakai
> ___
> Freetype mailing list
> [email protected]
> https://lists.nongnu.org/mailman/listinfo/freetype


___
Freetype mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/freetype