- error.c: applied the context output patch of the error

handling submitted by Chuck Griffith
- error/VC/*: this slightly change some error logs
Daniel

ChangeLog

@@ -1,3 +1,9 @@
+Tue Mar 27 02:30:23 CEST 2001 Daniel Veillard <Daniel.Veillard@imag.fr>
+
+	* error.c: applied the context output patch of the error
+	  handling submitted by Chuck Griffith
+	* error/VC/*: this slightly change some error logs
+
 Tue Mar 27 00:51:27 CEST 2001 Daniel Veillard <Daniel.Veillard@imag.fr>
 
 	* parser.c: fixed line number reporting on error

error.c

@@ -107,34 +107,44 @@ void
 xmlParserPrintFileContext(xmlParserInputPtr input) {
     const xmlChar *cur, *base;
     int n;
+    xmlChar  content[81];
+    xmlChar *ctnt;
 
     if (input == NULL) return;
     cur = input->cur;
     base = input->base;
+    /* skip backwards over any end-of-lines */
     while ((cur > base) && ((*cur == '\n') || (*cur == '\r'))) {
 	cur--;
     }
     n = 0;
+    /* search backwards for beginning-of-line maximum 80 characters */
     while ((n++ < 80) && (cur > base) && (*cur != '\n') && (*cur != '\r'))
         cur--;
     if ((*cur == '\n') || (*cur == '\r')) cur++;
-    base = cur;
+	/* search forward for end-of-line maximum 80 characters */
     n = 0;
+    ctnt = content;
     while ((*cur != 0) && (*cur != '\n') && (*cur != '\r') && (n < 79)) {
-        xmlGenericError(xmlGenericErrorContext,
+		*ctnt++ = *cur++;
-		"%c", (unsigned char) *cur++);
 	n++;
     }
-    xmlGenericError(xmlGenericErrorContext, "\n");
+    *ctnt = 0;
+    xmlGenericError(xmlGenericErrorContext,"%s\n", content);
+    /* create blank line with problem pointer */
     cur = input->cur;
-    while ((*cur == '\n') || (*cur == '\r'))
+    while ((cur > base) && ((*cur == '\n') || (*cur == '\r'))) {
-	cur--;
+		cur--;
+	}
     n = 0;
-    while ((cur != base) && (n++ < 80)) {
+	ctnt = content;
-        xmlGenericError(xmlGenericErrorContext, " ");
+    while ((n++ < 79) && (cur > base) && (*cur != '\n') && (*cur != '\r')) {
-        base++;
+	*ctnt++ = ' ';
+	cur--;
     }
-    xmlGenericError(xmlGenericErrorContext,"^\n");
+    *(--ctnt) = '^';
+    *(++ctnt) = 0;
+    xmlGenericError(xmlGenericErrorContext,"%s\n", content);
 }
 
 /**

result/VC/PENesting

@@ -1,6 +1,6 @@
 ./test/VC/PENesting:1: error: xmlParseStartTag: invalid element name
 <!ENTITY % pe1 "EMPTY> <!ELEMENT e2 EMPTY>"> 
- ^
+^
 ./test/VC/PENesting:1: error: Extra content at the end of the document
 <!ENTITY % pe1 "EMPTY> <!ELEMENT e2 EMPTY>"> 
- ^
+^

result/VC/PENesting2

@@ -1,6 +1,6 @@
 ./test/VC/PENesting2:1: error: xmlParseStartTag: invalid element name
 <!ENTITY % p1 "(A|B">
- ^
+^
 ./test/VC/PENesting2:1: error: Extra content at the end of the document
 <!ENTITY % p1 "(A|B">
- ^
+^

result/VC/UniqueElementTypeDeclaration

@@ -1,3 +1,3 @@
 test/VC/dtds/a.dtd:1: validity error: Redefinition of element a
 <!ELEMENT a (#PCDATA | b | c)*>
-                              ^
+                             ^

result/valid/xlink.xml.err

@@ -1,6 +1,6 @@
-./test/valid/xlink.xml:816: validity error: ID dt-arc already defined
+./test/valid/xlink.xml:450: validity error: ID dt-arc already defined
 	<p><termdef id="dt-arc" term="Arc">An <term>arc</term> is contained within an 
                                    ^
-./test/valid/xlink.xml:956: validity error: IDREF attribute def reference an unknown ID "dt-xlg"
+./test/valid/xlink.xml:530: validity error: IDREF attribute def reference an unknown ID "dt-xlg"
 
 ^