Re: Small fix in `shell--unquote&requote-argument' - please review

[Filipp Gunbin]

Hi Stefan,

On 31/08/2016 09:27 -0400, Stefan Monnier wrote:

>> 1. match is always less than (length str), so I guess they meant
>> `((< (1+ match) (length qstr))'.
>> 2. If `string-match' searching for ending single quote failed,
>> `(match-string 0)' is still called - be careful not to do this.
>
> Do you have corresponding recipes to trigger the corresponding errors
> (so we could write tests)?  This part of my code is in dire need of
> tests, otherwise it's much too easy to introduce regressions.

Honestly, no, I found this when studying the code.

Filipp