Re: jit-lock-antiblink-grace

[Eli Zaretskii]

> From: João Távora <address@hidden>
> Date: Tue, 26 Nov 2019 02:30:12 +0000
> Cc: Stefan Monnier <address@hidden>, emacs-devel <address@hidden>, 
>       Alan Mackenzie <address@hidden>
> 
> +---
> +** New customizable variable 'jit-lock-antiblink-grace'.
> +When adding strings to source code, this helps avoid
> +\"blinking\", an unwanted oscillation of certain regions between
> +string and non-string fontification.

Escaping quotes in NEWS is not needed/expected.

And this entry doesn't say what is the default value, and doesn't tell
how to get back old behavior.

> +(defcustom jit-lock-antiblink-grace 2
> +  "Idle time after which to refontify due to unterminated strings.
> +If the user creates a temporarily unterminated string up to the
> +end of the current line, that part of the line is fontified after
> +`jit-lock-context-time', but an extended idle \"grace\" period of
> +this many seconds is granted before deciding it is a multi-line
> +string and fontifying the remainder of the buffer accordingly.
> +When adding strings to source code, this helps avoid
> +\"blinking\", an unwanted oscillation of certain regions between
> +string and non-string fontification.  If nil, there is no grace
> +period."

This is too wordy.  If you think this description is necessary, we
probably need to move most of it into the user manual.

> +  :type '(number :tag "seconds")
> +  :group 'jit-lock)

Missing :version.

> +           (when jit-lock--antiblink-grace-timer
> +             ;; Do refontify immediately, adding a small delay.  This
> +             ;; is per Lars' request, and it makes sense because we
                      ^^^^^^^^^^^^^^^^^
Let's leave personae out of the code, okay?

Thanks.