Re: nxml-mode: consider changing nxml-sexp-element-flag default

["Jostein Kjønigsen" <jostein@secure.kjonigsen.net>]

[-- Attachment #1.1: Type: text/plain, Size: 604 bytes --]

On Sat, Mar 17, 2018, at 1:52 PM, Steinar Bang wrote:
>> Probably for fear of irking users used to the old behavior.
>> FWIW I'm mildly in favor of changing the default,
> 
> I'm a longtime nxml user, and I'm in favour of changing the default.

In case there's no objections and we decide to go through with this,
I've change the default and run "make check".
It seems no tests are impacted, and the patch is therefore pretty small.
Is there anything more we'll need to do?
--
Regards
Jostein Kjønigsen

jostein@kjonigsen.net 🍵 jostein@gmail.com
https://jostein.kjonigsen.net





[-- Attachment #1.2: Type: text/html, Size: 1513 bytes --]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: nxml-mode-defaults.patch --]
[-- Type: text/x-patch; name="nxml-mode-defaults.patch", Size: 771 bytes --]

From f3c7a19423d2a01c6e25dd017884b78c09a50d66 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jostein=20Kj=C3=B8nigsen?= <jostein@kjonigsen.net>
Date: Mon, 19 Mar 2018 21:48:33 +0100
Subject: [PATCH] nxml-mode: change nxml-sexp-element-flag default to t

---
 lisp/nxml/nxml-mode.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/nxml/nxml-mode.el b/lisp/nxml/nxml-mode.el
index 1df410e505..554b5acfb2 100644
--- a/lisp/nxml/nxml-mode.el
+++ b/lisp/nxml/nxml-mode.el
@@ -56,7 +56,7 @@ nxml-char-ref-display-glyph-flag
   :group 'nxml
   :type 'boolean)
 
-(defcustom nxml-sexp-element-flag nil
+(defcustom nxml-sexp-element-flag t
   "Non-nil means sexp commands treat an element as a single expression."
   :group 'nxml
   :type 'boolean)
-- 
2.14.1