[PATCH 1/8] riched20: Stub for ITextRange interface and implement ITextDocument::Range. (resend try 4)

Jactry Zeng jactry92 at gmail.com
Thu Sep 11 11:27:50 CDT 2014


2014-09-11 22:53 GMT+08:00 Huw Davies <huw at codeweavers.com>:
>
> Nice.  Well hopefully nothing depends on that, so we don't have to follow
> their broken implementation.  I think you should implement it as I
suggested
> and don't bother adding tests for the exact refcount number (nothing
should
> care about the exact number anyway).

So these should been added into the implementation:
1. AddRef(reOle) in CreateITextRange() (both ITextDocument::Range and
ITextRange::GetDuplicate
will call this function to fill a ITextRange interface);
2. Release(reOle) in ITextRange::Release()
3. Pass IRichEditOle and ITextRange into CreateITextRange() instead of
IRichEditOleImpl and
ITextRangeImpl

Is it the right way?

Thanks again. :)

btw, the rangelist was implemented for ITextRange::Delete.
Range of all ITextRange in one IRichEditOle should be reseted after we used
ITextRange::Delete to
delete some text in the RichEdit Control. And a rangelist will make this
work easier.

-- 
Regards,
Jactry Zeng
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20140912/46487d3b/attachment.html>


More information about the wine-devel mailing list