Re: [elpa] master 9173a44 1/2: [Fix #12] Don't autodisable beacon on hl-line-mode

[Artur Malabarba <bruce.connor.am@gmail.com>]

Then I'm clearly unaware of something here.
There was a discussion here a few months ago where I learned that
hooks should always be defined with nil initial values.
So, if it must be defined empty, and it's wrong to add entries to it
later, then what's the correct way for a package to have a non-empty
hook?

2015-10-18 4:20 GMT+01:00 Stefan Monnier <monnier@iro.umontreal.ca>:
>> +    (add-hook 'beacon-dont-blink-predicates
>> +              (lambda () (bound-and-true-p hl-line-mode)))"
>>    :type 'hook)
>
>> -(add-hook 'beacon-dont-blink-predicates (lambda () (bound-and-true-p hl-line-mode)))
>>  (add-hook 'beacon-dont-blink-predicates #'window-minibuffer-p)
>
> This add-hook will interfere with defcustom's handling of the variable
> (i.e. Customize will rightly complain that "the variable was modified
> outside of Customize").
>
>
>         Stefan