bug-sed
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#23635: possible bug in \c escape handling


From: Jim Meyering
Subject: bug#23635: possible bug in \c escape handling
Date: Sat, 28 May 2016 20:40:24 -0700

On Sat, May 28, 2016 at 7:31 PM, Assaf Gordon <address@hidden> wrote:
> Hello,
>
> Thank you for the review.
> Attached an improved version.
>
> Regarding when the bug was introduced (in the 'NEWS'), version 3.02 did not 
> support \c escapes, and version 4.0.6 had this bug (as does the first git 
> commit). I wrote:
>     [bug introduced in the sed-4.0.* releases]

Thanks for the quick update!
One more thing I noticed is that you use here docs
that interpolate. Just as I prefer to use single-quoted strings
most of the time, e.g., to avoid having to backslash-escape
every backslash, I prefer to use quoted here docs as in the
attached. Also, I prefer to space-delimit operators like '<', '<<', and '>'.
At least on one side.

Also, I removed a stray "EOF" after the final "Exit..." line.

Here's the proposed delta, on top of your patch:

Attachment: k.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]