Re: false error messages when doing replacements

Benno Schulenberg <[email protected]>
Newsgroups gmane.editors.nano.general
Message-ID <[email protected]>
Op 25-10-2017 om 21:06 schreef Benno Schulenberg:
> I don't know yet how to fix this, but it shouldn't be too hard.

A first attempt at a patch is attached.  I think it will do what
you want.  If you or someone else could test it and report your
findings, that would be great.

Oh...  I just realized that it should also suppress the message
"Search Wrapped" when doing replacements...  Not sure how to fix
that, though.

Benno

_______________________________________________
Help-nano mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/help-nano
show-notfound-only-when-needed.patch (text/x-patch, 1.2 KB)
diff --git a/src/search.c b/src/search.c
index 4ad15b85..008c2106 100644
--- a/src/search.c
+++ b/src/search.c
@@ -291,7 +291,6 @@ int findnextstr(const char *needle, bool whole_word_only, bool have_region,
 
 	/* If we're back at the beginning, then there is no needle. */
 	if (came_full_circle) {
-	    not_found_msg(needle);
 	    enable_waiting();
 	    return 0;
 	}
@@ -336,10 +335,8 @@ int findnextstr(const char *needle, bool whole_word_only, bool have_region,
 
     /* Ensure that the found occurrence is not beyond the starting x. */
     if (came_full_circle && ((!ISSET(BACKWARDS_SEARCH) && found_x > begin_x) ||
-			(ISSET(BACKWARDS_SEARCH) && found_x < begin_x))) {
-	not_found_msg(needle);
+			(ISSET(BACKWARDS_SEARCH) && found_x < begin_x)))
 	return 0;
-    }
 
     /* Set the current position to point at what we found. */
     openfile->current = line;
@@ -448,6 +445,8 @@ void go_looking(void)
     if (didfind == 1 && openfile->current == was_current &&
 		openfile->current_x == was_current_x)
 	statusbar(_("This is the only occurrence"));
+    else if (didfind == 0)
+	not_found_msg(last_search);
 
 #ifdef DEBUG
     statusline(HUSH, "Took: %.2f", (double)(clock() - start) / CLOCKS_PER_SEC);
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.