#367405: command-line completion backspaces instead

[myon]

Hi,

the following was submitted as Debian bug #367405:

---

Date: Mon, 15 May 2006 14:37:40 -0400
From: Chung-chieh Shan <ken@digitas.harvard.edu>
Reply-To: Chung-chieh Shan <ken@digitas.harvard.edu>,
	367405@bugs.debian.org
To: Debian Bug Tracking System <submit@bugs.debian.org>
Subject: Bug#367405: mutt: command-line completion backspaces instead

Hello,

When I type "|cat >/usr/local/bi" at Mutt's index screen and hit Tab, I expect Mutt to add "n" or "n/" to the command line. Instead, it removes "/bi" from the command line. (To reproduce this bug, there must be no space after ">".)

Thanks,

Ken

---

Christoph

[me]

I have confirmed this behavior and will fix it. There seem to be two problems: 1) the complete function in _mutt_enter_string() looks backward for the first space character, which means in the input used in the bug report, the greater-than symbol is included in the pathname, and 2) mutt_complete() destructively alters the string even if it can't complete the pathname.

[me]

Changeset [83440e7b2e52] fixes part of this problem. The trailing component of the pathname is no longer removed. Completion still does not work, but that will be addressed in a separate changeset.

[me]

[097deb080af0] changes the "complete" function to consider shell characters as word boundaries (previously only space was considered as such).