all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* comment-multi-line docstring
@ 2004-09-25 18:28 Luc Teirlinck
  2004-09-25 19:09 ` Luc Teirlinck
  2004-09-25 19:33 ` Stefan
  0 siblings, 2 replies; 4+ messages in thread
From: Luc Teirlinck @ 2004-09-25 18:28 UTC (permalink / raw)


>From C-h f comment-multi-line:

    *Non-nil means M-x comment-indent-new-line continues comments, with no
     new terminator or starter.
    This is obsolete because you might as well use C-j.

    You can customize this variable.

Is that "This is obsolete" really accurate?  The direct effect on an
interactively used `M-x comment-indent-new-line' might be obsolete,
but is this not used by auto-fill-mode?  If so, should the docstring
not mention this?

Assuming this _really_ is obsolete, then why clutter a Custom buffer
with it?

Sincerely,

Luc.

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

* Re: comment-multi-line docstring
  2004-09-25 18:28 comment-multi-line docstring Luc Teirlinck
@ 2004-09-25 19:09 ` Luc Teirlinck
  2004-09-25 19:33 ` Stefan
  1 sibling, 0 replies; 4+ messages in thread
From: Luc Teirlinck @ 2004-09-25 19:09 UTC (permalink / raw)
  Cc: emacs-devel

>From my previous message:

   From C-h f comment-multi-line:

C-h v comment-multi-line, of course.

Sincerely,

Luc.

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

* Re: comment-multi-line docstring
  2004-09-25 18:28 comment-multi-line docstring Luc Teirlinck
  2004-09-25 19:09 ` Luc Teirlinck
@ 2004-09-25 19:33 ` Stefan
  2004-09-26 22:47   ` Luc Teirlinck
  1 sibling, 1 reply; 4+ messages in thread
From: Stefan @ 2004-09-25 19:33 UTC (permalink / raw)
  Cc: emacs-devel

> From C-h v comment-multi-line:

>     *Non-nil means M-x comment-indent-new-line continues comments, with no
>      new terminator or starter.
>     This is obsolete because you might as well use C-j.

> Is that "This is obsolete" really accurate?

I think it isn't.

IIRC, when I wrote newcomment.el I was pretty concerned about backward
compatibility (I wanted to make sure that the package was going to be
accepted as a *replacement* for the old commenting code, rather than
like cperl-mode and ibuffer), so I preserved everything as much as I could.
This variable was one I didn't understand at the time, so I just did my best
to preserve the behavior (without actually knowing it, so it was basically
preserving the code) and touched it as little as possible.

I've since learned a bit more about this variable (mostly thanks to
bug reports from Martin Stjernholm) which seems to be actually used
by default in CC-mode and which I've since also used in sgml-mode.

So I think the docstring should be rewritten to remove the `obsolete' part
and to better explain what it does.  AFAIk, what it does it change
auto-fill-mode so that it doesn't add a "end+start" pair of comment markers
when breaking a line.


        Stefan

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

* Re: comment-multi-line docstring
  2004-09-25 19:33 ` Stefan
@ 2004-09-26 22:47   ` Luc Teirlinck
  0 siblings, 0 replies; 4+ messages in thread
From: Luc Teirlinck @ 2004-09-26 22:47 UTC (permalink / raw)
  Cc: emacs-devel

Stefan Monnier wrote:

   So I think the docstring should be rewritten to remove the `obsolete' part
   and to better explain what it does.  AFAIk, what it does it change
   auto-fill-mode so that it doesn't add a "end+start" pair of comment markers
   when breaking a line.

What about the following new docstring?  Too bad that \\[...] does not
recognize aliases.  I had to use \\[indent-new-comment-line] to get
C-M-j.  \\[comment-indent-new-line] produced `M-x comment-indent-new-line'.


===File ~/newcomment-diff===================================
*** newcomment.el	07 Sep 2004 09:18:13 -0500	1.79
--- newcomment.el	26 Sep 2004 17:26:27 -0500	
***************
*** 215,222 ****
  
  ;;;###autoload
  (defcustom comment-multi-line nil
!   "*Non-nil means \\[comment-indent-new-line] continues comments, with no new terminator or starter.
! This is obsolete because you might as well use \\[newline-and-indent]."
    :type 'boolean)
  
  (defcustom comment-empty-lines nil
--- 215,227 ----
  
  ;;;###autoload
  (defcustom comment-multi-line nil
!   "*Non-nil means `comment-indent-new-line' continues comments.
! That is, it inserts no new terminator or starter.
! This affects `auto-fill-mode', which is the main reason to
! customize this variable.
! 
! It also affects \\[indent-new-comment-line].  However, if you want this
! behavior for explicit filling, you might as well use \\[newline-and-indent]."
    :type 'boolean)
  
  (defcustom comment-empty-lines nil
============================================================

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

end of thread, other threads:[~2004-09-26 22:47 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-09-25 18:28 comment-multi-line docstring Luc Teirlinck
2004-09-25 19:09 ` Luc Teirlinck
2004-09-25 19:33 ` Stefan
2004-09-26 22:47   ` Luc Teirlinck

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.