bug-sed
[Top][All Lists]
Advanced

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

bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manu


From: Assaf Gordon
Subject: bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually
Date: Mon, 17 Oct 2016 20:25:57 -0400

Hello Norihiro,

Thank you for the updates.
On top of the patches you've sent, I suggest these two minor additions. We can 
squash them into your patch before the final commit.
The first adds the required 'memrchr' gnulib module, which is not available in 
non-gnu libc.
The second adds few lines of comments for the begline/endline conditionals, as 
I feel the sed code in general is a bit opaque and hard to understand. Of 
course I'd appreciate if you review my comments and ensure they are true to the 
code.

The last thing I'd like to add (and hope to complete by the end of the week) is 
to add tests to cover the new code (especially the new code for dfa/-z 
interaction).
Once completed, we can squash it all into one patch and commit.

WDYT ?

regards,
 - assaf


Attachment: 0001-maint-add-memrchr-gnulib-module.patch
Description: Binary data

Attachment: 0002-regexp-comment-begline-endline-optimizations.patch
Description: Binary data



reply via email to

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