mirrors/foot
1
0
Fork 0
mirror of https://codeberg.org/dnkl/foot.git synced 2026-02-05 04:06:08 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

config: no need to free 'line' between each call to getline()

Browse source 
getline() will re-use the allocated line if it large enough, or resize
it otherwise. Thus there's no need to free it and set it to NULL
between each call.
This commit is contained in:
Daniel Eklöf 2020-03-02 18:48:29 +01:00
parent 1f33b4a292
commit 1b2050de7b
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
1 changed files with 6 additions and 14 deletions
Download patch file Download diff file Expand all files Collapse all files

20
config.c
View file

@ -428,21 +428,20 @@ parse_config_file(FILE *f, struct config *conf, const char *path)
#endif
unsigned lineno = 0;
char *_line = NULL;
size_t count = 0;
while (true) {
errno = 0;
lineno++;
_line = NULL;
size_t count = 0;
ssize_t ret = getline(&_line, &count, f);
if (ret < 0) {
free(_line);
if (errno != 0) {
LOG_ERRNO("failed to read from configuration");
return false;
goto err;
}
break;
}
@ -461,10 +460,8 @@ parse_config_file(FILE *f, struct config *conf, const char *path)
}
/* Empty line, or comment */
if (line[0] == '\0' || line[0] == '#') {
free(_line);
if (line[0] == '\0' || line[0] == '#')
continue;
}
/* Check for new section */
if (line[0] == '[') {
@ -487,7 +484,6 @@ parse_config_file(FILE *f, struct config *conf, const char *path)
goto err;
}
free(_line);
continue;
}
@ -510,7 +506,6 @@ parse_config_file(FILE *f, struct config *conf, const char *path)
goto err;
}
free(_line);
continue;
}
@ -521,10 +516,8 @@ parse_config_file(FILE *f, struct config *conf, const char *path)
assert(!isspace(*(value + strlen(value) - 1)));
}
if (key[0] == '#') {
free(_line);
if (key[0] == '#')
continue;
}
LOG_DBG("section=%s, key='%s', value='%s'",
section_names[section], key, value);
@ -534,10 +527,9 @@ parse_config_file(FILE *f, struct config *conf, const char *path)
if (!section_parser(key, value, conf, path, lineno))
goto err;
free(_line);
}
free(_line);
return true;
err: