[PATCH 1/4] riched20: Basic ITextDocument implementation for ITextServices (try 3)
Alexandre Julliard
julliard at winehq.org
Wed Jul 31 12:21:20 CDT 2013
Caibin Chen <tigersoldi at gmail.com> writes:
> Thanks Jacek for reviewing. This time I send all the changes I have in
> 4 patches to make it clear that ReTxtDoc is reused between txtsrv.c
> and richole.c
> - Suppressed Patch 97331
> - Change log from try 2:
> - follow COM naming convention
> - move ReTxtDoc definition to header, removed
> ReTxtDoc_get_ITextDocument() function
> - remove COM aggregation test for ITextDocument, only test
> ITextService supports querying for the interface.
>
> ---
> dlls/riched20/Makefile.in | 1 +
> dlls/riched20/tests/txtsrv.c | 10 ++
> dlls/riched20/txtdoc.c | 270 +++++++++++++++++++++++++++++++++++++++++++
> dlls/riched20/txtdoc.h | 51 ++++++++
> dlls/riched20/txtsrv.c | 6 +
Please avoid adding too many source files. In particular, if you have to
add new headers and define objects there, it's a sign that things should
be grouped in the same file. That's even more true for ITextDocument
that already exists.
--
Alexandre Julliard
julliard at winehq.org
More information about the wine-devel
mailing list