[PATCH 1/2] gdi32: Add support for GCP_LIGATE to GetCharacterPlacement.

Huw Davies huw at codeweavers.com
Thu Jun 18 12:31:05 CDT 2020


On 18 Jun 2020, at 15:00, Dmitry Timoshkov <dmitry at baikal.ru> wrote:
> 
> Huw Davies <huw at codeweavers.com> wrote:
> 
>> This won't work very well if the ligature glyph doesn't exist in the font.
>> 
>> I suspect what you're supposed to do is to use font's gsub table to
>> find glyphs to combine rather than use Unicode normalization.
> 
> Thanks for the review. What do you think if I skip this patch and resend
> 2/2 alone? Are there any suggestions regarding that one?

Hi Dmitry,

Yes, please do that.  On first look it seems fine.

Thanks,
Huw.




More information about the wine-devel mailing list