trying to fix #331062, this is again a problem around interleave, there is
* relaxng.c: trying to fix #331062, this is again a problem
around interleave, there is no good fix unless reimplementing
but this works around some cases and allow to validate in that
case.
Daniel
diff --git a/ChangeLog b/ChangeLog
index 0c66d84..a98d4d0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+Sun Feb 19 16:20:43 CET 2006 Daniel Veillard <daniel@veillard.com>
+
+ * relaxng.c: trying to fix #331062, this is again a problem
+ around interleave, there is no good fix unless reimplementing
+ but this works around some cases and allow to validate in that
+ case.
+
Wed Feb 15 11:55:22 CET 2006 Kasimier Buchcik <libxml2-cvs@cazic.net>
* tree.c: Fixed bug #328896 reported by Liron. The path
diff --git a/relaxng.c b/relaxng.c
index be962eb..2d26667 100644
--- a/relaxng.c
+++ b/relaxng.c
@@ -45,27 +45,29 @@
(xmlStrEqual(node->ns->href, xmlRelaxNGNs)))
-/* #define DEBUG 1 */
+#if 0
+#define DEBUG 1
-/* #define DEBUG_GRAMMAR 1 */
+#define DEBUG_GRAMMAR 1
-/* #define DEBUG_CONTENT 1 */
+#define DEBUG_CONTENT 1
-/* #define DEBUG_TYPE 1 */
+#define DEBUG_TYPE 1
-/* #define DEBUG_VALID 1 */
+#define DEBUG_VALID 1
-/* #define DEBUG_INTERLEAVE 1 */
+#define DEBUG_INTERLEAVE 1
-/* #define DEBUG_LIST 1 */
+#define DEBUG_LIST 1
-/* #define DEBUG_INCLUDE */
+#define DEBUG_INCLUDE 1
-/* #define DEBUG_ERROR 1 */
+#define DEBUG_ERROR 1
-/* #define DEBUG_COMPILE 1 */
+#define DEBUG_COMPILE 1
-/* #define DEBUG_PROGRESSIVE 1 */
+#define DEBUG_PROGRESSIVE 1
+#endif
#define MAX_ERROR 5
@@ -9267,21 +9269,56 @@
} else if (ctxt->states != NULL) {
int j;
int found = 0;
+ int best = -1;
+ int lowattr = -1;
+
+ /*
+ * PBM: what happen if there is attributes checks in the interleaves
+ */
for (j = 0; j < ctxt->states->nbState; j++) {
cur = ctxt->states->tabState[j]->seq;
cur = xmlRelaxNGSkipIgnored(ctxt, cur);
if (cur == NULL) {
+ if (found == 0) {
+ lowattr = ctxt->states->tabState[j]->nbAttrLeft;
+ best = j;
+ }
found = 1;
- break;
- }
+ if (ctxt->states->tabState[j]->nbAttrLeft <= lowattr) {
+ /* try to keep the latest one to mach old heuristic */
+ lowattr = ctxt->states->tabState[j]->nbAttrLeft;
+ best = j;
+ }
+ if (lowattr == 0)
+ break;
+ } else if (found == 0) {
+ if (lowattr == -1) {
+ lowattr = ctxt->states->tabState[j]->nbAttrLeft;
+ best = j;
+ } else
+ if (ctxt->states->tabState[j]->nbAttrLeft <= lowattr) {
+ /* try to keep the latest one to mach old heuristic */
+ lowattr = ctxt->states->tabState[j]->nbAttrLeft;
+ best = j;
+ }
+ }
}
+ /*
+ * BIG PBM: here we pick only one restarting point :-(
+ */
if (ctxt->states->nbState > 0) {
xmlRelaxNGFreeValidState(ctxt, oldstate);
- oldstate =
- ctxt->states->tabState[ctxt->states->nbState - 1];
+ if (best != -1) {
+ oldstate = ctxt->states->tabState[best];
+ ctxt->states->tabState[best] = NULL;
+ } else {
+ oldstate =
+ ctxt->states->tabState[ctxt->states->nbState - 1];
+ ctxt->states->tabState[ctxt->states->nbState - 1] = NULL;
+ }
}
- for (j = 0; j < ctxt->states->nbState - 1; j++) {
+ for (j = 0; j < ctxt->states->nbState ; j++) {
xmlRelaxNGFreeValidState(ctxt, ctxt->states->tabState[j]);
}
xmlRelaxNGFreeStates(ctxt, ctxt->states);