[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: files.el: Patch to make project-settings optional/customizable
From: |
Stephen J. Turnbull |
Subject: |
Re: files.el: Patch to make project-settings optional/customizable |
Date: |
Mon, 24 Nov 2008 11:14:24 +0900 |
Dan Nicolaescu writes:
> Brittle is to allow the user to break something just by using a normal
> way of changing settings "setq". We have gone over this too many times
> already...
Agreed, making a defconst is not childproof (which seems to be the
goal of several senior developers these days). But I thought Emacs
Lisp was a language used by adults? You expect adults to read the
documentation and take responsibility for what they do. I really
don't see a problem with using a defconst here.
BTW, "brittle" doesn't mean that it breaks when the user uses normal
features of Lisp to intentionally do something inadvisable. "Brittle"
means it breaks *accidentally* in the normal process of programming.
Making this a defconst is the opposite of brittle: it is just good
programming practice, as it allows the byte compiler to check for
typos etc. and warn about attempts to set or bind the variable. You
can't do that with a string. (Well, you *could*, but then it would be
a symbol, wouldn't it?)
- Re: files.el: Patch to make project-settings optional/customizable, (continued)
- Re: files.el: Patch to make project-settings optional/customizable, Dan Nicolaescu, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable, Juri Linkov, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable, Dan Nicolaescu, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable, Juanma Barranquero, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable, Dan Nicolaescu, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable, Juanma Barranquero, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable, Dan Nicolaescu, 2008/11/23
- Re: files.el: Patch to make project-settings optional/customizable,
Stephen J. Turnbull <=
- Re: files.el: Patch to make project-settings optional/customizable, Dan Nicolaescu, 2008/11/24
- Re: files.el: Patch to make project-settings optional/customizable, Chong Yidong, 2008/11/24
- Re: files.el: Patch to make project-settings optional/customizable, Kevin Rodgers, 2008/11/21
- Re: files.el: Patch to make project-settings optional/customizable, Stephen J. Turnbull, 2008/11/18
- Re: files.el: Patch to make project-settings optional/customizable, Tom Tromey, 2008/11/13
- Re: files.el: Patch to make project-settings optional/customizable, Tom Tromey, 2008/11/13
- Re: files.el: Patch to make project-settings optional/customizable, Juanma Barranquero, 2008/11/13