[Emacs-bug-tracker] bug#3230: closed (dired-actual-switches is risky)

[GNU bug Tracking System]

Your message dated Mon, 28 Feb 2011 22:25:59 -0500
with message-id <address@hidden>
and subject line Re: bug#3230: 23.0.93; Make dired-actual-switches safe local 
variable?
has caused the GNU bug report #3230,
regarding dired-actual-switches is risky
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
3230: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=3230
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 23.0.93; Make dired-actual-switches safe local variable?
>
>
>
>
> Date: 
>
> Wed, 06 May 2009 15:20:57 +0100
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/23.0.93 (darwin)
>
>
>
>
> Hi there,
>
> The dired-x manual gives an example in using local variables for dired
> buffers. However, the variable dired-actual-switches has not been marked
> as safe local variable. I think this is an oversight.
>
> ,----[ (info "(dired-x)Local Variables") ]
> | For example, if the user puts
> | 
> |      Local Variables:
> |      dired-actual-switches: "-lat"
> |      dired-omit-mode: t
> |      End:
> `----
>
> Best wishes,
>
> Leo
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#3230: 23.0.93; Make dired-actual-switches safe local variable?
>
>
>
>
> Date: 
>
> Mon, 28 Feb 2011 22:25:59 -0500
>
>
>
>
> User-agent: 
>
> Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
>
>
>
>
> Version: 24.1
>
> Stefan Monnier wrote:
>
> > should then go to safe-local-variable.  I'd recommend something simple
> > like
> >
> >   (defun dired-safe-switches-p (switches)
> >     (string-match "\\`[- [[:alnum:]]]+\\'" switches))
>
> Added.
>
>
> --- End Message ---