xml.c:parse_region: Do not forget the first document child.

* src/xml.c (parse_region): Do not forget the first document child.

* test/automated/libxml-tests.el: New file.

4 files changed, 65 insertions, 1 deletions

diff --git a/src/ChangeLog b/src/ChangeLog
index 64dabbbfadb..25009f8a310 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,7 @@
+2014-10-28  Ulf Jasper  <ulf.jasper@web.de>
+
+        * xml.c (parse_region): Do not forget the first document child.
+
 2014-10-25  Jan Djärv  <jan.h.d@swipnet.se>
 
         * nsselect.m: pasteboard_changecount is new.
diff --git a/src/xml.c b/src/xml.c
index feabe00efab..7e99beb1d05 100644
--- a/src/xml.c
+++ b/src/xml.c
@@ -216,7 +216,7 @@ parse_region (Lisp_Object start, Lisp_Object end, Lisp_Object base_url, int html
     {
       /* If the document is just comments, then this should get us the
          nodes anyway. */
-      xmlNode *n = doc->children->next;
+      xmlNode *n = doc->children;
       Lisp_Object r = Qnil;
 
       while (n) {
diff --git a/test/ChangeLog b/test/ChangeLog
index fb19252cd34..86f8e240af4 100644
--- a/test/ChangeLog
+++ b/test/ChangeLog
@@ -1,3 +1,7 @@
+2014-10-28  Ulf Jasper  <ulf.jasper@web.de>
+
+        * automated/libxml-tests.el: New file.
+
 2014-10-22  Noam Postavsky  <npostavs@users.sourceforget.net>
 
         * test/automated/process-tests.el (process-test-quoted-batfile):
diff --git a/test/automated/libxml-tests.el b/test/automated/libxml-tests.el
new file mode 100644
index 00000000000..ced0df7b3c4
--- /dev/null
+++ b/test/automated/libxml-tests.el
@@ -0,0 +1,56 @@
+;;; libxml-parse-tests.el --- Test suite for libxml parsing.
+
+;; Copyright (C) 2014 Free Software Foundation, Inc.
+
+;; Author: Ulf Jasper <ulf.jasper@web.de>
+;; Keywords:       internal
+;; Human-Keywords: internal
+
+;; This file is part of GNU Emacs.
+
+;; GNU Emacs is free software: you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
+
+;; GNU Emacs is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;; GNU General Public License for more details.
+
+;; You should have received a copy of the GNU General Public License
+;; along with GNU Emacs.  If not, see <http://www.gnu.org/licenses/>.
+
+;;; Commentary:
+
+;;; Code:
+
+(require 'ert)
+
+(defvar libxml-tests--data
+  `(;; simple case
+    ("<?xml version=\"1.0\"?><foo baz=\"true\">bar</foo>"
+     . (foo ((baz . "true")) "bar"))
+    ;; toplevel comments -- first document child must not get lost
+    (,(concat "<?xml version=\"1.0\"?><foo>bar</foo><!--comment-1-->"
+              "<!--comment-2-->")
+     . (top nil (foo nil "bar") (comment nil "comment-1")
+            (comment nil "comment-2")))
+    (,(concat "<?xml version=\"1.0\"?><!--comment-a--><foo a=\"b\">"
+              "<bar>blub</bar></foo><!--comment-b--><!--comment-c-->")
+     . (top nil (comment nil "comment-a") (foo ((a . "b")) (bar nil "blub"))
+            (comment nil "comment-b") (comment nil "comment-c"))))
+  "Alist of XML strings and their expected parse trees.")
+
+
+(ert-deftest libxml-tests ()
+  "Test libxml."
+  (when (fboundp 'libxml-parse-xml-region)
+    (with-temp-buffer
+      (dolist (test libxml-tests--data)
+        (erase-buffer)
+        (insert (car test))
+        (should (equal (cdr test)
+                       (libxml-parse-xml-region (point-min) (point-max))))))))
+
+;;; libxml-tests.el ends here