Re: [PATCH 3/5 v2] riched20: Call ITextHost_TxGetCharFormat() for setting default charformat.

[Jactry Zeng]

Huw Davies  于2018年5月23日周三 下午4:36写道：
> > - /* replace some default charformat setting with charformat from host

*/
> > - hr = ITextHost_TxGetCharFormat(editor->texthost, (const CHARFORMATW

**)&host_cf);
> > - if (hr == S_OK)
> > - {
> > - if (host_cf->dwMask & CFM_FACE)
> > -
> >  lstrcpyW(cf.szFaceName, host_cf->szFaceName);
> > - if (host_cf->dwMask & CFM_SIZE)
> > -
> >  cf.yHeight = host_cf->yHeight;
> Does this really only use CFM_FACE and CFM_SIZE?  I'd expect a full merge

of
> char formats.

No, it had more effect masks. I only picked these two because it is enough
for
making real
applications (QQ and QQMusic etc.) look good. And I assumed more mask will
be added
when some other applications need more mask for displaying string correctly.
Do you think it is necessary to add a full effect mask for now?

--
Regards,
Jactry Zeng