Re: dired "dwim" mode seems to fail now

[whda]

>>> Some time in the past few weeks, dired-dwim-target mode seems to have
>>> forgotten how to work (it's not offering the directory of the other
>>> buffer in the same window as a prefix).  I haven't had a chance to
>>> narrow the commit down, but if that rings a bell for anyone, consider
>>> this a nudge.
>>
>> I didn't look at commits, but it seems `dired-dwim-target-directory'
>> calls `get-window-with-predicate' in such a way that the first window
>> returned is possibly the current window itself. It seems the
>> (unspecified?) order in which `get-window-with-predicate' tests windows
>> may have changed.
>
>That's been the case, indeed.  Thanks for catching this.  I've now tried
>to restore the previous behavior.  Please try again.

Confirmed to be working as before.

>> I think the solution is to exclude from consideration
>> dired buffers that have the same current directory as the current
>> window. This seems most consistent with the current (23.2) stable emacs.
>>
>> The real problem might be with window.el though. dired-aux.el hasn't
>> changed recently.
>>
>> Patch:
>
>I think installing your patch is a good idea anyway.  There's no good
>reason to _not_ return the selected window as first window provided it
>satisfies `get-window-with-predicate'.

Indeed. As I can't actually install the patch can someone else do it?

>Thanks again for finding this, martin
>
>
>PS: I now also restored the old behavior of `get-buffer-window-list' so
>it starts with the selected window.  As a consequence, windows on the
>selected frame that come before the selected window will be listed after
>windows on other frames.  But as the present example shows it's better
>to remain compatible with Emacs 23.