Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: [PATCH] internal: quotestring: Drop the 'e' parameter, which no caller uses.



On Sep 9,  9:38am, Daniel Shahaf wrote:
} Subject: [PATCH] internal: quotestring: Drop the 'e' parameter, which no c
}
} - * If e is non-zero, the
} - * pointer it points to may point to a position in s and in e the position
} - * of the corresponding character in the quoted string is returned.

Hrm ... this has to have been used at one point e.g. by completion
to keep track of the cursor position while wrapping a substring in
quotes.  I wonder when that stopped being necessary.

Found a use in addmatch() back in zsh-2.4 which persists through 3.0.8
but is gone by 4.3.  Looks like uses started disappearing after 3.1.5
around about workers/5959 in mid-1999.  (The git import from CVS gets
very confused by CVS branches when you go that far back.)

Recall what I've said in a couple of other threads about it being too
hard for completion to intuit what a user means to have remain quoted
and what he expects completion to expand or otherwise treat as unquoted?
The need for this "e" argument seems to have gone away when we decided
to give up and do minimal quote interpretation/insertion.

I don't know what this patch accomplishes other than some dead code
removal, but I guess there's no remaining reason not to commit it.



Messages sorted by: Reverse Date, Date, Thread, Author