TreeViewWidgetMgr - ApplyDataToWidget - Setting widget state

It is not IDocument that you shouldn't release, but an interface returned by a Get method. This returning you InDesign's reference to the IDocument without adding a reference - when you release it you are releasing InDesign's memory and it crashes. The standard is that a Get method is just a pointer that you use, while a Query needs a Release. This is in the doc's somewhere, but I never can remember where or find it quickly to refer others to it ;-)

Ian

Hi Dirk,

there was indeed a memory problem.
I used an InterfacePtr to IDocument by calling IActiveContext::GetContextDocument.
Changing this to IDocument* doc = IActiveContext::GetContextDocument works fine!

While we are on this, why does ID crash if i release my IDocument? Should i not call Release() on IDocument?

Thanks,
Florian.

Could it be that you have the opposite of a boss leak?

Set breakpoints on a few of your destructors and verify that the objects are not disposed / freed while they should still be in use. Then check your source for balance with InterfacePtrs, AddRef and so forth.

In such cases I use a little utility that walks all interfaces and dumps the reference counter. See HelperInterface.h with ID_IINTERFACEREFCOUNT, and IObjectModel.h .

Dirk