summaryrefslogtreecommitdiff
path: root/components/num_files.c
diff options
context:
space:
mode:
authorAaron Marcher <me@drkhsh.at>2018-05-06 22:28:56 +0200
committerAaron Marcher <me@drkhsh.at>2018-05-06 22:28:56 +0200
commit06cee3ffb6e76a7fd7430e4f8b97b1b6c4d3d18b (patch)
tree2cc3cd4e654671d509dceb572c7c72d3bfde5e9d /components/num_files.c
parent1bf48f9f1e1d900c9e1d015eebccb09dc5ef1706 (diff)
Fix coding style
- Use block for single statement ifs - Keep lines to reasonable length (current debate as to reasonable) - When functions return -1 for error test against 0 not -1 - Do not indent cases another level - Do not test against NULL and 0 explicitly - Use tabs for indentation, use spaces for alignment
Diffstat (limited to 'components/num_files.c')
-rw-r--r--components/num_files.c7
1 files changed, 4 insertions, 3 deletions
diff --git a/components/num_files.c b/components/num_files.c
index acf9dfa..1a918ac 100644
--- a/components/num_files.c
+++ b/components/num_files.c
@@ -13,14 +13,15 @@ num_files(const char *dir)
DIR *fd;
int num = 0;
- if ((fd = opendir(dir)) == NULL) {
+ if (!(fd = opendir(dir))) {
fprintf(stderr, "opendir '%s': %s\n", dir, strerror(errno));
return NULL;
}
- while ((dp = readdir(fd)) != NULL) {
- if (!strcmp(dp->d_name, ".") || !strcmp(dp->d_name, ".."))
+ while ((dp = readdir(fd))) {
+ if (!strcmp(dp->d_name, ".") || !strcmp(dp->d_name, "..")) {
continue; /* skip self and parent */
+ }
num++;
}