Subject: Re: bug#31713: 26.1.50; Docstring for 'empty' whitespace-style is clearer than manual

From: "Basil L. Contovounesios" <[email-masked]>
Date: Mon, 04 Jun 2018 18:06:01 +0100

tags 31713 patch
quit

>From 07fd69686c1a8a810d1d4e5a6f09f0785bdfa931 Mon Sep 17 00:00:00 2001
From: "Basil L. Contovounesios" <[email-masked]>
Date: Mon, 4 Jun 2018 17:39:40 +0100
Subject: [PATCH] Improve documentation of 'empty' whitespace-style

* doc/emacs/display.texi (Useless Whitespace): Clarify that the
'empty' whitespace-style option highlights empty lines only at
BOB/EOB, as per the docstring of whitespace-style. (bug#31713)
---
 doc/emacs/display.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/emacs/display.texi b/doc/emacs/display.texi
index c60cf64914..2f5ce80d60 100644
--- a/doc/emacs/display.texi
+++ b/doc/emacs/display.texi
@@ -1253,7 +1253,7 @@ Useless Whitespace
 Highlight newlines.
 
 @item empty
-Highlight empty lines.
+Highlight empty lines at the beginning and/or end of the buffer.
 
 @item big-indent
 @vindex whitespace-big-indent-regexp
-- 
2.17.1

"Basil L. Contovounesios" <[email-masked]> writes:

> The 'empty' whitespace-style option is described under
> "(emacs) Useless Whitespace" as follows:
>
>   ‘empty’
>        Highlight empty lines.
>
> The docstring of whitespace-style, on the other hand, is a bit clearer
> w.r.t. which lines are considered empty:
>
>   empty                empty lines at beginning and/or end of buffer
>                        are visualized via faces.
>                        It has effect only if `face' (see above)
>                        is present in `whitespace-style'.
>
> I think the manual should also mention the BOB/EOB restriction.

I attach a potential amendment.

Thanks,

-- 
Basil