[PATCH] usp10: fall back to 'dflt' language if shaping language tag not found

Aric Stewart aric at codeweavers.com
Tue Aug 15 09:27:25 CDT 2017



On 8/14/17 1:27 PM, Henri Verbeet wrote:
> On 14 August 2017 at 20:04, Aric Stewart <aric at codeweavers.com> wrote:
>> +                script = get_opentype_script(hdc,psa,psc,FALSE);
> Does that do anything? If I'm reading the code correctly that will set
> it to the same value it already was. (In which case most of this would
> simplify to "if (!script) script = MS_MAKE_TAG('l','a','t','n');")
> 
> 

Not quite. The above loop that sets script tries twice, once with get_opentype_script(..., FALSE) then with get_opentype_script(..., TRUE) so if script is set coming into that loop it is from the TRUE loop, which (mostly just for some Indic languages) is different. The quick search I did seemed that we wanted the first case not the second for the dflt language case. 

It may be for some fonts we end up needing to search both the old and new script tags but if we do we can expand this later.

-aric



More information about the wine-devel mailing list