[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Small fix in `shell--unquote&requote-argument' - please review
From: |
Stefan Monnier |
Subject: |
Re: Small fix in `shell--unquote&requote-argument' - please review |
Date: |
Thu, 01 Sep 2016 09:39:34 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) |
>>> 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.
Hmm... would you be able to artificially construct or describe a failing
case, then? Part of the issue is that I don't remember the code well
enough, so while your description makes some sense, I'm having
difficulty understanding really what the change does.
Stefan