Recent changes to bugs

#23 xml_document::parse broken for code points 256+

Fabian W — Fri, 10 Mar 2017 10:27:40 -0000

Works fine! Thanks a lot dude :D

Compiler warnings

Anders Skoglund — Mon, 15 Aug 2016 14:27:46 -0000

Local variables and arguments have the same name as member functions. In particular name, value, type, and name_size. This causes a number of warnings. I have attached a patch that does the bare minimum to get rid of them.

#18 Fix for compilation problem.

Manuel Schmitz — Tue, 06 Oct 2015 12:39:35 -0000

I ran into the same problem with gcc 4.9.3 on Cygwin. My own version the fix is attached to this post.

This bug seems to be related to bug # 16 (with Clang).

#23 xml_document::parse broken for code points 256+

鈴見咲君高 — Sat, 08 Aug 2015 07:26:22 -0000

...Sorry, the patch seems somethnig wrong.

#23 xml_document::parse broken for code points 256+

鈴見咲君高 — Sat, 08 Aug 2015 01:04:35 -0000

This patch may help you.

Multiple Compile Failures on g++ 4.8.2 with -Weffc++ and -Werror

gfarrand — Wed, 03 Dec 2014 08:57:51 -0000

Environment
Ubuntu 14.04 LTS, with g++ version 4.8.2.

RapidXML
Version 1.13

Problem
There are a fair amount of compiler warnings that pop up with the -Weffc++ is enabled in the compiler. Anyone that promotes warnings to errors (like me) will have this get in the way.

Steps to Reproduce

Create a .cpp file which #includes rapidxml.hpp, and makes very light use of any of the objects (xml_node<char>, xml_attribute<char>, etc.).
Compile that .cpp file with the following command: g++ -c -g -O0 -Weffc++ -Wall -Wextra -Werror -fno-exceptions -std=c++0x <INCLUDES> <FILE>.cpp -o <FILE>.o
Note that the compile failed with a bunch of errors relating to non-virtual destructors, public default assignment operators and copy constructors for classes with pointer members, etc.

A Patch Fix
I have attached a patch to v1.13 of RapidXML to address these issues. I have only done very light testing, but these changes seem to have no functional impact.

xml_document::parse broken for code points 256+

Daniel Perry — Tue, 02 Sep 2014 15:10:53 -0000

When using with wchar_t characters- parse hangs when parsing characters outside look up table size.

#22 Percent symbol replaced with the word "Success" in cdata

Seth Call — Thu, 03 Jul 2014 02:11:30 -0000

Should have caught that myself. Thanks!

#19 missing breaks for switches cases

Bernd — Wed, 02 Jul 2014 09:32:54 -0000

1846:
Please read the documentation in the header for RAPIDXML_PARSE_ERROR.
RAPIDXML_PARSE_ERROR is defined to NEVER return.
1852: Last case (a "Default"). Doesn't need a break.
2148: Last case... same as above.

I know.. would be nice to have a break here - at least for better readability.
But this should be mentioned in a bug-report!

#20 Memory leak

Bernd — Wed, 02 Jul 2014 09:26:37 -0000

VC 2008 know placements new.
Also STL is using it and a lot of other contrainer-frameworks.
I never encountered a memeory leak with it.

I am working aleady 8 years with a framework that use this concept massivly.

RapidXML is a header-only software. So the generated code depends on what you include before this header. With some c++ magic you can break everything - like defining some global new operators...
But this is not on issue of rapidXML.