[O] [PATCH] org-transpose-word: inherit text mode syntax table?

2014-11-03 Thread Eric Abrahamsen
Something that has subconsciously bothered me for years:
`org-transpose-words' does its transposition using a blank syntax table,
rather than inheriting from text-mode-syntax-table, meaning we don't get
any of the default text-mode transposition behavior. The most immediate
problem is that apostrophes get left behind during transposition. Ie:

I can't| transpose this -- I can transpose|'t this

and not

I can't| transpose this -- I transpose can't| this

The fix would be as simple as the attached patch. I don't know if this
would break anything elsewhere, but I can't imagine it would...

Eric
From a11bb44a6ab200c4159b03094948bc9b77c0e983 Mon Sep 17 00:00:00 2001
From: Eric Abrahamsen e...@ericabrahamsen.net
Date: Mon, 3 Nov 2014 11:22:13 -0800
Subject: [PATCH] Make org-transpose-words use text-mode-syntax-table

* lisp/org.el (org-mode-transpose-word-syntax-table): Make this table
  inherit from `text-mode-syntax-table'.
---
 lisp/org.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lisp/org.el b/lisp/org.el
index 1e7d8d9..c6b0a1c 100755
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -5542,7 +5542,7 @@ The following commands are available:
 		   (8.2.6 . 24.4)))
 
 (defvar org-mode-transpose-word-syntax-table
-  (let ((st (make-syntax-table)))
+  (let ((st (make-syntax-table text-mode-syntax-table)))
 (mapc (lambda(c) (modify-syntax-entry
 		  (string-to-char (car c)) w p st))
 	  org-emphasis-alist)
-- 
2.1.3



Re: [O] [PATCH] org-transpose-word: inherit text mode syntax table?

2014-11-03 Thread Nicolas Goaziou
Hello,

Eric Abrahamsen e...@ericabrahamsen.net writes:

 Something that has subconsciously bothered me for years:
 `org-transpose-words' does its transposition using a blank syntax table,
 rather than inheriting from text-mode-syntax-table, meaning we don't get
 any of the default text-mode transposition behavior. The most immediate
 problem is that apostrophes get left behind during transposition. Ie:

 I can't| transpose this -- I can transpose|'t this

 and not

 I can't| transpose this -- I transpose can't| this

 The fix would be as simple as the attached patch. I don't know if this
 would break anything elsewhere, but I can't imagine it would...

Applied. Thank you.


Regards,

-- 
Nicolas Goaziou