[patch] Do not run flyspell checks on org keywords

[patch] Do not run flyspell checks on org keywords

[Sébastien Delafond]

Here is a small patch that prevents flyspelle from checking all the Org
keywords. This is especially useful when using a dictionary other than
english.

Cheers,

--Seb

From 0a6493331f1d84a4c44533bdea1c3c9b41c997eb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Delafond?= <sdelafond@gmail.com>
Date: Mon, 23 May 2011 18:16:57 +0200
Subject: [PATCH] [flyspell] Do not run flyspell-checks on {todo,all-time,additional-option-like}-keywords

  * this is useful when using flyspell with a non-English dictionary,
    or if some of your keywords are in another language
---
 lisp/org.el |   12 +++++++++---
 1 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/lisp/org.el b/lisp/org.el
index 9f8fa92..676f08f 100644
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -20135,9 +20135,15 @@ To get rid of the restriction, use \\[org-agenda-remove-restriction-lock]."
 
 ;; Make flyspell not check words in links, to not mess up our keymap
 (defun org-mode-flyspell-verify ()
-  "Don't let flyspell put overlays at active buttons."
-  (and (not (get-text-property (max (1- (point)) (point-min)) 'keymap))
-       (not (get-text-property (max (1- (point)) (point-min)) 'org-no-flyspell))))
+  "Don't let flyspell put overlays at active buttons, or on
+   {todo,all-time,additional-option-like}-keywords."
+  (let ((pos (max (1- (point)) (point-min)))
+	(word (thing-at-point 'word)))
+    (and (not (get-text-property pos 'keymap))
+	 (not (get-text-property pos 'org-no-flyspell))
+	 (not (member word org-todo-keywords-1))
+	 (not (member word org-all-time-keywords))
+	 (not (member word org-additional-option-like-keywords)))))
 
 (defun org-remove-flyspell-overlays-in (beg end)
   "Remove flyspell overlays in region."
-- 
1.7.2.5

Re: [patch] Do not run flyspell checks on org keywords

[Sébastien Delafond]

No one interested in this one ? :)

Cheers,

--Seb

On 2011-05-24, Sébastien Delafond <sdelafond@gmail.com> wrote:
>
> Here is a small patch that prevents flyspelle from checking all the Org
> keywords. This is especially useful when using a dictionary other than
> english.
>
> Cheers,
>
> --Seb
>
> From 0a6493331f1d84a4c44533bdea1c3c9b41c997eb Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?S=C3=A9bastien=20Delafond?= <sdelafond@gmail.com>
> Date: Mon, 23 May 2011 18:16:57 +0200
> Subject: [PATCH] [flyspell] Do not run flyspell-checks on {todo,all-time,additional-option-like}-keywords
>
>   * this is useful when using flyspell with a non-English dictionary,
>     or if some of your keywords are in another language
> ---
>  lisp/org.el |   12 +++++++++---
>  1 files changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/lisp/org.el b/lisp/org.el
> index 9f8fa92..676f08f 100644
> --- a/lisp/org.el
> +++ b/lisp/org.el
> @@ -20135,9 +20135,15 @@ To get rid of the restriction, use \\[org-agenda-remove-restriction-lock]."
>  
>  ;; Make flyspell not check words in links, to not mess up our keymap
>  (defun org-mode-flyspell-verify ()
> -  "Don't let flyspell put overlays at active buttons."
> -  (and (not (get-text-property (max (1- (point)) (point-min)) 'keymap))
> -       (not (get-text-property (max (1- (point)) (point-min)) 'org-no-flyspell))))
> +  "Don't let flyspell put overlays at active buttons, or on
> +   {todo,all-time,additional-option-like}-keywords."
> +  (let ((pos (max (1- (point)) (point-min)))
> +	(word (thing-at-point 'word)))
> +    (and (not (get-text-property pos 'keymap))
> +	 (not (get-text-property pos 'org-no-flyspell))
> +	 (not (member word org-todo-keywords-1))
> +	 (not (member word org-all-time-keywords))
> +	 (not (member word org-additional-option-like-keywords)))))
>  
>  (defun org-remove-flyspell-overlays-in (beg end)
>    "Remove flyspell overlays in region."

Re: [patch] Do not run flyspell checks on org keywords

[Christian Moe]

I would be interested, if I could could get ispell.el to work with 
hunspell in a UTF-8 environment (it's the 21st century), or even in 
Emacs' Slovenian language environment (Latin-2), without breaking 
words for every non-ascii character it encounters...

This is off-topic (sorry), but if anyone has a working setup for 
something like the above, I'd be grateful if you'd share it with me 
off-list.

I'm using GNU Emacs 23 and hunspell under Mac OS X. Hunspell seems to 
work perfectly well from the command line with a UTF-8 text file, even 
when I'm using it with a Latin-2-encoded dictionary.

Yours,
Christian


On 5/31/11 3:14 PM, Sébastien Delafond wrote:
> No one interested in this one ? :)
>
> Cheers,
>
> --Seb
>
> On 2011-05-24, Sébastien Delafond<sdelafond@gmail.com>  wrote:
>>
>> Here is a small patch that prevents flyspelle from checking all the Org
>> keywords. This is especially useful when using a dictionary other than
>> english.
>>
>> Cheers,
>>
>> --Seb
>>
>>  From 0a6493331f1d84a4c44533bdea1c3c9b41c997eb Mon Sep 17 00:00:00 2001
>> From: =?UTF-8?q?S=C3=A9bastien=20Delafond?=<sdelafond@gmail.com>
>> Date: Mon, 23 May 2011 18:16:57 +0200
>> Subject: [PATCH] [flyspell] Do not run flyspell-checks on {todo,all-time,additional-option-like}-keywords
>>
>>    * this is useful when using flyspell with a non-English dictionary,
>>      or if some of your keywords are in another language
>> ---
>>   lisp/org.el |   12 +++++++++---
>>   1 files changed, 9 insertions(+), 3 deletions(-)
>>
>> diff --git a/lisp/org.el b/lisp/org.el
>> index 9f8fa92..676f08f 100644
>> --- a/lisp/org.el
>> +++ b/lisp/org.el
>> @@ -20135,9 +20135,15 @@ To get rid of the restriction, use \\[org-agenda-remove-restriction-lock]."
>>
>>   ;; Make flyspell not check words in links, to not mess up our keymap
>>   (defun org-mode-flyspell-verify ()
>> -  "Don't let flyspell put overlays at active buttons."
>> -  (and (not (get-text-property (max (1- (point)) (point-min)) 'keymap))
>> -       (not (get-text-property (max (1- (point)) (point-min)) 'org-no-flyspell))))
>> +  "Don't let flyspell put overlays at active buttons, or on
>> +   {todo,all-time,additional-option-like}-keywords."
>> +  (let ((pos (max (1- (point)) (point-min)))
>> +	(word (thing-at-point 'word)))
>> +    (and (not (get-text-property pos 'keymap))
>> +	 (not (get-text-property pos 'org-no-flyspell))
>> +	 (not (member word org-todo-keywords-1))
>> +	 (not (member word org-all-time-keywords))
>> +	 (not (member word org-additional-option-like-keywords)))))
>>
>>   (defun org-remove-flyspell-overlays-in (beg end)
>>     "Remove flyspell overlays in region."
>
>
>

Re: [patch] Do not run flyspell checks on org keywords

[Julien Danjou]

[-- Attachment #1: Type: text/plain, Size: 276 bytes --]

On Tue, May 24 2011, Sébastien Delafond wrote:

> Here is a small patch that prevents flyspelle from checking all the Org
> keywords. This is especially useful when using a dictionary other than
> english.

Pushed.

-- 
Julien Danjou
❱ http://julien.danjou.info

[-- Attachment #2: Type: application/pgp-signature, Size: 835 bytes --]