> Reverting the following change fixes the bug:
> 2007-01-30 Richard Stallman <[EMAIL PROTECTED]>
> * minibuf.c (read_minibuf):
> Save and clear Vminibuffer_completing_file_name.
> (read_minibuf_unwind): Restore it.
> (Vminibuffer_completion_table, Qminibuffer_completion_table)
> (Vminibuffer_completion_predicate, Qminibuffer_completion_predicate)
> (Vminibuffer_completion_confirm, Qminibuffer_completion_confirm)
> (Vminibuffer_completing_file_name): Definitions moved up.
> Yes I know that this change was made to fix some other bug,
> but the number of things broken by that change is increasing,
> so that change doesn't seem like the right fix!!
Indeed, but it's not clear what's The Right Fix. Maybe instead of let-binding
Vminibuffer_completing_file_name, we should set it buffer-locally.
Yes, I think it would be a better fix, but it's still not clear how to do
it, because we'd like to set it in Fread_file_name, but the buffer is only
created in read_minibuf.
Basically, one would want to use minibuffer-setup-hook, but this presents
similar difficulties: rather than a global var (i.e. an implicit arg), what
we really want is to pass it as an explicit `setup' argument.
Stefan
_______________________________________________
emacs-pretest-bug mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
