> On 6/5/2010 19:09, Andrew Eikum wrote: >> I'm not sure I follow. As the tests in the patch show, QI is called up >> to >> two times, once for IPersist and once for IPersistFolder if the first >> fails. I can certainly add tests for returning different CLSIDs, though >> that seems uninteresting to me since they all behave identically. >> > I mean, create different types of shellfolders, then test that function > accepts them all. And a simple check that QI isn't called for > IShellFolder, so > argument expected to be IShellFolder pointer.
Argh, webmail clients are frustrating. Here's the patch.
binDu0TYtRcOR.bin
Description: Binary data
