<div dir="ltr">Thanks for review! I sent a new one.</div><div class="gmail_extra"><br><div class="gmail_quote">2018-04-03 17:00 GMT+08:00 Huw Davies <span dir="ltr"><<a href="mailto:huw@codeweavers.com" target="_blank">huw@codeweavers.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Apr 02, 2018 at 12:08:15PM +0800, Jactry Zeng wrote:<br>
>  static HRESULT WINAPI ITextRange_fnMoveWhile(<wbr>ITextRange *me, VARIANT *charset, LONG count,<br>
> @@ -4695,12 +4741,12 @@ static HRESULT WINAPI ITextSelection_fnMoveEnd(<wbr>ITextSelection *me, LONG unit, LO<br>
>  {<br>
>      ITextSelectionImpl *This = impl_from_ITextSelection(me);<br>
><br>
> -    FIXME("(%p)->(%d %d %p): stub\n", This, unit, count, delta);<br>
> +    TRACE("(%p)->(%d %d %p)\n", This, unit, count, delta);<br>
><br>
>      if (!This->reOle)<br>
>          return CO_E_RELEASED;<br>
><br>
> -    return E_NOTIMPL;<br>
> +    return textrange_moveend((ITextRange*<wbr>)me, unit, count, delta);<br>
<br>
The cast here is messy (I know ITextSelection inherits ITextRange, but still).<br>
You should probably get a ITextRange ptr by calling QueryInterface.<br>
<span class="HOEnZb"><font color="#888888"><br>
Huw.<br>
<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Jactry Zeng</div><div><br></div></div></div>
</div>