debian/notmuch
1
0
Fork 0
mirror of https://git.notmuchmail.org/git/notmuch synced 2024-12-22 17:34:54 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

cli: for loop is more customary

Browse source 
With the happy day stop condition within the while, it was
confusing. Switch to the paradigm for loop. No functional changes.
This commit is contained in:
Jani Nikula 2014-01-19 22:32:25 +02:00 committed by David Bremner
parent 0b247cb411
commit f849b371cc
1 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
notmuch-new.c
View file

@ -704,8 +704,7 @@ count_files (const char *path, int *count, add_files_state_t *state)
char *next;
struct dirent **fs_entries = NULL;
int num_fs_entries = scandir (path, &fs_entries, 0, dirent_sort_inode);
int entry_type;
int i = 0;
int entry_type, i;
if (num_fs_entries == -1) {
fprintf (stderr, "Warning: failed to open directory %s: %s\n",
@ -713,11 +712,8 @@ count_files (const char *path, int *count, add_files_state_t *state)
goto DONE;
}
while (!interrupted) {
if (i == num_fs_entries)
break;
entry = fs_entries[i++];
for (i = 0; i < num_fs_entries && ! interrupted; i++) {
entry = fs_entries[i];
/* Ignore special directories to avoid infinite recursion.
* Also ignore the .notmuch directory and files/directories