Should `auto-fill-function' be a safe local variable?

Should `auto-fill-function' be a safe local variable?

[Leo]

Hi there,

I have to disable auto-fill-mode for one file and it seems setting
auto-fill-function to nil in the file's local variables is a good
way. However, since auto-fill-function is currently marked risky, it
requires a confirmation each time the file is opened.

So should `auto-fill-function' be a safe local variable?

Cheers,
-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

Re: Should `auto-fill-function' be a safe local variable?

[Stefan Monnier]

> I have to disable auto-fill-mode for one file and it seems setting
> auto-fill-function to nil in the file's local variables is a good
> way. However, since auto-fill-function is currently marked risky, it
> requires a confirmation each time the file is opened.

How do you confirm?  The confirmation dialog offers you to make your
choice stick, so that subsequent uses of this settings are accepted
without confirmation.  Did you use that?

> So should `auto-fill-function' be a safe local variable?

Of course not.  It's safe to set it to nil, but not to set it to some
arbitrary value such as (lambda (&rest) (wreak-havoc)).


        Stefan

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-06 21:39 +0000, Stefan Monnier wrote:
>> I have to disable auto-fill-mode for one file and it seems setting
>> auto-fill-function to nil in the file's local variables is a good
>> way. However, since auto-fill-function is currently marked risky, it
>> requires a confirmation each time the file is opened.
>
> How do you confirm?  The confirmation dialog offers you to make your
> choice stick, so that subsequent uses of this settings are accepted
> without confirmation.  Did you use that?

There is no such option.

,----
| The local variables list in example.tex
| contains variables that are risky (**).
| 
| Do you want to apply it?  You can type
| y  -- to apply the local variables list.
| n  -- to ignore the local variables list.
| 
|  ** auto-fill-function : nil
`----

>> So should `auto-fill-function' be a safe local variable?
>
> Of course not.  It's safe to set it to nil, but not to set it to some
> arbitrary value such as (lambda (&rest) (wreak-havoc)).

Maybe value nil should be marked safe as RMS suggested in another thread.

>         Stefan

Cheers,
-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

Re: Should `auto-fill-function' be a safe local variable?

[Richard M Stallman]

    So should `auto-fill-function' be a safe local variable?

In general, it cannot be safe.  Its value is a function.
But we could say that the value nil is safe for it.

Perhaps it is unfortunate that the variable which controls
Auto Fill mode has a function value rather than a boolean value.
If nobody makes use of the potential to specify different kinds of
auto-filling functions, perhaps we should replace it with a boolean
variable `auto-fill-mode'.

Re: Should `auto-fill-function' be a safe local variable?

[Stefan Monnier]

> If nobody makes use of the potential to specify different kinds of
> auto-filling functions, perhaps we should replace it with a boolean
> variable `auto-fill-mode'.

Several major modes do make use of this potential.


        Stefan

Re: Should `auto-fill-function' be a safe local variable?

[Richard M Stallman]

    > If nobody makes use of the potential to specify different kinds of
    > auto-filling functions, perhaps we should replace it with a boolean
    > variable `auto-fill-mode'.

    Several major modes do make use of this potential.

We could also add a local variable `auto-fill-mode' to enable
and disable the mode, and have `auto-fill-function' by default
have the usual function for auto-filling.  Then a major mode
could set `auto-fill-function' to specify HOW to do auto-fill
when it is enabled.

However...

I think it is a bad idea for a file to enable auto filling
in its local variables list, because that's a user preference
rather than a description of the file's format.

I think the OP wanted to a file disable auto filling, not enable it.
MAYBE that really does represent some info about the file's format.
We could make nil a safe value for `auto-fill-function'
and it would allow this.

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-08 11:38 +0000, Richard M Stallman wrote:
> I think the OP wanted to a file disable auto filling, not enable it.
> MAYBE that really does represent some info about the file's format.
> We could make nil a safe value for `auto-fill-function' and it would
> allow this.

I have auto-fill-mode enabled for all text modes however, there's one
particular file that I got from someone else which I don't want
auto-fill to happen.

Thanks,
-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

Re: Should `auto-fill-function' be a safe local variable?

[Richard M Stallman]

    I have auto-fill-mode enabled for all text modes however, there's one
    particular file that I got from someone else which I don't want
    auto-fill to happen.

The question is whether this is a matter of personal taste
or in some sense an objective statement about that particular file.
If the former, you should set up a text-mode-hook function to check
for that file.  If the latter, setting auto-save-function to nil
in that file's local variables list is the right solution.

What do that file's contents look like?

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-09 23:58 +0000, Richard M Stallman wrote:
> The question is whether this is a matter of personal taste
> or in some sense an objective statement about that particular file.
> If the former, you should set up a text-mode-hook function to check
> for that file.  If the latter, setting auto-save-function to nil
> in that file's local variables list is the right solution.

It is a personal taste I guess. Other editors may have its own way of
displaying long lines and thus not need something like
auto-fill-mode. Some people actual hate automatically inserting hard
newlines. So when you are working with them, you don't want auto-fill to
get in the way.

> What do that file's contents look like?

It is a tex file something like this:

\define@key{codeexample}{vbox}[]{\def\code@pre{\vbox\bgroup\setlength{\hsize}{\linewidth-6pt}}\def\code@post{\egroup}}

Best,

Leo

Re: Should `auto-fill-function' be a safe local variable?

[Richard M Stallman]

    It is a tex file something like this:

    \define@key{codeexample}{vbox}[]{\def\code@pre{\vbox\bgroup\setlength{\hsize}{\linewidth-6pt}}\def\code@post{\egroup}}

For that file, Emacs should use TeX mode, not Text mode.
Does it?
 

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-10 20:33 +0000, Richard M Stallman wrote:
>     It is a tex file something like this:
>
>     \define@key{codeexample}{vbox}[]{\def\code@pre{\vbox\bgroup\setlength{\hsize}{\linewidth-6pt}}\def\code@post{\egroup}}
>
> For that file, Emacs should use TeX mode, not Text mode.  Does it?

It seems it is derived from text-mode at least for the one in AUCTEX.

Best,

Leo

Re: Should `auto-fill-function' be a safe local variable?

[Richard M Stallman]

    > For that file, Emacs should use TeX mode, not Text mode.  Does it?

    It seems it is derived from text-mode at least for the one in AUCTEX.

The normal TeX-mode is also derived from text-mode, I see.

But your hook function on text-mode-hook, which turns on Auto Fill
mode, can test what the mode really is, and not operate for TeX files.

Re: Should `auto-fill-function' be a safe local variable?

[Reiner Steib]

On Sun, Jan 11 2009, Richard M Stallman wrote:

>     It seems it is derived from text-mode at least for the one in AUCTEX.
>
> The normal TeX-mode is also derived from text-mode, I see.
>
> But your hook function on text-mode-hook, which turns on Auto Fill
> mode, can test what the mode really is, and not operate for TeX files.

Most user like to have auto-fill in (La)TeX files, I'd guess.

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

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-11 19:12 +0000, Richard M Stallman wrote:
>     > For that file, Emacs should use TeX mode, not Text mode.  Does it?
>
>     It seems it is derived from text-mode at least for the one in AUCTEX.
>
> The normal TeX-mode is also derived from text-mode, I see.
>
> But your hook function on text-mode-hook, which turns on Auto Fill
> mode, can test what the mode really is, and not operate for TeX files.

Yes, normally I want auto-fill in TeX mode. But occasionally I want to
turn it off for some particular files.

Best,

Leo

Re: Should `auto-fill-function' be a safe local variable?

[Miles Bader]

Leo <sdl.web@gmail.com> writes:
>> But your hook function on text-mode-hook, which turns on Auto Fill
>> mode, can test what the mode really is, and not operate for TeX files.
>
> Yes, normally I want auto-fill in TeX mode. But occasionally I want to
> turn it off for some particular files.

It would seem much more convenient to have a separate boolean
`auto-fill-mode' variable, so the file-variables can fool with that, and
leave setting of auto-fill-function to the mode!

-Miles

-- 
Rational, adj. Devoid of all delusions save those of observation, experience
and reflection.

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-06 22:59 +0000, Richard M Stallman wrote:
> In general, it cannot be safe.  Its value is a function.  But we could
> say that the value nil is safe for it.

What's the solution to my original problem?

Thanks,
-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-01-06 22:59 +0000, Richard M Stallman wrote:
>     So should `auto-fill-function' be a safe local variable?
>
> In general, it cannot be safe.  Its value is a function.
> But we could say that the value nil is safe for it.
[...]

On 2009-01-06 21:39 +0000, Stefan Monnier wrote:
[...]
>> So should `auto-fill-function' be a safe local variable?
>
> Of course not.  It's safe to set it to nil, but not to set it to some
> arbitrary value such as (lambda (&rest) (wreak-havoc)).

Hi Richard and Stefan,

Our discussion a long while ago seems to converge to making nil safe for
auto-fill-function. I have been using

  (put 'auto-fill-function 'safe-local-variable 'null)

in my .emacs, which enables users to disable auto-fill on a per-file
basis. May it be possible to make that default in Emacs?


Best wishes,
Leo

Re: Should `auto-fill-function' be a safe local variable?

[Stefan Monnier]

> Our discussion a long while ago seems to converge to making nil safe for
> auto-fill-function. I have been using

>   (put 'auto-fill-function 'safe-local-variable 'null)

> in my .emacs, which enables users to disable auto-fill on a per-file
> basis. May it be possible to make that default in Emacs?

Yes, done, thank you, and sorry for the delay,


        Stefan

Re: Should `auto-fill-function' be a safe local variable?

[Leo]

On 2009-04-02 02:56 +0100, Stefan Monnier wrote:
>> Our discussion a long while ago seems to converge to making nil safe for
>> auto-fill-function. I have been using
>
>>   (put 'auto-fill-function 'safe-local-variable 'null)
>
>> in my .emacs, which enables users to disable auto-fill on a per-file
>> basis. May it be possible to make that default in Emacs?
>
> Yes, done, thank you, and sorry for the delay,

Many thanks!
>
>         Stefan

-- 
.:  Leo  :.  [ sdl.web AT gmail.com ]  .: I use Emacs :.

               www.git-scm.com
    git - the one true version control system