unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Arik Mitschang <arik.mitschang@gmail.com>
To: Glenn Morris <rgm@gnu.org>
Cc: 6902@debbugs.gnu.org
Subject: bug#6902: 23.2; indent-line-to possible devide by zero error bug
Date: Wed, 25 Aug 2010 16:04:45 -0400	[thread overview]
Message-ID: <19573.30557.896201.265597@gargle.gargle.HOWL> (raw)
In-Reply-To: <erlj7u7ajw.fsf@fencepost.gnu.org>

Glenn Morris writes:
 > Arik Mitschang wrote:
 > 
 > > In indent.el the indent-line-to function has a comparison that devides
 > > the current column count by tab-width, whose value is an integerp and
 > > allowed to be zero (and in my experience is often 0).
 > 
 > Why do you want to set tab-width to 0?

Honestly I cannot remember, but I had it set there and I assume there
was a reason.

 > 
 > A quick grep for tab-width shows other places that will break.

The fact that it's value can be zero (without even warning in the doc
string) and this value causes functionality to break doesn't seem good
to me. If nobody but me ever had it set to zero, then perhaps just
updating its doc-string to warn users away from that value would be
nice.

Thanks,
~Arik





  reply	other threads:[~2010-08-25 20:04 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-24 15:27 bug#6902: 23.2; indent-line-to possible devide by zero error bug Arik Mitschang
2010-08-25 19:32 ` Glenn Morris
2010-08-25 20:04   ` Arik Mitschang [this message]
2011-04-10 20:56     ` Chong Yidong

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.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=19573.30557.896201.265597@gargle.gargle.HOWL \
    --to=arik.mitschang@gmail.com \
    --cc=6902@debbugs.gnu.org \
    --cc=rgm@gnu.org \
    /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.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).