diff options
author | Mattias Andrée <maandree@kth.se> | 2023-01-09 21:36:56 +0100 |
---|---|---|
committer | Mattias Andrée <maandree@kth.se> | 2023-01-09 21:36:56 +0100 |
commit | 0a42c64cddca69cf2c572d669c5639645d98437f (patch) | |
tree | 95f2c9da512b7bb960f863ac8319360fb4e06409 | |
parent | Print warnings (diff) | |
download | libfonts-0a42c64cddca69cf2c572d669c5639645d98437f.tar.gz libfonts-0a42c64cddca69cf2c572d669c5639645d98437f.tar.bz2 libfonts-0a42c64cddca69cf2c572d669c5639645d98437f.tar.xz |
Implement libfonts_parse_double__ and last part of libfonts_get_output_rendering_settings (section aliases)
Signed-off-by: Mattias Andrée <maandree@kth.se>
-rw-r--r-- | libfonts_get_output_rendering_settings.c | 29 | ||||
-rw-r--r-- | libfonts_parse_double__.c | 13 |
2 files changed, 36 insertions, 6 deletions
diff --git a/libfonts_get_output_rendering_settings.c b/libfonts_get_output_rendering_settings.c index b9c1efd..9a1e83f 100644 --- a/libfonts_get_output_rendering_settings.c +++ b/libfonts_get_output_rendering_settings.c @@ -17,6 +17,7 @@ getn(const char *file_part1, size_t file_part1_len, const char *file_part2, int in_the_section = 0; int in_a_section = 0; char *value; + char *aliased_to = NULL; unsigned int found_fields = 0; size_t lineno = 0; @@ -64,8 +65,10 @@ open_again: if (len < 0) { if (errno == EINTR) continue; + fail: free(buf); free(path); + free(aliased_to); close(fd); return -1; } @@ -89,7 +92,16 @@ open_again: line = &line[1]; len -= 2; line[len] = '\0'; - found |= in_the_section = !strcmp(line, name); + in_the_section = !strcmp(line, name); + if (aliased_to) { + if (in_the_section) { + warning(ctx, 0, "libfonts_get_output_rendering_settings", + "[%s] aliased to [%s] but has its own section", name, aliased_to); + } else { + in_the_section = !strcmp(line, aliased_to); + } + } + found |= in_the_section; found_fields = 0; } else if (!in_a_section) { @@ -98,7 +110,17 @@ open_again: warning(ctx, 0, "libfonts_get_output_rendering_settings", "bad line in %s at line %zu", path, lineno); continue; } - /* TODO aliases should be declarable above the first "[%s]" */ + if (!strcmp(line, name)) { + if (aliased_to) { + warning(ctx, 0, "libfonts_get_output_rendering_settings", + "new alias for [%s] in %s at line %zu", name, path, lineno); + free(aliased_to); + aliased_to = NULL; + } + aliased_to = strdup(value); + if (!aliased_to) + goto fail; + } } else if (in_the_section) { value = libfonts_confsplit__(line); @@ -110,7 +132,7 @@ open_again: if (!strcmp(line, CONFNAME)) {\ if (found_fields & (1U << INDEX)) {\ warning(ctx, 0, "libfonts_get_output_rendering_settings",\ - "duplicate definition in %s at line %zu", path, lineno); \ + "duplicate definition in %s at line %zu", path, lineno);\ }\ found_fields |= (1U << INDEX);\ if (!PARSER(&settings->CNAME, value)) {\ @@ -131,6 +153,7 @@ open_again: out: free(path); + free(aliased_to); return found; } diff --git a/libfonts_parse_double__.c b/libfonts_parse_double__.c index 2fb182c..9aba573 100644 --- a/libfonts_parse_double__.c +++ b/libfonts_parse_double__.c @@ -6,9 +6,16 @@ int libfonts_parse_double__(double *outp, const char *value) { - (void) outp; - (void) value; - return 0; /* TODO implement */ + char *end; + int saved_errno = errno; + errno = 0; + *outp = strtod(value, &end); + if (errno || *end || isblank(*value)) { + errno = saved_errno; + return 0; + } + errno = saved_errno; + return 1; } |