file_search: Change return type of ff_wc_equal to bool.

This makes sense since the function returns only TRUE or FALSE.

1 files changed, 18 insertions, 17 deletions

diff --git a/src/nvim/file_search.c b/src/nvim/file_search.c
index 7b1ad85faa..9ee1aa5359 100644
--- a/src/nvim/file_search.c
+++ b/src/nvim/file_search.c
@@ -1046,27 +1046,27 @@ static ff_visited_list_hdr_T *ff_get_visited_list(char_u *filename, ff_visited_l
   return retptr;
 }
 
-/*
- * check if two wildcard paths are equal. Returns TRUE or FALSE.
- * They are equal if:
- *  - both paths are NULL
- *  - they have the same length
- *  - char by char comparison is OK
- *  - the only differences are in the counters behind a '**', so
- *    '**\20' is equal to '**\24'
- */
-static int ff_wc_equal(char_u *s1, char_u *s2)
+// Check if two wildcard paths are equal.
+// They are equal if:
+//  - both paths are NULL
+//  - they have the same length
+//  - char by char comparison is OK
+//  - the only differences are in the counters behind a '**', so
+//    '**\20' is equal to '**\24'
+static bool ff_wc_equal(char_u *s1, char_u *s2)
 {
   int c1 = NUL;
   int c2 = NUL;
   int prev1 = NUL;
   int prev2 = NUL;
 
-  if (s1 == s2)
-    return TRUE;
+  if (s1 == s2) {
+    return true;
+  }
 
-  if (s1 == NULL || s2 == NULL)
-    return FALSE;
+  if (s1 == NULL || s2 == NULL) {
+    return false;
+  }
 
   for (int i = 0, j = 0; s1[i] != NUL && s2[j] != NUL;) {
     c1 = PTR2CHAR(s1 + i);
@@ -1113,10 +1113,11 @@ static int ff_check_visited(ff_visited_T **visited_list, char_u *fname, char_u *
     if ((url && fnamecmp(vp->ffv_fname, ff_expand_buffer) == 0)
         || (!url && vp->file_id_valid
             && os_fileid_equal(&(vp->file_id), &file_id))) {
-      /* are the wildcard parts equal */
-      if (ff_wc_equal(vp->ffv_wc_path, wc_path) == TRUE)
-        /* already visited */
+      // are the wildcard parts equal
+      if (ff_wc_equal(vp->ffv_wc_path, wc_path)) {
+        // already visited
         return FAIL;
+      }
     }
   }