From b07d9df5fc905abebdbda833daf5dd6b8cf08688 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?P=C5=99emysl=20Janouch?=
Date: Sat, 1 Oct 2016 04:10:44 +0200
Subject: [PATCH] Simplify resolve_relative_filename_generic()
---
liberty.c | 20 ++++++--------------
1 file changed, 6 insertions(+), 14 deletions(-)
diff --git a/liberty.c b/liberty.c
index 4980a53..e87d47c 100644
--- a/liberty.c
+++ b/liberty.c
@@ -3187,29 +3187,21 @@ static char *
resolve_relative_filename_generic
(struct str_vector *paths, const char *tail, const char *filename)
{
- struct str file;
- str_init (&file);
-
- char *result = NULL;
for (unsigned i = 0; i < paths->len; i++)
{
// As per XDG spec, relative paths are ignored
if (*paths->vector[i] != '/')
continue;
- str_reset (&file);
- str_append_printf (&file, "%s/%s%s", paths->vector[i], tail, filename);
+ char *file = xstrdup_printf
+ ("%s/%s%s", paths->vector[i], tail, filename);
struct stat st;
- if (!stat (file.str, &st))
- {
- result = str_steal (&file);
- break;
- }
+ if (!stat (file, &st))
+ return file;
+ free (file);
}
-
- str_free (&file);
- return result;
+ return NULL;
}
static void