search

sabriallani / ticpp

Automatically exported from code.google.com/p/ticpp
0 stars 0 forks source link

Uninitialized m_impRC in NodeImp copy constructor #9

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The NodeImp copy constructor doesn't initialize the m_impRC member.

Original issue reported on code.google.com by ola...@gmail.com on 12 Sep 2007 at 2:09

GoogleCodeExporter commented 8 years ago 
could you update to svn rev 71 and let me know if this is still an issue??

Original comment by rjmy...@gmail.com on 1 Oct 2007 at 2:26

GoogleCodeExporter commented 8 years ago 
The issue happens when I'm loading a file in a document object, get an element 
object
out of the document and then use the element when the document have gone out of
scope. The Element then gets its pointers invalidated.

Am I supposed to keep the document in scope as long as I'm using elements from 
it?

Original comment by ola...@gmail.com on 3 Oct 2007 at 8:53

GoogleCodeExporter commented 8 years ago 
One of the ways that memory is managed in tinyxml is that when a node is 
deleted, it
deletes all of its children.

So, when the document goes out of scope, all of the nodes will be deleted. So, 
the
answer is yes, keep the document in scope as long as you are using elements 
from it.

Original comment by rjmy...@gmail.com on 3 Oct 2007 at 11:05

GoogleCodeExporter commented 8 years ago 
scope issue, not a bug

Original comment by rjmy...@gmail.com on 6 Nov 2007 at 3:03