unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* comment-column hard-coded and set twice
@ 2008-03-11  8:20 Andreas Röhler
  2008-03-11 16:29 ` Chong Yidong
  0 siblings, 1 reply; 3+ messages in thread
From: Andreas Röhler @ 2008-03-11  8:20 UTC (permalink / raw)
  To: bug-gnu-emacs


newcomment.el introduces comment-column

,----
| (defcustom comment-column 32
|   "Column to indent right-margin comments to.
`----

The same variable is set in lisp-mode.el again

,----
|   (make-local-variable 'comment-column)
|   (setq comment-column 40)
`----

In the result my customization isn't honoured, value
from lisp-mode.el is taken instead.

Looks like a bug for me.

Thanks

Andreas Röhler




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

* Re: comment-column hard-coded and set twice
  2008-03-11  8:20 comment-column hard-coded and set twice Andreas Röhler
@ 2008-03-11 16:29 ` Chong Yidong
  2008-03-11 18:18   ` Andreas Röhler
  0 siblings, 1 reply; 3+ messages in thread
From: Chong Yidong @ 2008-03-11 16:29 UTC (permalink / raw)
  To: bug-gnu-emacs

Andreas Röhler <andreas.roehler@online.de> writes:

> newcomment.el introduces comment-column
>
> ,----
> | (defcustom comment-column 32
> |   "Column to indent right-margin comments to.
> `----
>
> The same variable is set in lisp-mode.el again
>
> ,----
> |   (make-local-variable 'comment-column)
> |   (setq comment-column 40)
> `----
>
> In the result my customization isn't honoured, value
> from lisp-mode.el is taken instead.
>
> Looks like a bug for me.

This appears to be intentional.

Documentation:
Column to indent right-margin comments to.
Each mode may establish a different default value for this variable; you
can set the value for a particular mode using that mode's hook.
Comments might be indented to a different value in order not to go beyond
`comment-fill-column' or in order to align them with surrounding comments.





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

* Re: comment-column hard-coded and set twice
  2008-03-11 16:29 ` Chong Yidong
@ 2008-03-11 18:18   ` Andreas Röhler
  0 siblings, 0 replies; 3+ messages in thread
From: Andreas Röhler @ 2008-03-11 18:18 UTC (permalink / raw)
  To: bug-gnu-emacs

Am Dienstag, 11. März 2008 17:29 schrieb Chong Yidong:
> Andreas Röhler <andreas.roehler@online.de> writes:
> 
> > newcomment.el introduces comment-column
> >
> > ,----
> > | (defcustom comment-column 32
> > |   "Column to indent right-margin comments to.
> > `----
> >
> > The same variable is set in lisp-mode.el again
> >
> > ,----
> > |   (make-local-variable 'comment-column)
> > |   (setq comment-column 40)
> > `----
> >
> > In the result my customization isn't honoured, value
> > from lisp-mode.el is taken instead.
> >
> > Looks like a bug for me.
> 
> This appears to be intentional.
> 
> Documentation:
> Column to indent right-margin comments to.
> Each mode may establish a different default value for this variable; you
> can set the value for a particular mode using that mode's hook.

How will you do this setting? As shown above, the values in this hook are not 
to change from user side as easily as setting with defcustom.

Well, the experienced user writes a setq and that's it.

But let's think at people just looking for an editor too.

Andreas Röhler

> Comments might be indented to a different value in order not to go beyond
> `comment-fill-column' or in order to align them with surrounding comments.
> 
> 
> 
> 




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

end of thread, other threads:[~2008-03-11 18:18 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-03-11  8:20 comment-column hard-coded and set twice Andreas Röhler
2008-03-11 16:29 ` Chong Yidong
2008-03-11 18:18   ` Andreas Röhler

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