[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
0001-maint-add-memrchr-gnulib-module.patch
Description: Binary data
0002-regexp-comment-begline-endline-optimizations.patch
Description: Binary data
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Norihiro Tanaka, 2016/10/04
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Assaf Gordon, 2016/10/05
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Norihiro Tanaka, 2016/10/06
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually,
Assaf Gordon <=
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Norihiro Tanaka, 2016/10/18
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Assaf Gordon, 2016/10/20
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Norihiro Tanaka, 2016/10/21
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Assaf Gordon, 2016/10/30
- bug#24615: [PATCH] sed: handle the patterns which consist of ^ or $ manually, Norihiro Tanaka, 2016/10/30