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
commitee5ec756218c852385c5ba5ef0f75b47ba59ec39 (patch)
tree2cc3cd4e654671d509dceb572c7c72d3bfde5e9d /components/num_files.c
parent66a7fb16fc903ac7ef6ce1acdbb353f1072cd0ec (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++;
}