emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
From: Eric Schulte <schulte.eric@gmail.com>
To: Nicolas Goaziou <n.goaziou@gmail.com>
Cc: Org Mode List <emacs-orgmode@gnu.org>
Subject: Re: [ANN] Improved Flyspell check
Date: Wed, 06 Nov 2013 11:00:00 -0700	[thread overview]
Message-ID: <87li111kdb.fsf@gmail.com> (raw)
In-Reply-To: <87habt4b3r.fsf@gmail.com> (Nicolas Goaziou's message of "Sun, 03 Nov 2013 19:02:48 +0100")

I've been using this since it was sent, and I haven't noticed any bad
behavior.

Cheers,

Nicolas Goaziou <n.goaziou@gmail.com> writes:

> Nicolas Goaziou <n.goaziou@gmail.com> writes:
>
> Updated patch.
>
>
> -- 
> Nicolas Goaziou
>
> From 13f68648a0ff468385668f6b141748607d3f8103 Mon Sep 17 00:00:00 2001
> From: Nicolas Goaziou <n.goaziou@gmail.com>
> Date: Sun, 3 Nov 2013 15:15:07 +0100
> Subject: [PATCH] Improve Flyspell check
>
> * lisp/org.el (org-mode-flyspell-verify): Rewrite function using
>   "org-element.el".  In particular, it doesn't rely on fontification
>   anymore.
> (org-remove-flyspell-overlays-in): Remove function.
> (org-do-emphasis-faces, org-activate-plain-links)
> (org-fontify-meta-lines-and-blocks-1, org-activate-footnote-links)
> (org-activate-target-links, org-activate-tags, org-activate-code)
> (org-activate-angle-links): Don't call
> `org-remove-flyspell-overlays-in'.
> * contrib/lisp/org-wikinodes.el (org-wikinodes-activate-links): Don't
>   call `org-remove-flyspell-overlays-in'.
> ---
>  contrib/lisp/org-wikinodes.el |   5 --
>  lisp/org.el                   | 109 ++++++++++++++++++++++++++----------------
>  2 files changed, 68 insertions(+), 46 deletions(-)
>
> diff --git a/contrib/lisp/org-wikinodes.el b/contrib/lisp/org-wikinodes.el
> index 4efc373..c6f2006 100644
> --- a/contrib/lisp/org-wikinodes.el
> +++ b/contrib/lisp/org-wikinodes.el
> @@ -79,15 +79,10 @@ to `directory'."
>        (if (re-search-forward org-wikinodes-camel-regexp limit t)
>  	  (if (equal (char-after (point-at-bol)) ?*)
>  	      (progn
> -		;; in  heading - deactivate flyspell
> -		(org-remove-flyspell-overlays-in (match-beginning 0)
> -						 (match-end 0))
>  		(add-text-properties (match-beginning 0) (match-end 0)
>  				     '(org-no-flyspell t))
>  		t)
>  	    ;; this is a wiki link
> -	    (org-remove-flyspell-overlays-in (match-beginning 0)
> -					     (match-end 0))
>  	    (add-text-properties (match-beginning 0) (match-end 0)
>  				 (list 'mouse-face 'highlight
>  				       'face 'org-link
> diff --git a/lisp/org.el b/lisp/org.el
> index 2382a9c..452babb 100644
> --- a/lisp/org.el
> +++ b/lisp/org.el
> @@ -5459,8 +5459,6 @@ The following commands are available:
>    (abbrev-table-put org-mode-abbrev-table
>  		    :parents (list text-mode-abbrev-table)))
>  
> -(put 'org-mode 'flyspell-mode-predicate 'org-mode-flyspell-verify)
> -
>  (defsubst org-fix-ellipsis-at-bol ()
>    (save-excursion (goto-char (window-start)) (recenter 0)))
>  
> @@ -5685,9 +5683,6 @@ The time stamps may be either active or inactive.")
>  	    (font-lock-prepend-text-property (match-beginning 2) (match-end 2)
>  					     'face
>  					     (nth 1 a))
> -	    (and (nth 2 a)
> -		 (org-remove-flyspell-overlays-in
> -		  (match-beginning 0) (match-end 0)))
>  	    (add-text-properties (match-beginning 2) (match-end 2)
>  				 '(font-lock-multiline t org-emphasis t))
>  	    (when org-hide-emphasis-markers
> @@ -5753,7 +5748,6 @@ prompted for."
>    (let (f hl)
>      (when (and (re-search-forward (concat org-plain-link-re) limit t)
>  	       (not (org-in-src-block-p)))
> -      (org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>        (setq f (get-text-property (match-beginning 0) 'face))
>        (setq hl (org-match-string-no-properties 0))
>        (if (or (eq f 'org-tag)
> @@ -5770,7 +5764,6 @@ prompted for."
>  (defun org-activate-code (limit)
>    (if (re-search-forward "^[ \t]*\\(:\\(?: .*\\|$\\)\n?\\)" limit t)
>        (progn
> -	(org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>  	(remove-text-properties (match-beginning 0) (match-end 0)
>  				'(display t invisible t intangible t))
>  	t)))
> @@ -5812,7 +5805,6 @@ by a #."
>  	  (cond
>  	   ((member dc1 '("+html:" "+ascii:" "+latex:"))
>  	    ;; a single line of backend-specific content
> -	    (org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>  	    (remove-text-properties (match-beginning 0) (match-end 0)
>  				    '(display t invisible t intangible t))
>  	    (add-text-properties (match-beginning 1) (match-end 3)
> @@ -5901,7 +5893,6 @@ by a #."
>    (if (and (re-search-forward org-angle-link-re limit t)
>  	   (not (org-in-src-block-p)))
>        (progn
> -	(org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>  	(add-text-properties (match-beginning 0) (match-end 0)
>  			     (list 'mouse-face 'highlight
>  				   'keymap org-mouse-map))
> @@ -5913,7 +5904,6 @@ by a #."
>    (let ((fn (org-footnote-next-reference-or-definition limit)))
>      (when fn
>        (let ((beg (nth 1 fn)) (end (nth 2 fn)))
> -	(org-remove-flyspell-overlays-in beg end)
>  	(add-text-properties beg end
>  			     (list 'mouse-face 'highlight
>  				   'keymap org-mouse-map
> @@ -5941,7 +5931,6 @@ by a #."
>  		       'htmlize-link `(:uri ,hl))))
>  	;; We need to remove the invisible property here.  Table narrowing
>  	;; may have made some of this invisible.
> -	(org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>  	(remove-text-properties (match-beginning 0) (match-end 0)
>  				'(invisible nil))
>  	(if (match-end 3)
> @@ -5965,7 +5954,6 @@ by a #."
>    (if (and (re-search-forward org-tsr-regexp-both limit t)
>  	   (not (equal (char-before (match-beginning 0)) 91)))
>        (progn
> -	(org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>  	(add-text-properties (match-beginning 0) (match-end 0)
>  			     (list 'mouse-face 'highlight
>  				   'keymap org-mouse-map))
> @@ -5992,7 +5980,6 @@ by a #."
>      (let ((case-fold-search t))
>        (if (re-search-forward org-target-link-regexp limit t)
>  	  (progn
> -	    (org-remove-flyspell-overlays-in (match-beginning 0) (match-end 0))
>  	    (add-text-properties (match-beginning 0) (match-end 0)
>  				 (list 'mouse-face 'highlight
>  				       'keymap org-mouse-map
> @@ -6110,7 +6097,6 @@ between words."
>  (defun org-activate-tags (limit)
>    (if (re-search-forward (org-re "^\\*+.*[ \t]\\(:[[:alnum:]_@#%:]+:\\)[ \r\n]") limit t)
>        (progn
> -	(org-remove-flyspell-overlays-in (match-beginning 1) (match-end 1))
>  	(add-text-properties (match-beginning 1) (match-end 1)
>  			     (list 'mouse-face 'highlight
>  				   'keymap org-mouse-map))
> @@ -23861,34 +23847,75 @@ To get rid of the restriction, use \\[org-agenda-remove-restriction-lock]."
>  
>  ;;; Fixes and Hacks for problems with other packages
>  
> -;; Make flyspell not check words in links, to not mess up our keymap
> -(defvar org-element-affiliated-keywords) ; From org-element.el
> -(defvar org-element-block-name-alist)	 ; From org-element.el
>  (defun org-mode-flyspell-verify ()
> -  "Don't let flyspell put overlays at active buttons, or on
> -   {todo,all-time,additional-option-like}-keywords."
> -  (require 'org-element) ; For `org-element-affiliated-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-options-keywords))
> -	 (not (member word (mapcar 'car org-startup-options)))
> -	 (not (member-ignore-case word org-element-affiliated-keywords))
> -	 (not (member-ignore-case word (org-get-export-keywords)))
> -	 (not (member-ignore-case
> -	       word (mapcar 'car org-element-block-name-alist)))
> -	 (not (member-ignore-case word '("BEGIN" "END" "ATTR")))
> -	 (not (org-in-src-block-p)))))
> -
> -(defun org-remove-flyspell-overlays-in (beg end)
> -  "Remove flyspell overlays in region."
> -  (and (org-bound-and-true-p flyspell-mode)
> -       (fboundp 'flyspell-delete-region-overlays)
> -       (flyspell-delete-region-overlays beg end))
> -  (add-text-properties beg end '(org-no-flyspell t)))
> +  "Function used for `flyspell-generic-check-word-predicate'."
> +  (if (org-at-heading-p)
> +      ;; At a headline or an inlinetask, check title only.  This is
> +      ;; faster than relying on `org-element-at-point'.
> +      (and (save-excursion (beginning-of-line)
> +			   (and (let ((case-fold-search t))
> +				  (not (looking-at "\\*+ END[ \t]*$")))
> +				(looking-at org-complex-heading-regexp)))
> +	   (match-beginning 4)
> +	   (>= (point) (match-beginning 4))
> +	   (or (not (match-beginning 5))
> +	       (< (point) (match-beginning 5))))
> +    (let* ((element (org-element-at-point))
> +	   (post-affiliated (org-element-property :post-affiliated element))
> +	   (object-check
> +	    (function
> +	     ;; Non-nil if checks can be done for object at point.
> +	     (lambda ()
> +	       (let ((object (save-excursion
> +			       (when (org-looking-at-p "\\>") (backward-char))
> +			       (org-element-context element))))
> +		 (case (org-element-type object)
> +		   ;; Prevent checks in links due to keybinding conflict
> +		   ;; with Flyspell.
> +		   ((code entity export-snippet inline-babel-call
> +			  inline-src-block line-break latex-fragment link macro
> +			  statistics-cookie target timestamp verbatim)
> +		    nil)
> +		   (footnote-reference
> +		    ;; Only in inline footnotes, within the definition.
> +		    (and (eq (org-element-property :type object) 'inline)
> +			 (< (save-excursion
> +			      (goto-char (org-element-property :begin object))
> +			      (search-forward ":" nil t 2))
> +			    (point))))
> +		   (otherwise t)))))))
> +      (if (and post-affiliated (< (point) post-affiliated))
> +	  (and (save-excursion
> +		 (beginning-of-line)
> +		 (let ((case-fold-search t)) (looking-at "[ \t]*#\\+CAPTION:")))
> +	       (> (point) (match-end 0))
> +	       (funcall object-check))
> +	(case (org-element-type element)
> +	  ((comment quote-section) t)
> +	  (comment-block
> +	   ;; Allow checks between block markers, not on them.
> +	   (and (> (line-beginning-position)
> +		   (org-element-property :post-affiliated element))
> +		(save-excursion
> +		  (end-of-line)
> +		  (skip-chars-forward " \r\t\n")
> +		  (< (point) (org-element-property :end element)))))
> +	  ;; Arbitrary list of keywords where checks are meaningful.
> +	  ;; Make sure point is on the value part of the element.
> +	  (keyword
> +	   (and (member (org-element-property :key element)
> +			'("DESCRIPTION" "TITLE"))
> +		(< (save-excursion
> +		     (beginning-of-line) (search-forward ":") (point))
> +		   (point))))
> +	  ;; Check is globally allowed in paragraphs verse blocks and
> +	  ;; table rows (after affiliated keywords) but some objects
> +	  ;; must not be affected.
> +	  ((paragraph table-row verse-block)
> +	   (and (>= (point) (org-element-property :contents-begin element))
> +		(< (point) (org-element-property :contents-end element))
> +		(funcall object-check))))))))
> +(put 'org-mode 'flyspell-mode-predicate 'org-mode-flyspell-verify)
>  
>  ;; Make `bookmark-jump' shows the jump location if it was hidden.
>  (eval-after-load "bookmark"

-- 
Eric Schulte
https://cs.unm.edu/~eschulte
PGP: 0x614CA05D

  reply	other threads:[~2013-11-06 18:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-03 14:46 [ANN] Improved Flyspell check Nicolas Goaziou
2013-11-03 18:02 ` Nicolas Goaziou
2013-11-06 18:00   ` Eric Schulte [this message]
2013-11-06 22:06     ` Nicolas Goaziou
2013-11-08 20:29       ` Sebastien Vauban
2013-11-08 21:48         ` Nicolas Goaziou
2013-11-08 22:43           ` Sebastien Vauban
2013-11-08 23:22             ` Nicolas Goaziou
     [not found]               ` <87k3gi8omo.fsf-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-11-15 14:03                 ` Sebastien Vauban
2013-11-11  2:41       ` Eric Schulte
2013-11-24 16:44         ` Nicolas Goaziou
     [not found]   ` <87eh69e2ha.fsf@pank.eu>
2013-11-22  0:25     ` Rasmus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.orgmode.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87li111kdb.fsf@gmail.com \
    --to=schulte.eric@gmail.com \
    --cc=emacs-orgmode@gnu.org \
    --cc=n.goaziou@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).