file_search: Change return type of ff_wc_equal to bool.

This makes sense since the function returns only TRUE or FALSE.
This commit is contained in:
Jurica Bradaric
2016-02-09 23:12:33 +01:00
parent c9898e0ec3
commit b86553c7ad

View File

@@ -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;
}
}
}