tweak test
diff --git a/xmltest.cpp b/xmltest.cpp
index 1febf72..ae0031c 100644
--- a/xmltest.cpp
+++ b/xmltest.cpp
@@ -1758,17 +1758,17 @@
// Crashing reported via email.
const char* xml =
"<playlist id='playlist1'>"
- "<property name='track_name'>voice</property>"
- "<property name='audio_track'>1</property>"
- "<entry out = '604' producer = '4_playlist1' in = '0' />"
- "<blank length = '1' />"
- "<entry out = '1625' producer = '3_playlist' in = '0' />"
- "<blank length = '2' />"
- "<entry out = '946' producer = '2_playlist1' in = '0' />"
- "<blank length = '1' />"
- "<entry out = '128' producer = '1_playlist1' in = '0' />"
+ "<property name='track_name'>voice</property>"
+ "<property name='audio_track'>1</property>"
+ "<entry out = '604' producer = '4_playlist1' in = '0' />"
+ "<blank length = '1' />"
+ "<entry out = '1625' producer = '3_playlist' in = '0' />"
+ "<blank length = '2' />"
+ "<entry out = '946' producer = '2_playlist1' in = '0' />"
+ "<blank length = '1' />"
+ "<entry out = '128' producer = '1_playlist1' in = '0' />"
"</playlist>";
-
+
// It's not a good idea to delete elements as you walk the
// list. I'm not sure this technically should work; but it's
// an interesting test case.
@@ -1797,7 +1797,13 @@
playlist->Accept(&printer);
printf("%s\n", printer.CStr());
- // No test; it only need to not crash.
+ // No test; it only need to not crash.
+ // Still, wrap it up with a sanity check
+ int nProperty = 0;
+ for (const XMLElement* p = playlist->FirstChildElement("property"); p; p = p->NextSiblingElement("property")) {
+ nProperty++;
+ }
+ XMLTest("Crash bug parsing", nProperty, 2);
}
// ----------- Line Number Tracking --------------