🚀 go-pugleaf

"Martin C" <martinc@aol.com> wrote in message
news:3f941a75$1@newsgroups.borland.com...

> I always retrieve rubbish if any web site contains dhtml

You're going to have to be a little more specific.  "Rubbish" can mean any
number of things.  Lets also not forget that DHTML is not static like HTML
is (thus its name - Dynamic HTML) so some of its content may come from
elsewhere, or be generated via scripting.

>                                         WCHAR   *szText;
>                                         pHtml.p->toString(&szText);

You have a memory leak in that code, a you are never freeing the BSTR that
toString() returns.  I would suggest that you use  WideString instead:

    WideString wText;
    pHtml.p->toString(&wText);

With that said, you can't use your pHTML variable anyway because you did not
assign it yet before trying to access it, thus your code should crash.

If all you want to do is get the HTML for the document, then I would suggest
you use an IPersistStreamInit instead to stream the content into an IStream,
which you can then store into a TStream descendant (TMemoryStream,
TStringStream, etc) via the TStreamAdapter helper class, ie.

    #include <memory>

    CComQIPtr<IPersistStreamInit, &IID_IPersistStreamInit> pPersist;
    pPersist = pADocument;
    if( pPersist )
    {
        std::auto_ptr<TStringStream> StrStrm(new TStringStream(""));
        std::auto_ptr<TStreamAdapter> Adapter(new
TStreamAdapter(StrStrm.get(), soReference));
        pPersist->Save(*Adapter, FALSE);
        // use StrStrm->DataString as needed
    }


Gambit