Suggested improvements (patch supplied)

[GoogleCodeExporter]

Here is a patch which does the following:

- Adds support for ticpp::Document::Parse(), wrapping the functionality of
TiXmlDocument::Parse()

- Adds support to ticpp::Node for std::stream << and >> operators, wrapping
the functionality of TiXmlNode's friendly streaming operators.

- Adds an template Element() constructor that allws you to also specify
element text (any variable). This is especially useful, as you can now in
one line go:
node.LinkEndChild( &ticpp::Element( "some-tag", some_val ) );

- Improves THROW() macro (now called TICPPTHROW) so it doesn't clash with
MFC macro of the same name and now creates a {block}, so that funny-named
variable can be dispensed with!

- Bases ticpp::Exception on std::exception (as previously requested). This
also means you can now catch all STL and TICPP exceptions in one catch
statement if you want, or two if you want to distinguish.

- Fixes bug (unnecessary cleanup) in ticpp::NodeImp's copy constructor

If you want to apply the patch (for those that dont know), type the following:
  cd tinyxml++
  patch -p1 < tinyxml++.patch
if you're using windows, you might need to install gnu patch (or cygwin).

Original issue reported on code.google.com by e...@waxworlds.org on 14 Mar 2007 at 6:35

Attachments:

• tinyxml++.patch

[GoogleCodeExporter]

Sorry, I should just say, this patch is for tinyxml++ version 0.04.

Original comment by e...@waxworlds.org on 14 Mar 2007 at 9:27

[GoogleCodeExporter]

That's great. I just use TiXmlDocument::Parse() function to parse data in 
memory. 
But I can't apply your patch. Why don't you post the source code or apply the 
patch 
to svn directly.

Original comment by zhou.hao...@gmail.com on 25 May 2007 at 7:46

[GoogleCodeExporter]

Because I don't have commit access to the repository. I am not one of the 
developers.

Talking of the developers, what's happened to them?!

I don't know why you can't apply the patch (and can't help since you have not
provided any more info!). However, I have attached a zip of the patched source 
code
for you (and anyone else) who is not so familiar with patching.

Original comment by e...@waxworlds.org on 16 Jul 2007 at 10:44

Attachments:

• ticpp-0.04a.zip

[GoogleCodeExporter]

I've applied your patch, and am very grateful.
Please accept my apologies for being inactive for so long.
I did not setup the issue notifications correctly, and did not know there were 
any.

If you would like to join the project, please email me.

svn rev 71.

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

• Changed state: Fixed