all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: storm@cua.dk (Kim F. Storm)
Cc: "Michaël Cadilhac" <michael@cadilhac.name>, emacs-devel@gnu.org
Subject: Re: Whitespace cleanup, tab-width and religion.
Date: Tue, 02 Jan 2007 10:56:17 +0100	[thread overview]
Message-ID: <m3tzz94uri.fsf@kfs-l.imdomain.dk> (raw)
In-Reply-To: <a4cf0b680701011055j104897a8v6388d2cf9e669112@mail.gmail.com> (Rajesh Vaidheeswarran's message of "Mon\, 1 Jan 2007 13\:55\:51 -0500")

"Rajesh Vaidheeswarran" <rv@gnu.org> writes:

> As the author of whitespace.el, it was my intent to keep tabs at 8
> characters. 

I simply don't follow the logic of whitespace.el forcing tabs at a
fixed value if the value is customizable in Emacs.

If whitespace.el doesn't work correctly with basic Emacs' features,
we need to fix it.

BTW, after the release, we plan to make tabs even more dynamic than
given by tab-width, and in that case, a regexp is most likely
inadequate to detect tabs.  So a fix would probably make w-i-regexp
obsolete.

> To be clear, from whitespace.el's point of view, it is not relevant what the
> tab-width is set at. People set tab-widths to whatever they please on their
> favorite editors. 

favorite editor_s_ ?   Isn't this an Emacs package?

Shouldn't it _primarily_ work seamless with Emacs?

If people need code to interoperate with other editors, they probably
shouldn't change tab-width -- but in the specific case, Michaël actually
need to change tab-width because of interoperability issues!

So why make whitespace.el _enforce_ "standards" which are unnecessary at
best, and disruptive at worst.

>                   However, there is no way to portably move the definition
> of TAB across people and platforms, other than the value of 8 spaces. That
> is why this is a religious argument, and that is also why it is documented
> as such.

I don't see it as religious -- tab-width exists, so it's reality, and
there's no need to turn it into a religious issue.

>> >
>> > Also, putting w-i-regexp in a  Local Variable part produces a warning.
>> > And I don't really want to  force a certain value of this variable but
>> > just the amount of spaces it counts.
>> >
>>
>> Sounds like whitespace-indent-regexp should be marked as a safe local
>> variable.  Unless it is risky somehow ... ??

Do you object that that change?

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk

  parent reply	other threads:[~2007-01-02  9:56 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-20 14:11 Whitespace cleanup, tab-width and religion Michaël Cadilhac
2006-12-20 15:00 ` Kim F. Storm
2006-12-20 15:11   ` Michaël Cadilhac
2006-12-20 16:04     ` Kim F. Storm
2007-01-01 18:55       ` Rajesh Vaidheeswarran
2007-01-01 20:24         ` Michaël Cadilhac
2007-01-02  9:56         ` Kim F. Storm [this message]
2007-01-05  1:55           ` Rajesh Vaidheeswarran

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

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

  git send-email \
    --in-reply-to=m3tzz94uri.fsf@kfs-l.imdomain.dk \
    --to=storm@cua.dk \
    --cc=emacs-devel@gnu.org \
    --cc=michael@cadilhac.name \
    /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 external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.