[PATCH 2/3] dwrite: Fix name section strings lookup

Piotr Caban piotr.caban at gmail.com
Thu Apr 27 09:54:09 CDT 2017


On 04/27/17 16:52, Nikolay Sivov wrote:
> On 27.04.2017 17:03, Piotr Caban wrote:
>> Signed-off-by: Piotr Caban <piotr at codeweavers.com>
>> ---
>>  dlls/dwrite/opentype.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>>
>
> The problem with this patch is that we want to continue and add as many
> strings for different locales as possible. So the condition should set
> 'exists' to TRUE after first pair was added successfully and never touch
> it after.
>
>
Yes, sorry, this patch is incorrect. I think that it would be nice to 
not fail in hard way if any of the names were decoded. Also I'll check 
why name decoding fails here.

Thanks,
Piotr



More information about the wine-devel mailing list