- parser.c: fixed a reported bug in NOTATION parsing

- uri.c: accepted but not fixed bug 51876, added TODO
- Makefile.am: fixed bug 51876
Daniel

ChangeLog

@@ -1,3 +1,9 @@
+Thu Mar 22 15:36:45 CET 2001 Daniel Veillard <Daniel.Veillard@imag.fr>
+
+	* parser.c: fixed a reported bug in NOTATION parsing
+	* uri.c: accepted but not fixed bug 51876, added TODO 
+	* Makefile.am: fixed bug 51876
+
 Thu Mar 22 13:41:22 CET 2001 Daniel Veillard <Daniel.Veillard@imag.fr>
 
 	* config.h.in configure.in error.c: fix a compilation problem

Makefile.am

@@ -75,7 +75,7 @@ testURI_LDADD= $(LDADDS)
 check-local: tests
 
 $(srcdir)/libxml:
-	-$(RM) $(srcdir)/libxml
+	-$(RM) -f $(srcdir)/libxml
 	ln -s $(srcdir)/. $(srcdir)/libxml
 
 install-data: $(srcdir)/libxml

parser.c

@@ -2588,6 +2588,8 @@ xmlParseExternalID(xmlParserCtxtPtr ctxt, xmlChar **publicID, int strict) {
     xmlChar *URI = NULL;
 
     SHRINK;
+
+    *publicID = NULL;
     if ((RAW == 'S') && (NXT(1) == 'Y') &&
          (NXT(2) == 'S') && (NXT(3) == 'T') &&
 	 (NXT(4) == 'E') && (NXT(5) == 'M')) {

uri.c

@@ -987,6 +987,10 @@ xmlURIUnescapeString(const char *str, int len, char *target) {
  * It will try to escape the chars needing this, but this is heuristic
  * based it's impossible to be sure.
  *
+ * TODO: make the proper implementation of this function by calling
+ *       xmlParseURIReference() and escaping each section accordingly
+ *       to the rules (c.f. bug 51876)
+ *
  * Returns an copy of the string, but escaped
  */
 xmlChar *