Memory leak ?

[Felix Sima]

Hi.

Does the following line generate a memory leak or doesn't ?

*** foo( new CTest( ) ); *** //See code below.

Watching the output of the program seems that only the constructor
of the temporary object is called so that the memory allocated for
"new CTest( )" doesn't get deallocated.

Is it the right behaviour? If not, what can I do ?

Thank you.
Felix

********* CODE *********
#include <iostream>
using namespace std;

class CTest
{
public:
CTest()
{
cout << "Constructor" << endl;
};
~CTest()
{
cout << "Destructor" << endl;
};
};

void foo( CTest *p )
{
};

int main( int argc, char **argv )
{
foo( new CTest( ) ); //Memory leak here ?

return 0;
};

******** OUTPUT ************
Constructor

[Peter Koch Larsen]

"Felix Sima" <si*********@yahoo.com> skrev i en meddelelse
news:c0**********@newsfeed.cineca.it...

Hi.

Does the following line generate a memory leak or doesn't ?

*** foo( new CTest( ) ); *** //See code below.

Watching the output of the program seems that only the constructor
of the temporary object is called so that the memory allocated for
"new CTest( )" doesn't get deallocated.

Is it the right behaviour? If not, what can I do ?

Thank you.
Felix

********* CODE *********
#include <iostream>
using namespace std;

class CTest
{
public:
CTest()
{
cout << "Constructor" << endl;
};
~CTest()
{
cout << "Destructor" << endl;
};
};

void foo( CTest *p )
{
};

int main( int argc, char **argv )
{
foo( new CTest( ) ); //Memory leak here ?

return 0;
};

******** OUTPUT ************
Constructor

Of course. For every new there should be a corresponding delete. Do you come
from Java? ;-)

/Peter

[Martin SChukrazy]

LOL what about C# or Visual Basic..
hahaha

"Peter Koch Larsen" <pk*****@mailme.dk> wrote in message
news:Bt*********************@news000.worldonline.d k...

"Felix Sima" <si*********@yahoo.com> skrev i en meddelelse
news:c0**********@newsfeed.cineca.it...

Hi.

Does the following line generate a memory leak or doesn't ?

*** foo( new CTest( ) ); *** //See code below.

Watching the output of the program seems that only the constructor
of the temporary object is called so that the memory allocated for
"new CTest( )" doesn't get deallocated.

Is it the right behaviour? If not, what can I do ?

Thank you.
Felix

********* CODE *********
#include <iostream>
using namespace std;

class CTest
{
public:
CTest()
{
cout << "Constructor" << endl;
};
~CTest()
{
cout << "Destructor" << endl;
};
};

void foo( CTest *p )
{
};

int main( int argc, char **argv )
{
foo( new CTest( ) ); //Memory leak here ?

return 0;
};

******** OUTPUT ************
Constructor

Of course. For every new there should be a corresponding delete. Do you

come from Java? ;-)

/Peter

[Ron Samuel Klatchko]

Felix Sima <si*********@yahoo.com> wrote in message news:<c0**********@newsfeed.cineca.it>...

Does the following line generate a memory leak or doesn't ?

*** foo( new CTest( ) ); *** //See code below.

It's not that specific line that generates a memory leak, but the
design of all your code. The problem isn't that you new it, but that
you have no matching delete.

samuel

[Karl Heinz Buchegger]

Felix Sima wrote:

Hi.

Does the following line generate a memory leak or doesn't ?

*** foo( new CTest( ) ); *** //See code below.

Watching the output of the program seems that only the constructor
of the temporary object is called so that the memory allocated for
"new CTest( )" doesn't get deallocated.

Is it the right behaviour? If not, what can I do ?

Not doing dynamic allocation. Then you don't have to worry about freeing
it. ANd of course, not using pointers if there is no need to.

You could eg. do:

void foo( const CTest& Arg )
{
}

int main()
{
foo( CTest() );
}

--
Karl Heinz Buchegger
kb******@gascad.at