Re: safe-local-variable stuff

2007-01-03 Thread Reiner Steib
On Wed, Jan 03 2007, Stefan Monnier wrote:

>> +(default-directory   . string-or-null-p) ;; C source code
>
> Why "-or-null-" ?  default-directory should never be nil.

I though because the global value is nil.  But maybe this makes no
sense.

Bye, Reiner.
-- 
   ,,,
  (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/


___
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug


Re: safe-local-variable stuff

2007-01-03 Thread Stefan Monnier
> + (default-directory   . string-or-null-p) ;; C source code

Why "-or-null-" ?  default-directory should never be nil.


Stefan


___
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug


Re: safe-local-variable stuff

2007-01-03 Thread Reiner Steib
On Wed, Jan 03 2007, Peter Dyballa wrote:

> When I save a *compilation* buffer and I later visit that file, the
> GNU Emacsen from CVS (22.0.*, 23.0.0) complain about the
> default-directory variable used. Is there some reason why this
> variable is treated as unsafe? Why are the *compilation* buffers
> settings this unsafe variable at all?

I think it makes sense to mark `default-directory' as safe.  If nobody
objects, I'll commit this change:

--8<---cut here---start->8---
--- files.el03 Jan 2007 12:13:53 +0100  1.875
+++ files.el03 Jan 2007 14:21:21 +0100  
@@ -2442,6 +2442,7 @@
 (mapc (lambda (pair)
(put (car pair) 'safe-local-variable (cdr pair)))
   '((buffer-read-only. booleanp) ;; C source code
+   (default-directory   . string-or-null-p) ;; C source code
(fill-column . integerp) ;; C source code
(indent-tabs-mode. booleanp) ;; C source code
(left-margin . integerp) ;; C source code
--8<---cut here---end--->8---

We could also use `file-directory-p' or `stringp' instead of
`string-or-null-p', but I think `string-or-null-p' is the best choice.

> Or did I miss some customisation?
>
> One AUCTeX variable is treated as unsafe, too: TeX-command-default. 

emacs-pretest is the wrong list for this.  I'll answer to this part on
auctex-devel (Cc-ing you).

Bye, Reiner.
-- 
   ,,,
  (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/


___
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug


safe-local-variable stuff

2007-01-03 Thread Peter Dyballa

Hello!

When I save a *compilation* buffer and I later visit that file, the  
GNU Emacsen from CVS (22.0.*, 23.0.0) complain about the default- 
directory variable used. Is there some reason why this variable is  
treated as unsafe? Why are the *compilation* buffers settings this  
unsafe variable at all? Or did I miss some customisation?


One AUCTeX variable is treated as unsafe, too: TeX-command-default.  
Are there reasons for doing this? The command to become executed has  
to be registered before as a command for the AUCTeX Commands menu. So  
it should be safe. Or am I wrong?


In GNU Emacs 23.0.0.1 (powerpc-apple-darwin8.8.0, X toolkit, Xaw3d  
scroll bars)

of 2007-01-02 on Latsche.local
X server distributor `The XFree86 Project, Inc', version 11.0.4040
configured using `configure  '--without-ns' '--without-carbon' '-- 
with-x' '--without-sound' '--without-pop' '--with-xpm' '--with-jpeg'  
'--with-tiff' '--with-gif' '--with-png' '--enable-locallisppath=/ 
Library/Application Support/Emacs/calendar23:/Library/Application  
Support/Emacs/preview:/Library/Application Support/Emacs/auctex/ 
images:/Library/Application Support/Emacs/auctex:/Library/Application  
Support/Emacs' 'CFLAGS=-pipe -fPIC -mcpu=7450 -mtune=7450 -fast -mpim- 
altivec -ftree-vectorize -foptimize-register-move -freorder-blocks - 
freorder-blocks-and-partition -fthread-jumps -fpeephole -fno- 
crossjumping' 'CPPFLAGS=-no-cpp-precomp -I/usr/include/openssl -I/usr/ 
local/include -I/sw/include/libpng12 -I/sw/include -idirafter /usr/ 
X11R6/include' 'LDFLAGS=-dead_strip -bind_at_load -multiply_defined  
suppress -L/usr/X11R6/lib -L/usr/local/lib -L/sw/lib/ncurses -L/sw/ 
lib' 'ACLOCAL=aclocal -I /sw/share/aclocal -I /usr/local/share/ 
aclocal -I /usr/X11R6/share/aclocal' 'LIBTOOLIZE=/usr/local/bin/ 
libtoolize' 'PKG_CONFIG_PATH=/sw/lib/freetype219/lib/pkgconfig:/sw/ 
lib/fontconfig2/lib/pkgconfig:/usr/X11R7/lib/pkgconfig:/sw/lib/ 
pkgconfig:/sw/lib/system-openssl/lib/pkgconfig:/sw/share/pkgconfig:/ 
usr/lib/pkgconfig:/usr/local/lib/pkgconfig:/usr/local/clamXav/lib/ 
pkgconfig' 'PATH=/usr/local/bin:/sw/lib/freetype219/bin:/sw/lib/ 
odcctools/bin:/sw/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/X11R6/bin:/ 
usr/local/gwTeX/bin/powerpc-apple-darwin-current''


Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: de_DE.UTF-8
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: de_DE.UTF-8
  value of $XMODIFIERS: nil
  locale-coding-system: utf-8-unix
  default-enable-multibyte-characters: t



--
Greetings

  Pete

When confronted with actual numbers, a mathematician is at a loss.
 (Steffen Hokland)




___
emacs-pretest-bug mailing list
emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug