From 35bbbbacb64c49e9fed1a32d3044dca4b91fa523 Mon Sep 17 00:00:00 2001 From: Peter Stephenson Date: Sat, 26 Apr 2008 20:27:39 +0000 Subject: 24879: unpleasantness at the pattern search line duplication test club --- Src/Zle/zle_hist.c | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) (limited to 'Src/Zle/zle_hist.c') diff --git a/Src/Zle/zle_hist.c b/Src/Zle/zle_hist.c index 281de2e3b..ce3c284ca 100644 --- a/Src/Zle/zle_hist.c +++ b/Src/Zle/zle_hist.c @@ -1073,6 +1073,14 @@ doisearch(char **args, int dir, int pattern) * the above for the next pattern search. */ int revert_patpos = 0; + /* + * Another nasty feature related to the above. When + * we revert the position, we might advance the search to + * the same line again. When we do this the test for ignoring + * duplicates may trigger. This flag indicates that in this + * case it's OK. + */ + int dup_ok = 0; /* * savekeys records the unget buffer, so that if we have arguments * they don't pollute the input. @@ -1219,6 +1227,7 @@ doisearch(char **args, int dir, int pattern) * search started; see note above. */ revert_patpos = 0; + dup_ok = 1; he = quietgethist(hl = pat_hl); zt = GETZLETEXT(he); pos = pat_pos; @@ -1322,10 +1331,14 @@ doisearch(char **args, int dir, int pattern) hl = he->histnum; zt = GETZLETEXT(he); pos = (dir == 1) ? 0 : strlen(zt); - skip_line = isset(HISTFINDNODUPS) - ? !!(he->node.flags & HIST_DUP) - : !strcmp(zt, last_line); + if (dup_ok) + skip_line = 0; + else + skip_line = isset(HISTFINDNODUPS) + ? !!(he->node.flags & HIST_DUP) + : !strcmp(zt, last_line); } + dup_ok = 0; /* * If we matched above (t set), set the new line. * If we didn't, but are here because we are on a previous -- cgit 1.4.1