* bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list
@ 2009-07-10 11:14 Jan D.
2009-07-11 11:36 ` Alan Mackenzie
0 siblings, 1 reply; 5+ messages in thread
From: Jan D. @ 2009-07-10 11:14 UTC (permalink / raw)
To: emacs-pretest-bug
I customize c-cleanup-list. This is then the full contents of my
.emacs:
(custom-set-variables
;; custom-set-variables was added by Custom.
;; If you edit it by hand, you could mess it up, so be careful.
;; Your init file should contain only one such instance.
;; If there is more than one, they won't work right.
'(c-cleanup-list (quote (scope-operator space-before-funcall))))
(custom-set-faces
;; custom-set-faces was added by Custom.
;; If you edit it by hand, you could mess it up, so be careful.
;; Your init file should contain only one such instance.
;; If there is more than one, they won't work right.
)
I start emacs with --no-site-file, open a .c-file.
Check the value of c-cleanup-list:
c-cleanup-list is a variable defined in `cc-vars.el'.
Its value is (scope-operator)
Local in buffer xftfont.c; global value is
(scope-operator space-before-funcall)
And indeed, space-before-funcall is not in effect. It seems impossible to
set this variable. I also tried setting c-old-style-variable-behavior to t,
but it made no difference.
The manual needs to explain how to set variables in a manner that works,
an example with lisp to add to .emacs would be nice also.
In GNU Emacs 23.1.50.1 (x86_64-unknown-linux-gnu, GTK+ Version 2.16.1)
of 2009-07-10 on gaffa
Windowing system distributor `The X.Org Foundation', version 11.0.10600000
configured using `configure '--prefix=/opt/emacs-cvs' '--verbose' '--enable-asserts' '--with-x-toolkit=gtk' 'CFLAGS=-g''
Important settings:
value of $LC_ALL: nil
value of $LC_COLLATE: C
value of $LC_CTYPE: nil
value of $LC_MESSAGES: nil
value of $LC_MONETARY: nil
value of $LC_NUMERIC: nil
value of $LC_TIME: nil
value of $LANG: sv_SE.UTF-8
value of $XMODIFIERS: nil
locale-coding-system: utf-8-unix
default-enable-multibyte-characters: t
Major mode: C/l
Minor modes in effect:
tooltip-mode: t
tool-bar-mode: t
mouse-wheel-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
blink-cursor-mode: t
global-auto-composition-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
line-number-mode: t
transient-mark-mode: t
abbrev-mode: t
Recent input:
C-h v c - c l e <tab> <return> C-x 1 <next> <next>
<down-mouse-4> <mouse-4> <double-down-mouse-4> <double-mouse-4>
<triple-down-mouse-4> <triple-mouse-4> <down-mouse-1>
<mouse-1> s t a t i c SPC v o i d <return> f o o (
) <return> { <return> <tab> i n SPC <backspace> t SPC
i SPC = S-SPC b a r ( ) ; <backspace> <backspace> a
) ; C-a <up> <up> <up> C-_ C-_ C-_ C-_ C-_ C-_ C-_
C-_ C-_ C-_ C-_ C-_ C-_ C-_ C-_ C-_ C-_ C-_ C-_ C-_
C-_ C-x 1 M-x r e p o r t - e m <tab> <return>
Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.
Loading vc-cvs...done
Invalid image size (see `max-image-size') [30 times]
Type C-x 4 C-o RET to restore the other window, C-M-v to scroll help.
Undo! [20 times]
undo-more: No further undo information
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list
2009-07-10 11:14 bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list Jan D.
@ 2009-07-11 11:36 ` Alan Mackenzie
2009-07-12 10:33 ` Jan Djärv
0 siblings, 1 reply; 5+ messages in thread
From: Alan Mackenzie @ 2009-07-11 11:36 UTC (permalink / raw)
To: Jan D., 3808; +Cc: emacs-pretest-bug
Hi, Jan!
On Fri, Jul 10, 2009 at 01:14:45PM +0200, Jan D. wrote:
> I customize c-cleanup-list. This is then the full contents of my
> .emacs:
> (custom-set-variables
> ;; custom-set-variables was added by Custom.
> ;; If you edit it by hand, you could mess it up, so be careful.
> ;; Your init file should contain only one such instance.
> ;; If there is more than one, they won't work right.
> '(c-cleanup-list (quote (scope-operator space-before-funcall))))
> (custom-set-faces
> ;; custom-set-faces was added by Custom.
> ;; If you edit it by hand, you could mess it up, so be careful.
> ;; Your init file should contain only one such instance.
> ;; If there is more than one, they won't work right.
> )
> I start emacs with --no-site-file, open a .c-file.
Just done this, in a new userid, having copied your .emacs. To be sure,
I tried on a freshly updated CVS Emacs.
> Check the value of c-cleanup-list:
> c-cleanup-list is a variable defined in `cc-vars.el'.
> Its value is (scope-operator)
I can't reproduce the problem; I get:
Its value is
(scope-operator space-before-funcall)
, just as desired.
> Local in buffer xftfont.c; global value is
> (scope-operator space-before-funcall)
Well, that looks OK. :-)
Might it be that there's a local variables section at the end of
xftfont.c which is overriding c-cleanup-list?
> And indeed, space-before-funcall is not in effect. It seems impossible to
> set this variable. I also tried setting c-old-style-variable-behavior to t,
> but it made no difference.
> The manual needs to explain how to set variables in a manner that works,
> an example with lisp to add to .emacs would be nice also.
"The manual" being the CC Mode manual, of course. In the appendix
"Sample .emacs File", there's an example of c-cleanup-list being set in a
style. The instructions on how to set "style variables" in general (of
which c-cleanup-list is one) are on the page "Configuration Basics" and
its sup-page "Style Variables".
So I think the information is there, but perhaps it's not easy enough to
find. Could you, perhaps, reconstruct how you went about looking for the
info? Maybe the manual needs an extra hint or two somewhere. Thanks!
--
Alan Mackenzie (Nuremberg, Germany).
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list
2009-07-11 11:36 ` Alan Mackenzie
@ 2009-07-12 10:33 ` Jan Djärv
2009-07-12 14:06 ` Alan Mackenzie
0 siblings, 1 reply; 5+ messages in thread
From: Jan Djärv @ 2009-07-12 10:33 UTC (permalink / raw)
To: Alan Mackenzie; +Cc: 3808, emacs-pretest-bug
Alan Mackenzie skrev:
> Hi, Jan!
>
> On Fri, Jul 10, 2009 at 01:14:45PM +0200, Jan D. wrote:
>> I customize c-cleanup-list. This is then the full contents of my
>> .emacs:
>> (custom-set-variables
>> ;; custom-set-variables was added by Custom.
>> ;; If you edit it by hand, you could mess it up, so be careful.
>> ;; Your init file should contain only one such instance.
>> ;; If there is more than one, they won't work right.
>> '(c-cleanup-list (quote (scope-operator space-before-funcall))))
>> (custom-set-faces
>> ;; custom-set-faces was added by Custom.
>> ;; If you edit it by hand, you could mess it up, so be careful.
>> ;; Your init file should contain only one such instance.
>> ;; If there is more than one, they won't work right.
>> )
>
>> I start emacs with --no-site-file, open a .c-file.
>
> Just done this, in a new userid, having copied your .emacs. To be sure,
> I tried on a freshly updated CVS Emacs.
>
>> Check the value of c-cleanup-list:
>
>> c-cleanup-list is a variable defined in `cc-vars.el'.
>> Its value is (scope-operator)
>
> I can't reproduce the problem; I get:
>
> Its value is
> (scope-operator space-before-funcall)
>
> , just as desired.
>
>> Local in buffer xftfont.c; global value is
>> (scope-operator space-before-funcall)
>
> Well, that looks OK. :-)
>
> Might it be that there's a local variables section at the end of
> xftfont.c which is overriding c-cleanup-list?
>
No, but since this is in the emacs source tree, there is a .dir-locals.el
at the root of the source tree, and that comes in effect. So it is not that
it doesn't work, it is just this situation (setting c-file-style as a file
local variable has the same effect).
So what happens is:
c-mode sets style to GNU.
My customization is applied.
.dir-locals.el sets style to GNU, thus erasing my customization.
>> And indeed, space-before-funcall is not in effect. It seems impossible to
>> set this variable. I also tried setting c-old-style-variable-behavior to t,
>> but it made no difference.
>
>> The manual needs to explain how to set variables in a manner that works,
>> an example with lisp to add to .emacs would be nice also.
>
> "The manual" being the CC Mode manual, of course. In the appendix
> "Sample .emacs File", there's an example of c-cleanup-list being set in a
> style. The instructions on how to set "style variables" in general (of
> which c-cleanup-list is one) are on the page "Configuration Basics" and
> its sup-page "Style Variables".
>
> So I think the information is there, but perhaps it's not easy enough to
> find. Could you, perhaps, reconstruct how you went about looking for the
> info? Maybe the manual needs an extra hint or two somewhere. Thanks!
>
The situation above, i.e. how to preserve my customization when .dir-locals.el
or file variables are in effect is not described.
I tried the mode hook, it didn't work either. Is there a c-set-style hook?
Jan D.
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list
2009-07-12 10:33 ` Jan Djärv
@ 2009-07-12 14:06 ` Alan Mackenzie
2009-07-12 15:21 ` Jan Djärv
0 siblings, 1 reply; 5+ messages in thread
From: Alan Mackenzie @ 2009-07-12 14:06 UTC (permalink / raw)
To: Jan Djärv; +Cc: 3808, emacs-pretest-bug
Hi, Jan!
On Sun, Jul 12, 2009 at 12:33:01PM +0200, Jan Djärv wrote:
> Alan Mackenzie skrev:
> >Hi, Jan!
> >On Fri, Jul 10, 2009 at 01:14:45PM +0200, Jan D. wrote:
> >>I customize c-cleanup-list. This is then the full contents of my
> >>.emacs:
> >>(custom-set-variables
> >> ;; custom-set-variables was added by Custom.
> >> ;; If you edit it by hand, you could mess it up, so be careful.
> >> ;; Your init file should contain only one such instance.
> >> ;; If there is more than one, they won't work right.
> >> '(c-cleanup-list (quote (scope-operator space-before-funcall))))
> >>(custom-set-faces
> >> ;; custom-set-faces was added by Custom.
> >> ;; If you edit it by hand, you could mess it up, so be careful.
> >> ;; Your init file should contain only one such instance.
> >> ;; If there is more than one, they won't work right.
> >> )
> >>I start emacs with --no-site-file, open a .c-file.
> >Just done this, in a new userid, having copied your .emacs. To be sure,
> >I tried on a freshly updated CVS Emacs.
> >>Check the value of c-cleanup-list:
> >>c-cleanup-list is a variable defined in `cc-vars.el'.
> >>Its value is (scope-operator)
> >I can't reproduce the problem; I get:
> > Its value is
> > (scope-operator space-before-funcall)
> >, just as desired.
> >>Local in buffer xftfont.c; global value is
> >>(scope-operator space-before-funcall)
> >Well, that looks OK. :-)
> >Might it be that there's a local variables section at the end of
> >xftfont.c which is overriding c-cleanup-list?
> No, but since this is in the emacs source tree, there is a .dir-locals.el
> at the root of the source tree, and that comes in effect. So it is not
> that it doesn't work, it is just this situation (setting c-file-style as a
> file local variable has the same effect).
<MASSIVE GROAN> Ohhh nooooooo. I wasn't aware of this .dir-locals.el
facility. I suppose I should have been - it's been discussed on
emacs-devel before now. Complexity upon complexity! Who needs this?
Who wants it?
> So what happens is:
> c-mode sets style to GNU.
> My customization is applied.
> .dir-locals.el sets style to GNU, thus erasing my customization.
The bug would appear to be the time at which these directory level
customisations are done.
I think it makes some sort of sense for _file_ local variables to be
highest priority - It's a fairly rare thing, only done for a particular
need, and if somebody specifies a particular setting of, say,
`c-cleanup-list' in the file locals, there'll be a damn good reason for
it.
But directory level? Surely they are of LOWEST priority - to have them
overriding things set in hooks etc., is insanity.
> >>And indeed, space-before-funcall is not in effect. It seems impossible to
> >>set this variable. I also tried setting c-old-style-variable-behavior to
> >>t,
> >>but it made no difference.
> >>The manual needs to explain how to set variables in a manner that works,
> >>an example with lisp to add to .emacs would be nice also.
> >"The manual" being the CC Mode manual, of course. In the appendix
> >"Sample .emacs File", there's an example of c-cleanup-list being set in a
> >style. The instructions on how to set "style variables" in general (of
> >which c-cleanup-list is one) are on the page "Configuration Basics" and
> >its sup-page "Style Variables".
> >So I think the information is there, but perhaps it's not easy enough to
> >find. Could you, perhaps, reconstruct how you went about looking for the
> >info? Maybe the manual needs an extra hint or two somewhere. Thanks!
> The situation above, i.e. how to preserve my customization when
> .dir-locals.el or file variables are in effect is not described.
Yes. In the case of file local variables, the FM states (page "Config
Basics") that they supersede settings made in other ways. This is surely
their point - they're intended for rare (high priority) use. For
example, you might want to use a different style (C style, as described
on page "Styles" in the fine manual) for a particular source file that
was imported from somewhere else, so as not to cause masses of
reformatting diffs.
> I tried the mode hook, it didn't work either. Is there a c-set-style hook?
Now, you're being sarcastic here, aren't you? PLEASE tell me you're
being sarcastic. :-) (Note, I don't blame you. I would be too, in your
position.)
I'll bring this matter up on emacs-devel. Bare with me!
> Jan D.
--
Alan Mackenzie (Nuremberg, Germany).
^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list
2009-07-12 14:06 ` Alan Mackenzie
@ 2009-07-12 15:21 ` Jan Djärv
0 siblings, 0 replies; 5+ messages in thread
From: Jan Djärv @ 2009-07-12 15:21 UTC (permalink / raw)
To: Alan Mackenzie; +Cc: 3808, emacs-pretest-bug
Alan Mackenzie skrev:
> Hi, Jan!
Hi.
>
>> I tried the mode hook, it didn't work either. Is there a c-set-style hook?
>
> Now, you're being sarcastic here, aren't you? PLEASE tell me you're
> being sarcastic. :-) (Note, I don't blame you. I would be too, in your
> position.)
Not really, I could think of uses for such a hook.
>
> I'll bring this matter up on emacs-devel. Bare with me!
>
Okay.
Jan D.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-07-12 15:21 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-10 11:14 bug#3808: 23.1.50; CC-mode: Impossible to set c-cleanup-list Jan D.
2009-07-11 11:36 ` Alan Mackenzie
2009-07-12 10:33 ` Jan Djärv
2009-07-12 14:06 ` Alan Mackenzie
2009-07-12 15:21 ` Jan Djärv
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).