[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
master 9ae212fb1e5 4/7: xml.el: remove incorrect and botched attribute w
From: |
Mattias Engdegård |
Subject: |
master 9ae212fb1e5 4/7: xml.el: remove incorrect and botched attribute whitespace collapse |
Date: |
Tue, 30 May 2023 11:46:53 -0400 (EDT) |
branch: master
commit 9ae212fb1e5ef8cffd10ee174004caabf2027ba3
Author: Mattias Engdegård <mattiase@acm.org>
Commit: Mattias Engdegård <mattiase@acm.org>
xml.el: remove incorrect and botched attribute whitespace collapse
* lisp/xml.el (xml-parse-attlist): Don't attempt to collapse multiple
consecutive whitespace characters into a single space, which is wrong
to do a this point when attributes must be assumed to be CDATA.
The code was actually unintentionally correct since it forgot to use
the return value of `replace-regexp-in-string` (bug#63740).
---
lisp/xml.el | 3 ---
1 file changed, 3 deletions(-)
diff --git a/lisp/xml.el b/lisp/xml.el
index 58d17a4b340..9095653416e 100644
--- a/lisp/xml.el
+++ b/lisp/xml.el
@@ -669,10 +669,7 @@ Leave point at the first non-blank character after the
tag."
(if (assoc name attlist)
(error "XML: (Not Well-Formed) Each attribute must be unique within
an element"))
- ;; Multiple whitespace characters should be replaced with a single one
- ;; in the attributes
(let ((string (match-string-no-properties 1)))
- (replace-regexp-in-string "\\s-\\{2,\\}" " " string)
(let ((expansion (xml-substitute-special string)))
(unless (stringp expansion)
;; We say this is the constraint. It is actually that
- master updated (f8a79c0055f -> 49c56f33352), Mattias Engdegård, 2023/05/30
- master 9ae212fb1e5 4/7: xml.el: remove incorrect and botched attribute whitespace collapse,
Mattias Engdegård <=
- master 49c56f33352 7/7: Add function declarations, Mattias Engdegård, 2023/05/30
- master ace9f677558 2/7: Avoid using rx `any` for `not-newline`, Mattias Engdegård, 2023/05/30
- master 27d68d790f7 6/7: ; * admin/cus-test.el (cus-test-apropos): remove TODO (already done), Mattias Engdegård, 2023/05/30
- master 845f97d475b 5/7: ; * admin/admin.el (cusver-scan): remove broken warning, Mattias Engdegård, 2023/05/30
- master 08291e6f175 1/7: Clean up defcustom type quote check, Mattias Engdegård, 2023/05/30
- master 6e255ddfffa 3/7: * test/lisp/dnd-tests.el (dnd-tests-begin-text-drag): Fix typo., Mattias Engdegård, 2023/05/30