GNU bug report logs - #22874
25.0.91; whitespace.el documentation looks incorrect

Previous Next

Package: emacs;

Reported by: David Caldwell <david <at> porkrind.org>

Date: Tue, 1 Mar 2016 19:06:01 UTC

Severity: minor

Tags: moreinfo, notabug

Found in version 25.0.91

Done: Bhavin Gandhi <bhavin7392 <at> gmail.com>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 22874 in the body.
You can then email your comments to 22874 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#22874; Package emacs. (Tue, 01 Mar 2016 19:06:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to David Caldwell <david <at> porkrind.org>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Tue, 01 Mar 2016 19:06:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: David Caldwell <david <at> porkrind.org>
To: bug-gnu-emacs <at> gnu.org
Subject: 25.0.91; whitespace.el documentation looks incorrect
Date: Tue, 1 Mar 2016 11:04:37 -0800
[Message part 1 (text/plain, inline)]
Hello,

M-x describe-variable RET whitespace-style RET

says:

[snip]

   indentation::tab     8 or more SPACEs at beginning of line are
                        visualized via faces.
                        It has effect only if ‘face’ (see above)
                        is present in ‘whitespace-style’.

   indentation::space   TABs at beginning of line are visualized via
                        faces.
                        It has effect only if ‘face’ (see above)
                        is present in ‘whitespace-style’.

[snip]

That looks backward to me. Attached is a patch that fixes it.

-David
[whitespace.doc-fix.patch (text/plain, attachment)]
[smime.p7s (application/pkcs7-signature, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#22874; Package emacs. (Mon, 29 Jul 2019 13:35:02 GMT) Full text and rfc822 format available.

Message #8 received at 22874 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: David Caldwell <david <at> porkrind.org>
Cc: 22874 <at> debbugs.gnu.org
Subject: Re: bug#22874: 25.0.91; whitespace.el documentation looks incorrect
Date: Mon, 29 Jul 2019 15:34:43 +0200
[Message part 1 (text/plain, inline)]
David Caldwell <david <at> porkrind.org> writes:

> Hello,
>
> M-x describe-variable RET whitespace-style RET
>
> says:
>
> [snip]
>
>    indentation::tab     8 or more SPACEs at beginning of line are
>                         visualized via faces.
>                         It has effect only if ‘face’ (see above)
>                         is present in ‘whitespace-style’.
>
>    indentation::space   TABs at beginning of line are visualized via
>                         faces.
>                         It has effect only if ‘face’ (see above)
>                         is present in ‘whitespace-style’.

I tried to reproduce this bug with the following file:

[white.bin (application/octet-stream, attachment)]
[Message part 3 (text/plain, inline)]
But with either

(setq whitespace-style '(face indentation::tab))
(setq whitespace-style '(face indentation::space))

and then `M-x whitespace-mode'

just the lines with leading spaces were fontified.

So this either doesn't seem to work at all, or I'm just doing it wrong.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#22874; Package emacs. (Sat, 15 Aug 2020 04:34:02 GMT) Full text and rfc822 format available.

Message #11 received at 22874 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Kangas <stefan <at> marxist.se>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 22874 <at> debbugs.gnu.org, David Caldwell <david <at> porkrind.org>
Subject: Re: bug#22874: 25.0.91; whitespace.el documentation looks incorrect
Date: Fri, 14 Aug 2020 21:32:55 -0700
tags 22874 + moreinfo
thanks

Hi David,

Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> David Caldwell <david <at> porkrind.org> writes:
>
>> Hello,
>>
>> M-x describe-variable RET whitespace-style RET
>>
>> says:
>>
>> [snip]
>>
>>    indentation::tab     8 or more SPACEs at beginning of line are
>>                         visualized via faces.
>>                         It has effect only if ‘face’ (see above)
>>                         is present in ‘whitespace-style’.
>>
>>    indentation::space   TABs at beginning of line are visualized via
>>                         faces.
>>                         It has effect only if ‘face’ (see above)
>>                         is present in ‘whitespace-style’.
>
> I tried to reproduce this bug with the following file:
>
>
>
> But with either
>
> (setq whitespace-style '(face indentation::tab))
> (setq whitespace-style '(face indentation::space))
>
> and then `M-x whitespace-mode'
>
> just the lines with leading spaces were fontified.
>
> So this either doesn't seem to work at all, or I'm just doing it wrong.

We seem to be needing more info to make any progress here.  Could you
take a look at the above and explain the issue you are seeing?

If I don't hear back from you within a couple of weeks, I'll just
close this bug as unreproducible.

Best regards,
Stefan Kangas




Added tag(s) moreinfo. Request was from Stefan Kangas <stefan <at> marxist.se> to control <at> debbugs.gnu.org. (Sat, 15 Aug 2020 04:34:02 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#22874; Package emacs. (Sat, 15 Aug 2020 18:41:02 GMT) Full text and rfc822 format available.

Message #16 received at 22874 <at> debbugs.gnu.org (full text, mbox):

From: David Caldwell <david <at> porkrind.org>
To: Stefan Kangas <stefan <at> marxist.se>, Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 22874 <at> debbugs.gnu.org
Subject: Re: bug#22874: 25.0.91; whitespace.el documentation looks incorrect
Date: Sat, 15 Aug 2020 11:40:13 -0700
Hi,

On 8/14/20 9:32 PM, Stefan Kangas wrote:
> tags 22874 + moreinfo
> thanks
> 
> Hi David,
> 
> Lars Ingebrigtsen <larsi <at> gnus.org> writes:
> 
>> David Caldwell <david <at> porkrind.org> writes:
>>
>>> Hello,
>>>
>>> M-x describe-variable RET whitespace-style RET
>>>
>>> says:
>>>
>>> [snip]
>>>
>>>    indentation::tab     8 or more SPACEs at beginning of line are
>>>                         visualized via faces.
>>>                         It has effect only if ‘face’ (see above)
>>>                         is present in ‘whitespace-style’.
>>>
>>>    indentation::space   TABs at beginning of line are visualized via
>>>                         faces.
>>>                         It has effect only if ‘face’ (see above)
>>>                         is present in ‘whitespace-style’.
>>
>> I tried to reproduce this bug with the following file:
>>
>> But with either
>>
>> (setq whitespace-style '(face indentation::tab))
>> (setq whitespace-style '(face indentation::space))
>>
>> and then `M-x whitespace-mode'
>>
>> just the lines with leading spaces were fontified.
>>
>> So this either doesn't seem to work at all, or I'm just doing it wrong.
>
> We seem to be needing more info to make any progress here.  Could you
> take a look at the above and explain the issue you are seeing?

I just tested it in Emacs 27.1 and they both seem to work exactly as
documented.

I was initially confused because why would setting `indentation::tab`
show spaces at the beginning of a line and why would
`indentation::space` show tabs? I thought they just got switched around
in the docs, but I guess maybe it's supposed to act like an assertion?
`indentation::space` means "I expect my file to be indented with spaces
so please highlight any stray tab indents", and `indentation::tab` means
"I expect my file to be indented with tabs so highlight any `tab-width`
long runs of spaces".

Bottom line: I'm wrong, my patch is wrong, and the docs *are* correct.
Sorry to waste your time. :-)

-David




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#22874; Package emacs. (Sat, 15 Aug 2020 19:54:02 GMT) Full text and rfc822 format available.

Message #19 received at 22874 <at> debbugs.gnu.org (full text, mbox):

From: Bhavin Gandhi <bhavin7392 <at> gmail.com>
To: David Caldwell <david <at> porkrind.org>
Cc: Lars Ingebrigtsen <larsi <at> gnus.org>, Stefan Kangas <stefan <at> marxist.se>,
 22874 <at> debbugs.gnu.org
Subject: Re: bug#22874: 25.0.91; whitespace.el documentation looks incorrect
Date: Sun, 16 Aug 2020 01:22:26 +0530
tags 22874 + notabug
close 22874
thanks

On Sun, 16 Aug 2020 at 00:11, David Caldwell <david <at> porkrind.org> wrote:
> >
> > We seem to be needing more info to make any progress here.  Could you
> > take a look at the above and explain the issue you are seeing?
>
> I just tested it in Emacs 27.1 and they both seem to work exactly as
> documented.

Thank you, closing this.




Added tag(s) notabug. Request was from Bhavin Gandhi <bhavin7392 <at> gmail.com> to control <at> debbugs.gnu.org. (Sun, 16 Aug 2020 05:56:02 GMT) Full text and rfc822 format available.

bug closed, send any further explanations to 22874 <at> debbugs.gnu.org and David Caldwell <david <at> porkrind.org> Request was from Bhavin Gandhi <bhavin7392 <at> gmail.com> to control <at> debbugs.gnu.org. (Sun, 16 Aug 2020 05:56:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sun, 13 Sep 2020 11:24:06 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 196 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.