Re: [PATCH 1/2] riched20: Stub for ITextFont interface and implement ITextRange::GetFont. (try 2)

[Huw Davies]

On Thu, Sep 18, 2014 at 09:15:11PM +0800, Jactry Zeng wrote:
> +static IRichEditOleImpl *get_reOle(ITextFontImpl *txtFont)
> +{
> - if (txtFont->txtRge)
> - return txtFont->txtRge->reOle;
> - else
> - return txtFont->txtSel->reOle;
>
> +}

You're using two-space indents here, while the rest is four-space.
A couple of these have sneaked in in earlier patches too - make
sure your editor is set up correctly!
> +static HRESULT CreateITextFont(ITextFontImpl **ptxtFont)
> +{
> - ITextFontImpl *txtFont = NULL;

This initialization isn't very useful.
> - txtFont = heap_alloc(sizeof(ITextFontImpl));
> - if (!txtFont)
> -
>    return E_OUTOFMEMORY;
> -
> - txtFont->ITextFont_iface.lpVtbl = &tfvt;
> - txtFont->ref = 1;
> - *ptxtFont = txtFont;
> - return S_OK;
>
> +}

It would make more sense to initialize txtSel and txtRge to NULL
here - the caller then only needs to touch the one it needs.

Otherwise, this one and the next one look good to me.

Huw.