all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Stefan Monnier <monnier@iro.umontreal.ca>
To: 12418@debbugs.gnu.org
Subject: bug#12418: emacs 24.2; idlw-shell.el: filename completion in IDL shell is not working
Date: Thu, 13 Sep 2012 08:36:03 -0400	[thread overview]
Message-ID: <jwvoblads22.fsf-monnier+emacs@gnu.org> (raw)
In-Reply-To: <qqmx0vacna.fsf@fencepost.gnu.org> (Glenn Morris's message of "Wed, 12 Sep 2012 04:06:17 -0400")

>> -    (comint-filename-completion)))
>> +    (comint-dynamic-complete-filename)))

> The obsolescence message for comint-dynamic-complete-as-filename seems
> misleading to me:

>    This function is obsolete since 24.1;
>    use `comint-filename-completion' instead.

> Surely it would be better to say `use comint-dynamic-complete-filename
> instead',

Actually, that depends on what comint-dynamic-complete-as-filename was
used for: if it was used as a *command*, then
comint-dynamic-complete-filename is a good replacement.  But if it was
used within something like comint-dynamic-complete-functions, then the
better replacement is comint-filename-completion.

IOW, in the case of idlw-shell.el, the better replacement is indeed
comint-filename-completion, tho it requires changes in
idlwave-shell-complete (basically making it an alias to
completion-at-point and turning its body into a list of functions to put
on completion-at-point-functions).


        Stefan





  reply	other threads:[~2012-09-13 12:36 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-11 18:44 bug#12418: emacs 24.2; idlw-shell.el: filename completion in IDL shell is not working Jose Marino
2012-09-12  8:06 ` Glenn Morris
2012-09-13 12:36   ` Stefan Monnier [this message]
2012-09-13 18:24     ` Glenn Morris
2012-09-14  1:17       ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=jwvoblads22.fsf-monnier+emacs@gnu.org \
    --to=monnier@iro.umontreal.ca \
    --cc=12418@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.