GNU bug report logs - #23816
adaptive-wrap-extra-indent should be safe as a file local variable

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 23816 in the body.
You can then email your comments to 23816 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Jonathan Kotta <jpkotta <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Subject: adaptive-wrap-extra-indent should be safe as a file local variable
Date: Tue, 21 Jun 2016 11:02:22 -0500

[Message part 1 (text/plain, inline)]

The variable adaptive-wrap-extra-indent from adaptive-wrap.el in ELPA
should be a safe as a file local variable.  It can be useful to set as a
file local and shouldn't pose any danger from untrusted files.

diff --git a/packages/adaptive-wrap/adaptive-wrap.el
b/packages/adaptive-wrap/adaptive-wrap.el
index 9455789..01024de 100644
--- a/packages/adaptive-wrap/adaptive-wrap.el
+++ b/packages/adaptive-wrap/adaptive-wrap.el
@@ -57,6 +57,7 @@ extra indent = 2
   :type 'integer
   :group 'visual-line)
 (make-variable-buffer-local 'adaptive-wrap-extra-indent)
+(put 'adaptive-wrap-extra-indent 'safe-local-variable 'integerp)

 (defun adaptive-wrap-fill-context-prefix (beg en)
   "Like `fill-context-prefix', but with length adjusted by
`adaptive-wrap-extra-indent'."

-- 
Thanks,

Jonathan Kotta

Hofstadter's Law:
    It always takes longer than you expect, even
    when you take into account Hofstadter's Law.

[Message part 2 (text/html, inline)]

Message #8 received at 23816 <at> debbugs.gnu.org (full text, mbox):

From: Jonathan Kotta <jpkotta <at> gmail.com>
To: 23816 <at> debbugs.gnu.org
Subject: Re: bug#23816: adaptive-wrap-extra-indent should be safe as a file
 local variable
Date: Thu, 4 May 2017 12:00:15 -0500

[Message part 1 (text/plain, inline)]

Ping.  If there's another place I should report this, please let me know.
I figured that since this is part of GNU ELPA, this is the correct place.

On Tue, Jun 21, 2016 at 11:02 AM, Jonathan Kotta <jpkotta <at> gmail.com> wrote:

> The variable adaptive-wrap-extra-indent from adaptive-wrap.el in ELPA
> should be a safe as a file local variable.  It can be useful to set as a
> file local and shouldn't pose any danger from untrusted files.
>
> diff --git a/packages/adaptive-wrap/adaptive-wrap.el
> b/packages/adaptive-wrap/adaptive-wrap.el
> index 9455789..01024de 100644
> --- a/packages/adaptive-wrap/adaptive-wrap.el
> +++ b/packages/adaptive-wrap/adaptive-wrap.el
> @@ -57,6 +57,7 @@ extra indent = 2
>    :type 'integer
>    :group 'visual-line)
>  (make-variable-buffer-local 'adaptive-wrap-extra-indent)
> +(put 'adaptive-wrap-extra-indent 'safe-local-variable 'integerp)
>
>  (defun adaptive-wrap-fill-context-prefix (beg en)
>    "Like `fill-context-prefix', but with length adjusted by
> `adaptive-wrap-extra-indent'."
>
> --
> Thanks,
>
> Jonathan Kotta
>
> Hofstadter's Law:
>     It always takes longer than you expect, even
>     when you take into account Hofstadter's Law.
>



-- 
Thanks,

Jonathan Kotta

Hofstadter's Law:
    It always takes longer than you expect, even
    when you take into account Hofstadter's Law.

[Message part 2 (text/html, inline)]

Message #11 received at 23816 <at> debbugs.gnu.org (full text, mbox):

From: npostavs <at> users.sourceforge.net
To: Jonathan Kotta <jpkotta <at> gmail.com>
Cc: 23816 <at> debbugs.gnu.org
Subject: Re: bug#23816: adaptive-wrap-extra-indent should be safe as a file
 local variable
Date: Thu, 04 May 2017 20:40:51 -0400

tags 23816 fixed
close 23816 
quit

Jonathan Kotta <jpkotta <at> gmail.com> writes:

>  The variable adaptive-wrap-extra-indent from adaptive-wrap.el in ELPA
>  should be a safe as a file local variable. It can be useful to set as
>  a file local and shouldn't pose any danger from untrusted files.

Makes sense, done in [1: daa268a77].

[1: daa268a77]: 2017-05-04 20:36:00 -0400
  Mark adaptive-wrap-extra-indent as safe if integerp (Bug#23816)
  http://git.savannah.gnu.org/cgit/emacs/elpa.git/commit/?id=daa268a7794de3f2638c6a807c5147c14fb647bb

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.