new & null

[gopal]

Hi i have the following code

int length = _tcslen(pszSourceString);
m_pstrStartingPos = new TCHAR[length + 1];
memset(m_pstrStartingPos, 0, (length * sizeof(TCHAR)));

when i enter this code in a function first time then
m_pstrStartingPos is NULL

next when i enter the function i just initialize to NULL, but i am not
using DELETE to delete the allocated memory, is this is the corect way?

Using new
then set the variable to NULL

or should delete be used

Regards
JK

 

[David Resnick]

Hi i have the following code

int length = _tcslen(pszSourceString);
m_pstrStartingPos = new TCHAR[length + 1];
memset(m_pstrStartingPos, 0, (length * sizeof(TCHAR)));

Try comp.lang.c++.

-David

 

[Vladimir S. Oka]

Hi i have the following code

int length = _tcslen(pszSourceString);
m_pstrStartingPos = new TCHAR[length + 1];
memset(m_pstrStartingPos, 0, (length * sizeof(TCHAR)));

when i enter this code in a function first time then
m_pstrStartingPos is NULL

next when i enter the function i just initialize to NULL, but i am not
using DELETE to delete the allocated memory, is this is the corect way?

Using new
then set the variable to NULL

or should delete be used

Regards
JK

This is a question for comp.lang.c++

Followups set...

 

[John Bode]

Hi i have the following code

int length = _tcslen(pszSourceString);
m_pstrStartingPos = new TCHAR[length + 1];
memset(m_pstrStartingPos, 0, (length * sizeof(TCHAR)));

when i enter this code in a function first time then
m_pstrStartingPos is NULL

next when i enter the function i just initialize to NULL, but i am not
using DELETE to delete the allocated memory, is this is the corect way?

No. If you do not deallocate the old memory before overwriting the
pointer, you will have a memory leak.

Using new
then set the variable to NULL

or should delete be used

Regards
JK

Change it to:

int length = _tcslen(pszSourceString);
if (m_pstrStartingPos != NULL)
{
delete m_pstrStartingPos;
}
m_pstrStartingPos = new TCHAR[length + 1];
memset(m_pstrStartingPos, 0, (length * sizeof(TCHAR)));

Please note that new and delete are C++ operators, not C operators, and
that this question should have been asked in comp.lang.c++.

Followups set.

 

[Martin Ambuhl]

Hi i have the following code

int length = _tcslen(pszSourceString);

_tcslen is a meaningless identifier, as is pszSourceString.

m_pstrStartingPos = new TCHAR[length + 1];

C++ is a different language from C. It has its own newsgroup
< Your question belongs there, if anywhere, but
before posting there check their FAQ and follow the newsgroup. You
failed to follow these simple guidelines before posting here.

 

[Keith Thompson]

Hi i have the following code

int length = _tcslen(pszSourceString);
m_pstrStartingPos = new TCHAR[length + 1];
memset(m_pstrStartingPos, 0, (length * sizeof(TCHAR)));

[snip]
Change it to:
[snip]

Please note that new and delete are C++ operators, not C operators, and
that this question should have been asked in comp.lang.c++.

Followups set.

Your answer should also have been posted only to comp.lang.c++.