]> git.wh0rd.org Git - nano.git/commitdiff
Removing two redundant returns, and condensing six comments.
authorBenno Schulenberg <bensberg@justemail.net>
Thu, 17 Mar 2016 10:06:15 +0000 (10:06 +0000)
committerBenno Schulenberg <bensberg@justemail.net>
Thu, 17 Mar 2016 10:06:15 +0000 (10:06 +0000)
git-svn-id: svn://svn.savannah.gnu.org/nano/trunk/nano@5739 35c25a1d-7b9e-4130-9fde-d3aeb78583b8

ChangeLog
src/search.c

index 0e85cc407e55eef682af4c353630a209f63c5ec3..a578180295781040048bb0538670a3127d077542 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,7 @@
 2016-03-17  Benno Schulenberg  <bensberg@justemail.net>
        * src/search.c (do_research): Use the Search key bindings also during
        a Re-search, to enable cancelling.  This fixes Savannah bug #47159.
+       * src/search.c (do_replace): Remove two redundant returns.
 
 2016-03-14  Benno Schulenberg  <bensberg@justemail.net>
        * src/color.c (color_update): Don't dereference a possible NULL.
index 38c0bceb45a1f03d312f0a04a2c0a66009ece26c..dd85fe441dbeda4a367661074e54fe13e97c28b8 100644 (file)
@@ -424,15 +424,12 @@ void do_search(void)
 
     i = search_init(FALSE, FALSE);
 
-    if (i == -1)
-       /* Cancel, Go to Line, blank search string, or regcomp() failed. */
+    if (i == -1)       /* Cancelled, or some other exit reason. */
        search_replace_abort();
-    else if (i == -2)
-       /* Replace. */
+    else if (i == -2)  /* Do a replace instead. */
        do_replace();
 #if !defined(NANO_TINY) || defined(HAVE_REGEX_H)
-    else if (i == 1)
-       /* Case Sensitive, Backwards, or Regexp search toggle. */
+    else if (i == 1)   /* Toggled something. */
        do_search();
 #endif
 
@@ -848,16 +845,12 @@ void do_replace(void)
     }
 
     i = search_init(TRUE, FALSE);
-    if (i == -1) {
-       /* Cancel, Go to Line, blank search string, or regcomp() failed. */
+
+    if (i == -1)       /* Cancelled, or some other exit reason. */
        search_replace_abort();
-       return;
-    } else if (i == -2) {
-       /* No Replace. */
+    else if (i == -2)  /* Do a search instead. */
        do_search();
-       return;
-    } else if (i == 1)
-       /* Case Sensitive, Backwards, or Regexp search toggle. */
+    else if (i == 1)   /* Toggled something. */
        do_replace();
 
     if (i != 0)