[1/2] ole32/tests: Add a test for OleSave() called for a IOleCache storage.

Dmitry Timoshkov dmitry at baikal.ru
Thu Oct 19 04:53:27 CDT 2017


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.

-- 
Dmitry.



More information about the wine-devel mailing list