Re: [PATCH v2 0/7] MR8837: opcservices: Add basic support for reading OPC packages.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/opcservices/tests/opcservices.c:
> - IOpcPartEnumerator *partenum;
> - LARGE_INTEGER start = {0};
> - IOpcFactory *factory;
> - IOpcPackage *package;
> - IOpcPartSet *partset;
> - IOpcPartUri *uri;
> - IStream *stream;
> - IOpcPart *part;
> - BYTE buf[256];
> - HRESULT hr;
> - BOOL next;
> - int i;
> -
> - factory = create_factory();
> - hr = IOpcFactory_CreatePackage(factory, &package);
> - ok(hr == S_OK || broken(hr == E_NOTIMPL) /* Vista */, "Failed to create a package, hr %#lx.\n", hr);

I don't think we test on Vista anymore, so I wouldn't bother.