Huw Davies huw@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.