* bug#32003: 26.0.90; nxml wrong indentation after multi-line attribute
@ 2018-06-29 9:38 Benjamin Riefenstahl
2019-04-19 3:48 ` Noam Postavsky
0 siblings, 1 reply; 3+ messages in thread
From: Benjamin Riefenstahl @ 2018-06-29 9:38 UTC (permalink / raw
To: 32003
Hi all,
Consider this XML:
---- cut ----
<settings
xmlns="http://maven.apache.org/SETTINGS/1.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
https://maven.apache.org/xsd/settings-1.0.0.xsd">
<mirrors>
...
</mirrors>
</settings>
---- cut ----
Save to a file, nxml-mode should be activated, type tab on the line
"<mirrors>".
The element <mirrors> should be indented at 4 spaces, but it is indented
according to the attribute value of the previous element, unless I
eliminate the line-feed there.
I can reproduce this in "emacs -Q".
benny
^ permalink raw reply [flat|nested] 3+ messages in thread
* bug#32003: 26.0.90; nxml wrong indentation after multi-line attribute
2018-06-29 9:38 bug#32003: 26.0.90; nxml wrong indentation after multi-line attribute Benjamin Riefenstahl
@ 2019-04-19 3:48 ` Noam Postavsky
2019-05-09 11:41 ` Noam Postavsky
0 siblings, 1 reply; 3+ messages in thread
From: Noam Postavsky @ 2019-04-19 3:48 UTC (permalink / raw
To: Benjamin Riefenstahl
Cc: Vincent Lefevre, David Carlisle, Stefan Monnier, 32003
[-- Attachment #1: Type: text/plain, Size: 570 bytes --]
forcemerge 32003 33873
tags 32003 + patch
quit
Benjamin Riefenstahl <Riefenstahl@mecom.de> writes:
> <settings
> xmlns="http://maven.apache.org/SETTINGS/1.0.0"
> xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
> xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
> https://maven.apache.org/xsd/settings-1.0.0.xsd">
> <mirrors>
> ...
> </mirrors>
> </settings>
Here's a patch to fix this bug, Bug#33873, and the indentation part of Bug#32897.
[-- Attachment #2: patch --]
[-- Type: text/plain, Size: 6536 bytes --]
From fb5f2cfd8404acc3591ac38dae2eaeb14807d853 Mon Sep 17 00:00:00 2001
From: Noam Postavsky <npostavs@gmail.com>
Date: Thu, 18 Apr 2019 23:36:04 -0400
Subject: [PATCH] Fix nxml-get-inside (Bug#32003)
The change from 2016-01-16 "lisp/nxml: Use syntax-tables for comments"
made nxml-get-inside return non-nil for any string or comment,
including attribute strings. This caused incorrect and therefore
indentation.
* lisp/nxml/nxml-rap.el: Update commentary to reflect changes to
nxml-mode parsing.
(nxml-get-inside): Only return non-nil when inside comments and
generic strings, not normal quote-delimited strings.
* test/lisp/nxml/nxml-mode-tests.el: New tests.
---
lisp/nxml/nxml-rap.el | 40 ++++++++++++-------------
test/lisp/nxml/nxml-mode-tests.el | 62 +++++++++++++++++++++++++++++++++++++++
2 files changed, 80 insertions(+), 22 deletions(-)
create mode 100644 test/lisp/nxml/nxml-mode-tests.el
diff --git a/lisp/nxml/nxml-rap.el b/lisp/nxml/nxml-rap.el
index 2bd758be3a..4a535b2c40 100644
--- a/lisp/nxml/nxml-rap.el
+++ b/lisp/nxml/nxml-rap.el
@@ -35,35 +35,25 @@
;;
;; Our strategy is to keep track of just the problematic things.
;; Specifically, we keep track of all comments, CDATA sections and
-;; processing instructions in the instance. We do this by marking all
-;; except the first character of these with a non-nil nxml-inside text
-;; property. The value of the nxml-inside property is comment,
-;; cdata-section or processing-instruction. The first character does
-;; not have the nxml-inside property so we can find the beginning of
-;; the construct by looking for a change in a text property value
-;; (Emacs provides primitives for this). We use text properties
-;; rather than overlays, since the implementation of overlays doesn't
-;; look like it scales to large numbers of overlays in a buffer.
-;;
-;; We don't in fact track all these constructs, but only track them in
-;; some initial part of the instance.
+;; processing instructions in the instance. We do this by marking
+;; the first character of these with the generic string syntax by setting
+;; a 'syntax-table' text property in `sgml-syntax-propertize'.
;;
;; Thus to parse some random point in the file we first ensure that we
-;; have scanned up to that point. Then we search backwards for a
-;; <. Then we check whether the < has an nxml-inside property. If it
-;; does we go backwards to first character that does not have an
-;; nxml-inside property (this character must be a <). Then we start
-;; parsing forward from the < we have found.
+;; have scanned up to that point. Then we search backwards for a <.
+;; Then we check whether the < has the generic string syntax. If it
+;; does we go backwards to first character of the generic string (this
+;; character must be a <). Then we start parsing forward from the <
+;; we have found.
;;
;; The prolog has to be parsed specially, so we also keep track of the
;; end of the prolog in `nxml-prolog-end'. The prolog is reparsed on
;; every change to the prolog. This won't work well if people try to
;; edit huge internal subsets. Hopefully that will be rare.
;;
-;; We keep track of the changes by adding to the buffer's
-;; after-change-functions hook. Scanning is also done as a
-;; prerequisite to fontification by adding to fontification-functions
-;; (in the same way as jit-lock). This means that scanning for these
+;; We rely on the `syntax-propertize-function' machinery to keep track
+;; of the changes in the buffer. Fontification also relies on correct
+;; `syntax-table' properties. This means that scanning for these
;; constructs had better be quick. Fortunately it is. Firstly, the
;; typical proportion of comments, CDATA sections and processing
;; instructions is small relative to other things. Secondly, to scan
@@ -79,7 +69,13 @@ (defvar-local nxml-prolog-end nil
"Integer giving position following end of the prolog.")
(defsubst nxml-get-inside (pos)
- (save-excursion (nth 8 (syntax-ppss pos))))
+ "Return non-nil if inside comment, CDATA, or PI."
+ (let ((ppss (save-excursion (syntax-ppss pos))))
+ (or
+ ;; Inside comment.
+ (nth 4 ppss)
+ ;; Inside "generic" string, happens for CDATA, and PI.
+ (eq t (nth 3 ppss)))))
(defun nxml-inside-end (pos)
"Return the end of the inside region containing POS.
diff --git a/test/lisp/nxml/nxml-mode-tests.el b/test/lisp/nxml/nxml-mode-tests.el
new file mode 100644
index 0000000000..57a731ad18
--- /dev/null
+++ b/test/lisp/nxml/nxml-mode-tests.el
@@ -0,0 +1,62 @@
+;;; nxml-mode-tests.el --- Test NXML Mode -*- lexical-binding: t; -*-
+
+;; Copyright (C) 2019 Free Software Foundation, Inc.
+
+;; 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 <https://www.gnu.org/licenses/>.
+
+;;; Code:
+
+(require 'ert)
+(require 'nxml-mode)
+
+(defun nxml-mode-tests-correctly-indented-string (str)
+ (with-temp-buffer
+ (nxml-mode)
+ (insert str)
+ (indent-region (point-min) (point-max))
+ (equal (buffer-string) str)))
+
+(ert-deftest nxml-indent-line-after-attribute ()
+ (should (nxml-mode-tests-correctly-indented-string "
+<settings
+ xmlns=\"http://maven.apache.org/SETTINGS/1.0.0\"
+ xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\"
+ xsi:schemaLocation=\"http://maven.apache.org/SETTINGS/1.0.0
+ https://maven.apache.org/xsd/settings-1.0.0.xsd\">
+ <mirrors>
+ ...
+ </mirrors>
+</settings>
+"))
+ (should (nxml-mode-tests-correctly-indented-string "\
+<x>
+ <abc xx=\"x/x/x/x/x/x/x/
+ y/y/y/y/y/y/
+ \">
+ <zzz/>
+ </abc>
+ <nl> </nl>
+</x>
+")))
+
+(ert-deftest nxml-balanced-close-start-tag-inline ()
+ (with-temp-buffer
+ (nxml-mode)
+ (insert "<a><b c=\"\"</a>")
+ (search-backward "</a>")
+ (nxml-balanced-close-start-tag-inline)
+ (should (equal (buffer-string) "<a><b c=\"\"></b></a>"))))
+
+(provide 'nxml-mode-tests)
+;;; nxml-mode-tests.el ends here
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* bug#32003: 26.0.90; nxml wrong indentation after multi-line attribute
2019-04-19 3:48 ` Noam Postavsky
@ 2019-05-09 11:41 ` Noam Postavsky
0 siblings, 0 replies; 3+ messages in thread
From: Noam Postavsky @ 2019-05-09 11:41 UTC (permalink / raw
To: Benjamin Riefenstahl
Cc: Vincent Lefevre, David Carlisle, 32003, Stefan Monnier
tags 32003 fixed
close 32003 26.3
quit
Noam Postavsky <npostavs@gmail.com> writes:
> forcemerge 32003 33873
> tags 32003 + patch
> quit
>
> Benjamin Riefenstahl <Riefenstahl@mecom.de> writes:
>
>> <settings
>> xmlns="http://maven.apache.org/SETTINGS/1.0.0"
>> xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
>> xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.0.0
>> https://maven.apache.org/xsd/settings-1.0.0.xsd">
>> <mirrors>
>> ...
>> </mirrors>
>> </settings>
>
> Here's a patch to fix this bug, Bug#33873, and the indentation part of Bug#32897.
Pushed to emacs-26.
ca14dd1d46 2019-05-09T06:42:40-04:00 "Fix nxml-get-inside (Bug#32003)"
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=ca14dd1d4628094dd33d5d94694dcf5f29e843b8
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-05-09 11:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-06-29 9:38 bug#32003: 26.0.90; nxml wrong indentation after multi-line attribute Benjamin Riefenstahl
2019-04-19 3:48 ` Noam Postavsky
2019-05-09 11:41 ` Noam Postavsky
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.