Kaushal Modi <kaushal.m...@gmail.com> writes:

> On Mon, Jun 26, 2017 at 4:19 PM Nicolas Goaziou <m...@nicolasgoaziou.fr>
> wrote:
>
>>
>> Meanwhile, we could simply change :safe keyword from `org-ellipsis' to
>>
>>   (lambda (v) (and (string-or-null-p v) (not (equal v ""))))
>>
>>
>> WDYT?
>>
>
> But that would still allow the user to use setq to set that variable.

True, but can we always prevent them from shooting themselves in the
foot?

Anyway, I updated the :safe value and changed the test below:

  (vconcat (mapcar (lambda (c) (make-glyph-code c 'org-ellipsis))
                   (if (stringp org-ellipsis) org-ellipsis "...")))


Regards,

Reply via email to