unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: longlines-auto-wrap: doc string wrong, or bug?
       [not found] <21A24698-B3FE-46F9-AEE7-8A861CC515E3@inf.ed.ac.uk>
@ 2007-05-12 21:52 ` Richard Stallman
  2007-05-12 23:21   ` Chong Yidong
  0 siblings, 1 reply; 3+ messages in thread
From: Richard Stallman @ 2007-05-12 21:52 UTC (permalink / raw)
  To: David Reitter; +Cc: emacs-devel

    It appears that `longlines-auto-wrap' as a variable can't be set  
    directly - at least to no effect. I have to explicitly do (longlines- 
    auto-wrap 0) to turn it off.

I agree that the function `longlines-auto-wrap' should toggle when
given no argument.

The function `longlines-auto-wrap' seems to expect the variable to be
buffer-local, since it wraps the current buffer in addition to setting
the variable.  Should we add a call to `make-variable-buffer-local'
for it?

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: longlines-auto-wrap: doc string wrong, or bug?
  2007-05-12 21:52 ` longlines-auto-wrap: doc string wrong, or bug? Richard Stallman
@ 2007-05-12 23:21   ` Chong Yidong
  2007-05-14  8:09     ` Richard Stallman
  0 siblings, 1 reply; 3+ messages in thread
From: Chong Yidong @ 2007-05-12 23:21 UTC (permalink / raw)
  To: rms; +Cc: David Reitter, emacs-devel

Richard Stallman <rms@gnu.org> writes:

>     It appears that `longlines-auto-wrap' as a variable can't be set  
>     directly - at least to no effect. I have to explicitly do (longlines- 
>     auto-wrap 0) to turn it off.
>
> I agree that the function `longlines-auto-wrap' should toggle when
> given no argument.
>
> The function `longlines-auto-wrap' seems to expect the variable to be
> buffer-local, since it wraps the current buffer in addition to setting
> the variable.  Should we add a call to `make-variable-buffer-local'
> for it?

I've fixed this on the trunk.

(It's probably not important enough to check in to the branch, because
frankly turning off auto-wrap is a silly thing to do; it mostly
defeats the purpose of longlines mode.  It's there primarily due to
legacy reasons.)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: longlines-auto-wrap: doc string wrong, or bug?
  2007-05-12 23:21   ` Chong Yidong
@ 2007-05-14  8:09     ` Richard Stallman
  0 siblings, 0 replies; 3+ messages in thread
From: Richard Stallman @ 2007-05-14  8:09 UTC (permalink / raw)
  To: Chong Yidong; +Cc: dreitter, emacs-devel

    I've fixed this on the trunk.

Thanks.

    (It's probably not important enough to check in to the branch, because
    frankly turning off auto-wrap is a silly thing to do; it mostly
    defeats the purpose of longlines mode.  It's there primarily due to
    legacy reasons.)

Ok.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-05-14  8:09 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <21A24698-B3FE-46F9-AEE7-8A861CC515E3@inf.ed.ac.uk>
2007-05-12 21:52 ` longlines-auto-wrap: doc string wrong, or bug? Richard Stallman
2007-05-12 23:21   ` Chong Yidong
2007-05-14  8:09     ` Richard Stallman

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).