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

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

[Jonathan Kotta]

[-- Attachment #1: Type: text/plain, Size: 914 bytes --]

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.

[-- Attachment #2: Type: text/html, Size: 1262 bytes --]

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

[Jonathan Kotta]

[-- Attachment #1: Type: text/plain, Size: 1338 bytes --]

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@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.

[-- Attachment #2: Type: text/html, Size: 2168 bytes --]

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

[npostavs]

tags 23816 fixed
close 23816 
quit

Jonathan Kotta <jpkotta@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