summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndrew Gregory <andrew.gregory.8@gmail.com>2013-07-22 08:46:49 +0200
committerAllan McRae <allan@archlinux.org>2013-08-21 03:06:41 +0200
commitab84249a58b0976c941860e48461956c57b67d10 (patch)
treeb4e311c78936db089ea502f3c9a0459cd9439101
parent26da037fd55040bcda834e2c95bf25e52f8a739e (diff)
downloadpacman-ab84249a58b0976c941860e48461956c57b67d10.tar.gz
pacman-ab84249a58b0976c941860e48461956c57b67d10.tar.xz
conf.c: pass _parse_directive as a callback
This will allow passing arbitrary key/value handlers. Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com> Signed-off-by: Allan McRae <allan@archlinux.org>
-rw-r--r--src/pacman/conf.c20
1 files changed, 12 insertions, 8 deletions
diff --git a/src/pacman/conf.c b/src/pacman/conf.c
index df8dead9..564231e2 100644
--- a/src/pacman/conf.c
+++ b/src/pacman/conf.c
@@ -841,8 +841,9 @@ cleanup:
}
static int _parse_directive(const char *file, int linenum, const char *name,
- char *key, char *value, struct section_t *section)
+ char *key, char *value, void *data)
{
+ struct section_t *section = data;
if(!key && !value) {
int ret = finish_section(section);
pm_printf(ALPM_LOG_DEBUG, "config: new section '%s'\n", name);
@@ -872,6 +873,9 @@ static int _parse_directive(const char *file, int linenum, const char *name,
return 0;
}
+typedef int (ini_parser_fn)(const char *file, int line, const char *section,
+ char *key, char *value, void *data);
+
/** The "real" parseconfig. Each "Include" directive will recall this method so
* recursion and stack depth are limited to 10 levels. The publicly visible
* parseconfig calls this with a NULL section argument so we can recall from
@@ -881,7 +885,7 @@ static int _parse_directive(const char *file, int linenum, const char *name,
* @param depth the current recursion depth
* @return 0 on success, 1 on failure
*/
-static int _parseconfig(const char *file, struct section_t *section,
+static int _parseconfig(const char *file, ini_parser_fn cb, void *data,
char **section_name, int depth)
{
FILE *fp = NULL;
@@ -936,7 +940,7 @@ static int _parseconfig(const char *file, struct section_t *section,
name = strdup(line + 1);
name[line_len - 2] = '\0';
- ret = _parse_directive(file, linenum, name, NULL, NULL, section);
+ ret = cb(file, linenum, name, NULL, NULL, data);
free(*section_name);
*section_name = name;
@@ -994,7 +998,7 @@ static int _parseconfig(const char *file, struct section_t *section,
for(gindex = 0; gindex < globbuf.gl_pathc; gindex++) {
pm_printf(ALPM_LOG_DEBUG, "config file %s, line %d: including %s\n",
file, linenum, globbuf.gl_pathv[gindex]);
- _parseconfig(globbuf.gl_pathv[gindex], section,
+ _parseconfig(globbuf.gl_pathv[gindex], cb, data,
section_name, depth + 1);
}
break;
@@ -1002,13 +1006,13 @@ static int _parseconfig(const char *file, struct section_t *section,
globfree(&globbuf);
continue;
}
- if((ret = _parse_directive(file, linenum, *section_name, key, value, section)) != 0) {
+ if((ret = cb(file, linenum, *section_name, key, value, data)) != 0) {
goto cleanup;
}
}
if(depth == 0) {
- ret = _parse_directive(NULL, 0, NULL, NULL, NULL, section);
+ ret = cb(NULL, 0, NULL, NULL, NULL, data);
}
cleanup:
@@ -1041,7 +1045,7 @@ int parseconfig(const char *file)
/* call the real parseconfig function with a null section & db argument */
pm_printf(ALPM_LOG_DEBUG, "parseconfig: options pass\n");
section.parse_options = 1;
- if((ret = _parseconfig(file, &section, &section_name, 0))) {
+ if((ret = _parseconfig(file, _parse_directive, &section, &section_name, 0))) {
return ret;
}
if((ret = setup_libalpm())) {
@@ -1050,7 +1054,7 @@ int parseconfig(const char *file)
/* second pass, repo section parsing */
pm_printf(ALPM_LOG_DEBUG, "parseconfig: repo pass\n");
section.parse_options = 0;
- return _parseconfig(file, &section, &section_name, 0);
+ return _parseconfig(file, _parse_directive, &section, &section_name, 0);
}
/* vim: set ts=2 sw=2 noet: */