Will my memory be deallocated?

[s]

Regarding the following pseudocode:

<pseudocode>

char *buffer = new char[10];
//put some values in buffer
if( some_condition )
{
char *temp = new char[10];
//put some values in temp
delete[] buffer;
buffer = temp;
temp = NULL;
}
//continue doing stuff with buffer

</pseudocode>

Question:
Is the "temp = NULL;" statement necessary? If it still points to what
it was originally set, will that memory be deallocated when I leave the
scope of that if block?

Thanks,
s

[John Harrison]

"s" <yo*********@home.with.yourself> wrote in message
news:40***************@home.with.yourself...

Regarding the following pseudocode:

<pseudocode>

char *buffer = new char[10];
//put some values in buffer
if( some_condition )
{
char *temp = new char[10];
//put some values in temp
delete[] buffer;
buffer = temp;
temp = NULL;
}
//continue doing stuff with buffer

</pseudocode>

Question:
Is the "temp = NULL;" statement necessary?
If it still points to what
it was originally set, will that memory be deallocated when I leave the
scope of that if block?
No, definitely no. Memory is allocated with new[] is deallocated using
delete[], no other way.

Thanks,
s

john

[Dave]

"s" <yo*********@home.with.yourself> wrote in message
news:40***************@home.with.yourself...

Regarding the following pseudocode:

<pseudocode>

char *buffer = new char[10];
//put some values in buffer
if( some_condition )
{
char *temp = new char[10];
//put some values in temp
delete[] buffer;
buffer = temp;
temp = NULL;
}
//continue doing stuff with buffer

</pseudocode>

Question:
Is the "temp = NULL;" statement necessary? If it still points to what
it was originally set, will that memory be deallocated when I leave the
scope of that if block?

Thanks,
s

temp = NULL has nothing to do with deallocation. You need to specifically
delete [] the address allocated in line "char *temp = new char[10];".

[Mike Wahler]

"s" <yo*********@home.with.yourself> wrote in message
news:40***************@home.with.yourself...

Regarding the following pseudocode:

<pseudocode>

char *buffer = new char[10];
//put some values in buffer
if( some_condition )
{
char *temp = new char[10];
//put some values in temp
delete[] buffer;
buffer = temp;
temp = NULL;
}
//continue doing stuff with buffer

</pseudocode>

Question:
Is the "temp = NULL;" statement necessary?
No. (However it might be useful for testing purposes).

Don't lose the value now in 'buffer', or you'll leak memory.
If it still points to what
it was originally set,

Whether it does or not ...
will that memory be deallocated when I leave the
scope of that if block?

No. You allocated it, you must free it. This is why you
need to retain the value orignally returned by 'new' (as
you've done above), whether in the original pointer or some
copy of it.

-Mike

[David Harmon]

On Wed, 3 Mar 2004 21:58:16 -0000 in comp.lang.c++, "John Harrison"
<jo*************@hotmail.com> was alleged to have written:

If it still points to what
it was originally set, will that memory be deallocated when I leave the
scope of that if block?

No, definitely no. Memory is allocated with new[] is deallocated using
delete[], no other way.

But don't forget std::auto_ptr etc.

[John Harrison]

"David Harmon" <so****@netcom.com> wrote in message
news:40****************@news.west.earthlink.net...

On Wed, 3 Mar 2004 21:58:16 -0000 in comp.lang.c++, "John Harrison"
<jo*************@hotmail.com> was alleged to have written:

If it still points to what
it was originally set, will that memory be deallocated when I leave the
scope of that if block?

No, definitely no. Memory is allocated with new[] is deallocated using
delete[], no other way.

But don't forget std::auto_ptr etc.

Well no, but what does auto_ptr use to deallocate memory? Either something
non-standard, or it uses delete (not delete[] of course).

john

[Duane Hebert]

"Mike Wahler" <mk******@mkwahler.net> wrote in message news:MC******************@newsread2.news.pas.earth link.net...

"s" <yo*********@home.with.yourself> wrote in message
news:40***************@home.with.yourself...

Regarding the following pseudocode:

<pseudocode>

char *buffer = new char[10];
//put some values in buffer
if( some_condition )
{
char *temp = new char[10];
//put some values in temp
delete[] buffer;
buffer = temp;
temp = NULL;
}
//continue doing stuff with buffer

</pseudocode>

Question:
Is the "temp = NULL;" statement necessary?

No. (However it might be useful for testing purposes).

Or preventing possible delete from the wrong pointer.

[Mike Wahler]

"Duane Hebert" <sp**@flarn2.com> wrote in message
news:nC*******************@weber.videotron.net...

"Mike Wahler" <mk******@mkwahler.net> wrote in message

news:MC******************@newsread2.news.pas.earth link.net...

"s" <yo*********@home.with.yourself> wrote in message
news:40***************@home.with.yourself...

Regarding the following pseudocode:

<pseudocode>

char *buffer = new char[10];
//put some values in buffer
if( some_condition )
{
char *temp = new char[10];
//put some values in temp
delete[] buffer;
buffer = temp;
temp = NULL;
}
//continue doing stuff with buffer

</pseudocode>

Question:
Is the "temp = NULL;" statement necessary?

No. (However it might be useful for testing purposes).

Or preventing possible delete from the wrong pointer.

That's still a 'test' (delegated to operator 'delete').

But assigning NULL blindly for that only purpose imo
is poor practice. One could easily be doing such a
'safe' delete, but leaving something else 'undeleted'
which should have been, possibly having it 'bite' you
later (e.g. assigning a new value to a pointer that's
still pointing to allocated storage.)

-Mike

[Duane Hebert]

"Mike Wahler" <mk******@mkwahler.net> wrote in message news:Qo42c.22928

Or preventing possible delete from the wrong pointer.
That's still a 'test' (delegated to operator 'delete').

But assigning NULL blindly for that only purpose imo
is poor practice. One could easily be doing such a
'safe' delete, but leaving something else 'undeleted'
which should have been, possibly having it 'bite' you
later (e.g. assigning a new value to a pointer that's
still pointing to allocated storage.)

I don't disagree. I don't use this idiom. Just commenting
on a "possible" use.