[1/2] ole32/tests: Add a test for OleSave() called for a IOleCache storage.
Huw Davies
huw at codeweavers.com
Thu Oct 19 04:59:15 CDT 2017
On Thu, Oct 19, 2017 at 05:53:27PM +0800, Dmitry Timoshkov wrote:
> Huw Davies <huw at codeweavers.com> wrote:
>
> > This looks ok, but could you please call IPersistStorage_Save()
> > directly rather than using the OleSave() helper? Also fix
> > the function name to reflect that.
>
> I was considering that, but using OleSave instead of IPersistStorage_Save
> simplifies the test, saves quite a bit of typing and helps the test avoid
> unnecessary details. I'd really prefer keeping to use OleSave() in these
> tests.
How's that? The Storage_Commit() hunk could go for one.
The point is though, that this should really test the data
cache's IPersitStorge_Save() implementation, not what some
higher level helper is doing.
Huw.
More information about the wine-devel
mailing list