#19 missing breaks for switches cases

[Anonymous]

I think these missing breaks are not intentional:

No break at the end of case rapidxml.hpp line 1846
No break at the end of case rapidxml.hpp line 1852
No break at the end of case rapidxml.hpp line 2148

[Bernd]

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!