Hi - nice job. I was recently surprised to find that GetText() only works "If the first child of 'this' is a TiXmlText" (from tinyxml.h). Being somewhat new to the project, I was wondering if this is a structural requirement? From what I can tell nothing would be broken by the following:
Or, now that this message is written, it occurs to me that possibly two new functions would be appropriate, GetFirstText() and GetNextText()…which is getting pretty close to the existing ability to simply iterate through all the nodes that are text elements. Hmm.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Hi - nice job. I was recently surprised to find that GetText() only works "If the first child of 'this' is a TiXmlText" (from tinyxml.h). Being somewhat new to the project, I was wondering if this is a structural requirement? From what I can tell nothing would be broken by the following:
Change from (existing)
to (proposed)
Or, now that this message is written, it occurs to me that possibly two new functions would be appropriate, GetFirstText() and GetNextText()…which is getting pretty close to the existing ability to simply iterate through all the nodes that are text elements. Hmm.