Re: [PATCH v4 0/3] MR417: riched20: Add font fallback support.

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/riched20/wrap.c:
> +    WORD *glyphs;
> +    int len;
> +
> +    if (!run->script_tag)
> +        return FALSE;
> +
> +    len = run->len;
> +    if (!(glyphs = heap_calloc( len, sizeof( *glyphs ) )))
> +        return FALSE;
> +
> +    text = get_text( run, 0 );
> +    if (ScriptGetCMap( dc, &script_cache, text, len, 0, glyphs ) != S_OK)
> +    {
> +        heap_free( glyphs );
> +        return TRUE;
> +    }
> This doesn't seem like the right way to do this.  I'd have expected a fallback font to be used if `ScriptShape(OpenType)()` returned missing glyphs.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/417#note_6407