Re: pop-to-buffer and friends new behavior or bug?

[Thierry Volpiatto <thierry.volpiatto@gmail.com>]

martin rudalics <rudalics@gmx.at> writes:

>> I have fixed my problem in anything.el using this:
>>
>> --8<---------------cut here---------------start------------->8---
>> [...]
>>     (when (boundp 'display-buffer-alist)
>>       (setq not-this-window '((reuse-window same other nil)
>>                               (pop-up-window (other)))))
>>     (display-buffer buf not-this-window)))
>> --8<---------------cut here---------------end--------------->8---
>>
>> It is working fine like this, but i would like to know if that is
>> correct.
>
> I suppose it's reusing another window now because the reuse-window
> specifier is correct - try the selected window if it shows another
> buffer.  The pop-up-window specifier wants a list of window . side
> conses like (pop-up-window (largest . below) (selected .right)) to split
> the largest window with the new window below or the selected window with
> the new window on the right.
Ok, so the (pop-up-window (other)) is incorrect (or incomplete) and
maybe unuseful in this case.

> But please don't fix this before I found the cause of the bug.  Having a
> sample where it tries to use the minibuffer window is important for me.
> (Unfortunately, I haven't git installed so I first have to get this from
> somewhere else.  Or you send me the files exposing the bug via private
> mail?)
No problem if you have not git, just download the tarball here:
http://repo.or.cz/w/anything-config.git

Last revision have bug fixed with the code above, just comment it out,
or download the revision just before to have the bug:

--8<---------------cut here---------------start------------->8---
author	Thierry Volpiatto <thierry.volpiatto@gmail.com>	
Mon, 20 Jun 2011 20:28:20 +0000 (22:28 +0200)
committer	Thierry Volpiatto <thierry.volpiatto@gmail.com>	
Mon, 20 Jun 2011 20:28:20 +0000 (22:28 +0200)
commit	fe786b9d1cbcb8431468f309ed7fa5f5676d47b0
tree	21694f5a4f95654dac54d7e580dab1ca4b40fcd4
parent	9e7236e34f98ba1a5ad8b6414f7f45df70bc531e
--8<---------------cut here---------------end--------------->8---

The relevant code is in anything.el starting at line 3123 at:
;; (@* "Utility: Persistent Action")

and more exactly in `anything-persistent-action-display-buffer'.

Follow the recipe i sent in precedent mail.
To see what is expected use the fixed code.

You will find a pdf file in the ../doc directory, even if it's not
up to date, it can help you if you are not familiar with anything.

Thanks.

-- 
A+ Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997