Another dynamic memory doubt

[pradeep]

Hello friends:

I have the following code,

char* MyClass::MyMethod()
{
char* ptr = new char[6]; // space for NULL terminator
ptr = "hello";
return ptr;
delete[] ptr;
}

What I'm trying to do is return ptr, but delete[] it afterwards. In the
code above, I believe the delete[] line will not be executed following
the return statement.

How should I achieve this?

Thanks.

[Antoninus Twink]

On 26 Apr 2008 at 9:46, pradeep wrote:

I have the following code,

There are several problems and misunderstandings in your code...

char* MyClass::MyMethod()

In C++, it's usually better to #include <stringand then use
std::string instead of C-style strings. You can always get a good old
char * back if necessary by using the c_str() method.

{
char* ptr = new char[6]; // space for NULL terminator
ptr = "hello";

This does not copy the string "hello" into the array you've just
allocated, but rather makes ptr point to the start of a string literal,
which will be stored in the data segment of your object file.

A consequence of this is that you have leaked the memory you allocated
on the line before, and trying to delete ptr will cause problems.

return ptr;
delete[] ptr;
}

What I'm trying to do is return ptr, but delete[] it afterwards. In the
code above, I believe the delete[] line will not be executed following
the return statement.

Of course not! A return statement moves you up a stack frame, and in
particular sets the program counter to where it was when you called the
function.

How should I achieve this?

In the documentation for your function, explain that it allocates some
memory and that the caller is responsible for deleting ptr.

Or don't allocate memory at all, but just have

char* MyClass::MyMethod()
{
return "hello";
}

[santosh]

pradeep wrote:

Hello friends:

I have the following code,

char* MyClass::MyMethod()

This is C++. There is a dedicated group for C++ called comp.lang.c++.
Try that.

<snip>

[Martin Ambuhl]

pradeep wrote:

Hello friends:

I have the following code,

char* MyClass::MyMethod()

^^^^^^^^ This line is a syntax error in C.

{
char* ptr = new char[6]; // space for NULL terminator

^^^^^^^^ This line is a syntax error in C.

ptr = "hello";

^^^^^^^^ If the preceding non-C line allocated space for a char[6] and
assigned the address of the first element to ptr, then
this line creates a memory leak and loses the address of
the char[6] array.

return ptr;
delete[] ptr;

^^^^^^^^ This line is a syntax error in C, and even if it weren't,
the return in the line before makes this line unreachable.

}

What I'm trying to do is return ptr, but delete[] it afterwards.

In the foolish language you are grossly misusing, you will need to
delete the array separately. However, since you have assigned the
addess of the anonymous char literal "hello" to ptr, that is almost
guaranteed to blow up in your face. After you go back and read your
textbook and read the FAQ for the newsgroup for the language you are
using, ask in that newsgroup. It is very likely that the language you
are misusing in C++, and its newsgroup is <news:comp.lang.c++>.

In the

code above, I believe the delete[] line will not be executed following
the return statement.

How should I achieve this?

By not coding randomly and praying that whatever you type is legal. Put
some effort into actually learning the language.

[santosh]

Antoninus Twink wrote:

On 26 Apr 2008 at 9:46, pradeep wrote:

>I have the following code,

There are several problems and misunderstandings in your code...

<snip C++ code and response>

Nowhere in your post did you mention to the OP that there is in fact a
dedicated Usenet group for C++ where answers are likely to be more
correct than OT responses in other groups. Why is that? Do you
deliberately want the OP to remain ignorant of the best forum for help
with his problem and risk betting on possibly incorrect or outdated
advice in other groups?

Answering questions on non-conforming C code is one thing but answering
questions on totally separate languages and other wildly OT matters
(like the recent thread on Ethernet cables) seems a deliberate tactic
to lower the S:N ratio and wreck this group.

[Walter Roberson]

In article <fu**********@aioe.org>, pradeep <no****@nospam.comwrote:

>I have the following code,

>char* MyClass::MyMethod()

That line is a syntax error. The only place that a colon (':')
can occur in a type description in C is as a bit-field length
within a structure definition.

>{
char* ptr = new char[6]; // space for NULL terminator

That line is a syntax error. There is no storage class named 'new'
in C.

ptr = "hello";
return ptr;
delete[] ptr;

Another syntax error.

>}

>What I'm trying to do is return ptr, but delete[] it afterwards. In the
code above, I believe the delete[] line will not be executed following
the return statement.

>How should I achieve this?

How can you be assured that the pointer would not be deleted until
after the calling routine had finished accessing the data??
If you create storage and return a pointer to that storage, you cannot
also delete that storage, not unless the calling routine will never
attempt to access the storage nor to test the pointer against anything
other than a null pointer constant.

One would suspect that your code is not in fact C code, in
which case our most charitable assumption would be that you
have -accidently- posted your question to the wrong newsgroup.

But even if so, you need to figure out what it is supposed to
mean to delete a pointer to a storage area when you have returned
that pointer outwards. There are not many languages around
(-none- that I can think of right now) in which returning a pointer
to a memory block is understood to mean that the procedure
calling mechanism should take a -copy- of the storage pointed to
and put that copy somewhere for future use.
--
"History is a pile of debris" -- Laurie Anderson

[Malcolm McLean]

"pradeep" <no****@nospam.comwrote in message news:fu**********@aioe.org...

Hello friends:

I have the following code,

char* MyClass::MyMethod()
{
char* ptr = new char[6]; // space for NULL terminator
ptr = "hello";
return ptr;
delete[] ptr;
}

What I'm trying to do is return ptr, but delete[] it afterwards. In the
code above, I believe the delete[] line will not be executed following the
return statement.

How should I achieve this?

You want C++, next door.
However most programming languages have the feature that statements after a
return are not executed. The answer is to put those statements in the
calling code.

--
Free games and programming goodies.
http://www.personal.leeds.ac.uk/~bgy1mm

[Ben Bacarisse]

Antoninus Twink <no****@nospam.invalidwrites:

On 26 Apr 2008 at 9:46, pradeep wrote:

>I have the following code,
char* MyClass::MyMethod()

In C++, ...

I won't point out the C++ errors you made (that would be to fall into
your trap) but the original poster should know:

(a) that comp.lang.c++ is the place to ask;
(b) answers here will not checked by people who really know C++;
(c) your answer was not correct (IMO, but see (b)).

--
Ben.

[Richard]

ro******@ibd.nrc-cnrc.gc.ca (Walter Roberson) writes:

In article <fu**********@aioe.org>, pradeep <no****@nospam.comwrote:

>>I have the following code,

>>char* MyClass::MyMethod()

That line is a syntax error. The only place that a colon (':')
can occur in a type description in C is as a bit-field length
within a structure definition.

>>{
char* ptr = new char[6]; // space for NULL terminator

That line is a syntax error. There is no storage class named 'new'
in C.

> ptr = "hello";
return ptr;
delete[] ptr;

Another syntax error.

>>}

>>What I'm trying to do is return ptr, but delete[] it afterwards. In the
code above, I believe the delete[] line will not be executed following
the return statement.

>>How should I achieve this?

How can you be assured that the pointer would not be deleted until
after the calling routine had finished accessing the data??
If you create storage and return a pointer to that storage, you cannot
also delete that storage, not unless the calling routine will never
attempt to access the storage nor to test the pointer against anything
other than a null pointer constant.

One would suspect that your code is not in fact C code, in
which case our most charitable assumption would be that you
have -accidently- posted your question to the wrong newsgroup.

But even if so, you need to figure out what it is supposed to
mean to delete a pointer to a storage area when you have returned
that pointer outwards. There are not many languages around
(-none- that I can think of right now) in which returning a pointer
to a memory block is understood to mean that the procedure
calling mechanism should take a -copy- of the storage pointed to
and put that copy somewhere for future use.

It amazed me that there are already about 5 people who have told him
this is C++ code and yet you feel the need to go lengths to belittle the
OP.

You did know there had been OT replies. You did know that it was
C++. One can only guess at your motives for posting such a long and
relatively useless reply.

[Malcolm McLean]

"Richard" <de***@gmail.comwrote in message
news:fu**********@registered.motzarella.org...

ro******@ibd.nrc-cnrc.gc.ca (Walter Roberson) writes:

>In article <fu**********@aioe.org>, pradeep <no****@nospam.comwrote:

>>>I have the following code,

>>>char* MyClass::MyMethod()

That line is a syntax error. The only place that a colon (':')
can occur in a type description in C is as a bit-field length
within a structure definition.

It amazed me that there are already about 5 people who have told him
this is C++ code and yet you feel the need to go lengths to belittle the
OP.

You did know there had been OT replies. You did know that it was
C++. One can only guess at your motives for posting such a long and
relatively useless reply.

Unfortunately Walter's sort of reply - and we can all be clever Dicks to
people new to programming - encourages people like Antonius Twink to give
off-topic answers as a sort of compensation.
Why can't people just say "this is C++"?

--
Free games and programming goodies.
http://www.personal.leeds.ac.uk/~bgy1mm

[Antoninus Twink]

On 26 Apr 2008 at 11:39, Ben Bacarisse wrote:

Antoninus Twink <no****@nospam.invalidwrites:

>In C++, ...

I won't point out the C++ errors you made (that would be to fall into
your trap)

If there were real errors (not just the usual not standard, not portable
bullshit) then point them out and cross-post to clc++ if you insist.

[Richard]

"Malcolm McLean" <re*******@btinternet.comwrites:

"Richard" <de***@gmail.comwrote in message
news:fu**********@registered.motzarella.org...

>ro******@ibd.nrc-cnrc.gc.ca (Walter Roberson) writes:

>>In article <fu**********@aioe.org>, pradeep <no****@nospam.comwrote:

I have the following code,

char* MyClass::MyMethod()

That line is a syntax error. The only place that a colon (':')
can occur in a type description in C is as a bit-field length
within a structure definition.

It amazed me that there are already about 5 people who have told him
this is C++ code and yet you feel the need to go lengths to belittle the
OP.

You did know there had been OT replies. You did know that it was
C++. One can only guess at your motives for posting such a long and
relatively useless reply.

Unfortunately Walter's sort of reply - and we can all be clever Dicks
to people new to programming - encourages people like Antonius Twink
to give off-topic answers as a sort of compensation.
Why can't people just say "this is C++"?

or why not post nothing at all when they KNOW Santosh, Falconer or Default Luser
will correct them.

[Ben Bacarisse]

Antoninus Twink <no****@nospam.invalidwrites:

On 26 Apr 2008 at 11:39, Ben Bacarisse wrote:

>Antoninus Twink <no****@nospam.invalidwrites:

>>In C++, ...

I won't point out the C++ errors you made (that would be to fall into
your trap)

If there were real errors (not just the usual not standard, not portable
bullshit) then point them out and cross-post to clc++ if you insist.

There is nothing stopping you posting your code in comp.lang.c++ and
there is nothing I can do to stop you if you choose to do something so
daft as cross-posting C++ code in both groups, but you can't sucker me
into doing it for you.

--
Ben.

[Antoninus Twink]

On 26 Apr 2008 at 10:24, santosh wrote:

Antoninus Twink wrote:

>There are several problems and misunderstandings in your code...

<snip C++ code and response>

Nowhere in your post did you mention to the OP that there is in fact a
dedicated Usenet group for C++ where answers are likely to be more
correct than OT responses in other groups. Why is that? Do you
deliberately want the OP to remain ignorant of the best forum for help
with his problem and risk betting on possibly incorrect or outdated
advice in other groups?

There is never any shortage of people who point out "more appropriate
groups" (usually with considerable bile and hostility, but
that's by the by).

Answering questions on non-conforming C code is one thing but answering
questions on totally separate languages and other wildly OT matters
(like the recent thread on Ethernet cables) seems a deliberate tactic
to lower the S:N ratio and wreck this group.

Here are some of the things I discussed with my colleagues at lunch
yesterday: C programming; Java programming; tree data structures; a
problem with someone's car; the new Ubuntu release; national politics. I
can promise you that my group hasn't been "wrecked" by this terrible
"lowering of the S:N ratio". On the contrary, in real life discussions
are eclectic by their nature, and talking about other things besides the
intricate minutiae of the C standard actually help build a positive
community, and that's good for exchanging programming knowledge too.

[santosh]

Antoninus Twink wrote:

On 26 Apr 2008 at 10:24, santosh wrote:

<snip>

>Answering questions on non-conforming C code is one thing but
answering questions on totally separate languages and other wildly OT
matters (like the recent thread on Ethernet cables) seems a
deliberate tactic to lower the S:N ratio and wreck this group.

Here are some of the things I discussed with my colleagues at lunch
yesterday: C programming; Java programming; tree data structures; a
problem with someone's car; the new Ubuntu release; national politics.
I can promise you that my group hasn't been "wrecked" by this terrible
"lowering of the S:N ratio".

Why are you comparing a face-to-face group discussion with a newsgroup?
They are totally different beyond the basic fact that both are used for
exchange of ideas.

On the contrary, in real life discussions

^^^^^^^^^^
<snip>

So anything further you say has no bearing for a newsgroup.

As I said, answering non-standard C questions is one thing but answering
questions on everthing under the Sun is only going to encourage newbies
and lurkers to start posting OT and drown the topical content.

There is also the fact that non-C answers are likely to have lower rate
of correctness, and this combined with deliberately concealing from the
newbie the existence of a dedicated group and inviting him to post
further non-C questions here is a disservice not only to the group
(which I understand that you don't like) but also to the newbie.

are eclectic by their nature, and talking about other things besides
the intricate minutiae of the C standard actually help build a
positive community, and that's good for exchanging programming
knowledge too.

No. Newsgroups are not communities. They are places to ask questions and
air ideas on the specified topic and receive responses. They don't
function like real-life informal communities or groups. They can be
better compared to a real-life support group. Do you also go to
Alcoholics Anonymous and complain of your C problems?

[Default User]

santosh wrote:

Antoninus Twink wrote:

[]

Why are you comparing a face-to-face group discussion with a
newsgroup?

Please don't feed the trolls.


Brian

[Richard]

"Default User" <de***********@yahoo.comwrites:

santosh wrote:

>Antoninus Twink wrote:

[]

>Why are you comparing a face-to-face group discussion with a
newsgroup?

Please don't feed the trolls.


Brian

Please do not wake up dead threads with your unnecessary net nannying.

[Walter Roberson]

In article <fu**********@registered.motzarella.org>,
Richard <de***@gmail.comwrote:

>It amazed me that there are already about 5 people who have told him
this is C++ code and yet you feel the need to go lengths to belittle the
OP.

Check out the postings times on the messages. My message was in
composition while the other (shorter) messages were sent.

>You did know that it was C++.

I do *not* know that it was C++. It did -look- sort of like C++,
but there are other languages that look like C++ as well. For
example, I don't know enough C# to be able to tell C++ apart from C#.
--
"Allegories are in the realm of thoughts, what ruins are in
the realm of things." -- Walter Benjamin

[Walter Roberson]

In article <7N******************************@bt.com>,
Malcolm McLean <re*******@btinternet.comwrote:

>Unfortunately Walter's sort of reply

"sort of reply"? I am the only one who dealt with the fundamental
issue of storage lifetimes and wanting to delete a storage area
while it was still live.
--
"I want to be remembered as the guy who gave his all whenever
he was on the field." -- Walter Payton

[Malcolm McLean]

"Walter Roberson" <ro******@ibd.nrc-cnrc.gc.cawrote in message
news:fu**********@canopus.cc.umanitoba.ca...

In article <7N******************************@bt.com>,
Malcolm McLean <re*******@btinternet.comwrote:

>>Unfortunately Walter's sort of reply

"sort of reply"? I am the only one who dealt with the fundamental
issue of storage lifetimes and wanting to delete a storage area
while it was still live.

OP:

>>char* MyClass::MyMethod()

Walter

>That line is a syntax error. The only place that a colon (':')
can occur in a type description in C is as a bit-field length
within a structure definition.

This is clever Dickery, or to use a more accurate term, disingenuous.

--
Free games and programming goodies.
http://www.personal.leeds.ac.uk/~bgy1mm

[Walter Roberson]

In article <I-******************************@bt.com>,
Malcolm McLean <re*******@btinternet.comwrote:

>OP:

>>>char* MyClass::MyMethod()

>Walter

>>That line is a syntax error. The only place that a colon (':')
can occur in a type description in C is as a bit-field length
within a structure definition.

>This is clever Dickery, or to use a more accurate term, disingenuous.

What, did I forget a case? Hmmm, I guess I did -- a ':' could occur
as part of a :? operation on compile-time constants that were
designating an array size. Okay, so perhaps that part was
"non-ingenuous" in that it was lacking the genius to create the
second case. But "disingenuous" implies an attempt to mislead,
whereas my reply was an attempt to set the OP straight.

--
Current spam load: 750-800 messages per day (March 4, 2008)

[Old Wolf]

On Apr 26, 9:59 pm, Antoninus Twink <[e-mail]:

This does not copy the string "hello" into the array you've just
allocated, but rather makes ptr point to the start of a string literal,
which will be stored in the data segment of your object file.

Just stick to your trolling, and let the people who
actually know anything about C answer the questions.
It'll be less painful for all of us, believe me.

[jacob navia]

Old Wolf wrote:

On Apr 26, 9:59 pm, Antoninus Twink <[e-mail]:

>This does not copy the string "hello" into the array you've just
allocated, but rather makes ptr point to the start of a string literal,
which will be stored in the data segment of your object file.

Just stick to your trolling, and let the people who
actually know anything about C answer the questions.
It'll be less painful for all of us, believe me.

That's why you have (and should) remain silent. You know nothing
about C. All C implementations have a data segment.

All initial values of tables or numbers are stored there.
--
jacob navia
jacob at jacob point remcomp point fr
logiciels/informatique
http://www.cs.virginia.edu/~lcc-win32

[Old Wolf]

On Apr 26, 11:42 pm, Richard <[e-mail]:

You did know there had been OT replies. You did know that it was
C++. One can only guess at your motives for posting such a long and
relatively useless reply.

My money's on the OP being a sock puppet of 'Richard',
so that the trolling trio of c.l.c* can unleash their
incomparable wit on any non-troll who replies.

[Antoninus Twink]

On 27 Apr 2008 at 10:09, Old Wolf wrote:

Just stick to your trolling, and let the people who actually know
anything about C answer the questions. It'll be less painful for all
of us, believe me.

If "the people who actually know anything about C" is a (presumably
satirical) reference to The Clique, then I should point out that I
didn't notice a single member of that august group answering the OP's
question, although several decided to use bandwidth by telling him to
get lost.

If there is an error in any of my posts, point it out. But there was no
error, and all you can do is carry on with your stupid "neh neh not
portable" sniping from the sidelines.

[Flash Gordon]

jacob navia wrote, On 27/04/08 11:13:

Old Wolf wrote:

<snip>

That's why you have (and should) remain silent. You know nothing
about C. All C implementations have a data segment.

All initial values of tables or numbers are stored there.

Incorrect. On the TMS320C2xx/5xx compiler initial values are normally
stored in a .cinit section, the .data section (not data if I recall
correctly) is used for other purposes.

Even excluding implementations which are interpreters there may well be
implementations that do not have a data section or don't even have sections.
--
Flash Gordon

[Flash Gordon]

Antoninus Twink wrote, On 27/04/08 11:55:

On 27 Apr 2008 at 10:09, Old Wolf wrote:

>Just stick to your trolling, and let the people who actually know
anything about C answer the questions. It'll be less painful for all
of us, believe me.

If "the people who actually know anything about C" is a (presumably
satirical) reference to The Clique, then I should point out that I
didn't notice a single member of that august group answering the OP's
question, although several decided to use bandwidth by telling him to
get lost.

What does knowing C have to do with answering a C++ question?

If there is an error in any of my posts, point it out. But there was no
error, and all you can do is carry on with your stupid "neh neh not
portable" sniping from the sidelines.

No one claimed that C++ is not portable, only that it is a different
language with its own group. Do you deny that there are more people who
know C++ on comp.lang.c++ than in this group?
--
Flash Gordon

[Kenny McCormack]

In article <34**********************************@c19g2000prf. googlegroups.com>,
Old Wolf <ol*****@inspire.net.nzwrote:

>On Apr 26, 11:42 pm, Richard <[e-mail]:

>You did know there had been OT replies. You did know that it was
C++. One can only guess at your motives for posting such a long and
relatively useless reply.

My money's on the OP being a sock puppet of 'Richard',
so that the trolling trio of c.l.c* can unleash their
incomparable wit on any non-troll who replies.

Hurts to be so out-classed, doesn't it?

[CBFalconer]

jacob navia wrote:

Old Wolf wrote:

>Antoninus Twink <[e-mail]:

>>This does not copy the string "hello" into the array you've just
allocated, but rather makes ptr point to the start of a string
literal, which will be stored in the data segment of your object
file.

Just stick to your trolling, and let the people who
actually know anything about C answer the questions.
It'll be less painful for all of us, believe me.

That's why you have (and should) remain silent. You know nothing
about C. All C implementations have a data segment.

All initial values of tables or numbers are stored there.

FYI Twink is the troll, not Old Wolf.

In addition, immediate operand values are stored in code segments,
and often used to initiate tables, values, etc.

--
[mail]: Chuck F (cbfalconer at maineline dot net)
[page]: <http://cbfalconer.home.att.net>
Try the download section.
** Posted from http://www.teranews.com **

[Walter Roberson]

In article <fv**********@aioe.org>, jacob navia <ja***@nospam.orgwrote:

>All C implementations have a data segment.

ELF file formats (e.g., SGI IRIX) do not have segments: they
have sections.
--
"No one has the right to destroy another person's belief by
demanding empirical evidence." -- Ann Landers

[santosh]

Walter Roberson wrote:

In article <fv**********@aioe.org>, jacob navia <ja***@nospam.org>
wrote:

>>All C implementations have a data segment.

ELF file formats (e.g., SGI IRIX) do not have segments: they
have sections.

The situation is similar for DOS programs under the real mode flat
address model, i.e., .COM programs.

[Kenny McCormack]

In article <48**************@yahoo.com>,
CBFalconer <cb********@maineline.netwrote:
....

>FYI Twink is the troll, not Old Wolf.

Surely, trolling is in the eye of the beholder.

At this stage, most of the useful information being posted in CLC is
being posted by Mr. Twink, so it hardly seem accurate to call him a
troll. And, obviously, if you do, it is clearly a twisted definition of
the word.

Now, though I would not call any of the "old guard" regulars, such as
KT & RH, "trolls" (though I certainly have called them other things),
lately, it has gotten a lot harder to tell the "trolls" from the "reg
wannabees", this later category being the one in which, I trust we all
agree, all observers would class you.