* bug#23816: adaptive-wrap-extra-indent should be safe as a file local variable
@ 2016-06-21 16:02 Jonathan Kotta
2017-05-04 17:00 ` Jonathan Kotta
0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Kotta @ 2016-06-21 16:02 UTC (permalink / raw)
To: 23816
[-- 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 --]
^ permalink raw reply related [flat|nested] 3+ messages in thread
* bug#23816: adaptive-wrap-extra-indent should be safe as a file local variable
2016-06-21 16:02 bug#23816: adaptive-wrap-extra-indent should be safe as a file local variable Jonathan Kotta
@ 2017-05-04 17:00 ` Jonathan Kotta
2017-05-05 0:40 ` npostavs
0 siblings, 1 reply; 3+ messages in thread
From: Jonathan Kotta @ 2017-05-04 17:00 UTC (permalink / raw)
To: 23816
[-- 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 --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* bug#23816: adaptive-wrap-extra-indent should be safe as a file local variable
2017-05-04 17:00 ` Jonathan Kotta
@ 2017-05-05 0:40 ` npostavs
0 siblings, 0 replies; 3+ messages in thread
From: npostavs @ 2017-05-05 0:40 UTC (permalink / raw)
To: Jonathan Kotta; +Cc: 23816
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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-05-05 0:40 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-21 16:02 bug#23816: adaptive-wrap-extra-indent should be safe as a file local variable Jonathan Kotta
2017-05-04 17:00 ` Jonathan Kotta
2017-05-05 0:40 ` npostavs
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).