Is it a proper free memory code?

[huohaodian]

Hi,

I have a variable char ** test;

initialized as
test = new char*[6];
for (int i=0;i<6;i++)
{
test = new char[MAX_PATH];
}

and try to free memory as

for (int j=0;i<6;j++)
{
delete [] test[j]; // free space
test = NULL; // remove pointer
}
delete []test;


Thank in advance.

 

[red floyd]

Hi,

I have a variable char ** test;

initialized as
test = new char*[6];
for (int i=0;i<6;i++)
{
test = new char[MAX_PATH];
}

and try to free memory as

for (int j=0;i<6;j++)
{
delete [] test[j]; // free space
test = NULL; // remove pointer
}
delete []test;

As far as I can see, yes, the code should function as you think.

However... You should prefer to use a std::vector of std:;string.

That way you don't even have to deal with memory management, or worry
about buffer overrun.

 

[Default User]

Hi,

I have a variable char ** test;

initialized as
test = new char*[6];
for (int i=0;i<6;i++)
{
test = new char[MAX_PATH];
}

and try to free memory as

for (int j=0;i<6;j++)
{
delete [] test[j]; // free space
test = NULL; // remove pointer

The line above doesn't look right. I suspect you wanted j rather than
i.




Brian

 

[Micah Cowan]

Hi,

I have a variable char ** test;

initialized as
test = new char*[6];
for (int i=0;i<6;i++)
{
test = new char[MAX_PATH];
}

and try to free memory as

for (int j=0;i<6;j++)
{
delete [] test[j]; // free space
test = NULL; // remove pointer

The line above doesn't look right. I suspect you wanted j rather than
i.

Also in the middle clause of the "for" line.

 

[red floyd]

Hi,

I have a variable char ** test;

initialized as
test = new char*[6];
for (int i=0;i<6;i++)
{
test = new char[MAX_PATH];
}

and try to free memory as

for (int j=0;i<6;j++)
{
delete [] test[j]; // free space
test = NULL; // remove pointer
}
delete []test;

As far as I can see, yes, the code should function as you think.

However... You should prefer to use a std::vector of std:;string.

That way you don't even have to deal with memory management, or worry
about buffer overrun.

Modulo typos, of course (test[j], not test)

 

[Default User]

Hi,

I have a variable char ** test;

initialized as
test = new char*[6];
for (int i=0;i<6;i++)
{
test = new char[MAX_PATH];
}

and try to free memory as

for (int j=0;i<6;j++)
{
delete [] test[j]; // free space
test = NULL; // remove pointer

The line above doesn't look right. I suspect you wanted j rather
than i.

Also in the middle clause of the "for" line.

Yep. Cut and paste, with a var name switch. Unless he skipped some
code, i shouldn't even be declared at that point. I suspect he's using
VC6.




Brian