bug#81512: [PATCH] Add command `outline-occur'

Juri Linkov <[email protected]> writes:

>>> It's fine for packages to add own rules to the existing value of
>>> 'display-buffer-default-alist'.  Then the last package wins
>>> when the first rule will match, e.g.:
>>>
>>>    (((category . bookmark-jump)
>>>       display-buffer-for-package-2
>>>       (package . 2))
>>>     ((category . bookmark-jump)
>>>       display-buffer-for-package-1
>>>       (package . 1))
>>
>> This means that once the user has loaded package-2, the behavior
>> prescribed by the previously loaded package-1 is lost.  This cannot
>> happen if package designers are told to let-bind that variable.
>
> We can't tell authors what they should do.  But it's fine to add
> a suggestion in the docstring that it's intended to be used with let-bind.

If we all agree in the changes below, I'll send the v4 of the patch.

- Rename the variable to `display-buffer-default-alist'.
- Add a note in the docstring of `display-buffer-default-alist' as well
  as the corresponding section of the Elisp manual saying that this
  variable is intended to be let-bound (rather than saying that "it
  should never be assigned buffer-locally or globally" as in the v2 of
  patch).

Juri, Martin are you OK?

        Roi