[PATCH 10/10] pickaxe: simplify kwset loop in contains()
- From
René Scharfe <l.s.r@web.de>
- Date
- Mar 22, 2014, 17:16 UTC
- Message-ID
- <1395508560-19893-11-git-send-email-l.s.r@web.de>
- In-Reply-To
- <1395508560-19893-1-git-send-email-l.s.r@web.de>
Inlining the variable "found" actually makes the code shorter and easier to read.
Signed-off-by: Rene Scharfe <l.s.r@web.de> --- diffcore-pickaxe.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/diffcore-pickaxe.c b/diffcore-pickaxe.c index 70753d0..185f86b 100644 --- a/diffcore-pickaxe.c +++ b/diffcore-pickaxe.c @@ -94,13 +94,10 @@ static unsigned int contains(mmfile_t *mf, regex_t *regexp, kwset_t kws) while (sz) { struct kwsmatch kwsm; size_t offset = kwsexec(kws, data, sz, &kwsm); - const char *found; if (offset == -1) break; - else - found = data + offset; - sz -= found - data + kwsm.size[0]; - data = found + kwsm.size[0]; + sz -= offset + kwsm.size[0]; + data += offset + kwsm.size[0]; cnt++; } }
-- 1.9.1