Re: [PATCH v2 0/3] MR5685: Implement MarkupServices_ParseString

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/tests/htmldoc.c:
> - ITargetContainer *target_container_doc;
> - IInternetSecurityManager *sec_manager;
> - IMarkupContainer *container = NULL;
> - ICustomDoc *custom_doc;
> - IHTMLLocation *location = NULL;
> - IHTMLElement *body = NULL;
> - LONG elements_count;
> - HRESULT hres;
> -
> - hres = IMarkupServices_CreateMarkupContainer(markup_services, &container);
> - todo_wine ok(hres == S_OK, "got 0x%08lx\n", hres);
> - todo_wine ok(container != NULL, "MarkupContainer is null.\n");
> - if (!container) return;
> -
> - hres = IMarkupContainer_QueryInterface(container, &IID_IHTMLDocument2, (void**)&markup_container_doc);
> - todo_wine ok(hres == S_OK, "got 0x%08lx\n", hres);

This `todo_wine` and all bellow in this function are never needed, we return after previous todo anyway. Same to `test_MarkupServices_ParseString`.