chiark / gitweb /
cocci: use strempty() at more places
authorLennart Poettering <lennart@poettering.net>
Wed, 10 Jan 2018 16:11:19 +0000 (17:11 +0100)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:50:03 +0000 (07:50 +0200)
This shortens the code by a few lines.

src/basic/fileio.c
src/basic/string-util.h

index 8906c0b829d9858f9ae2902991064c740bb55a1e..c16ef3b1e0e154e9212e4d917a15d653009a14b9 100644 (file)
@@ -1231,8 +1231,7 @@ int tempfn_xxxxxx(const char *p, const char *extra, char **ret) {
         if (!filename_is_valid(fn))
                 return -EINVAL;
 
-        if (!extra)
-                extra = "";
+        extra = strempty(extra);
 
         t = new(char, strlen(p) + 2 + strlen(extra) + 6 + 1);
         if (!t)
@@ -1265,8 +1264,7 @@ int tempfn_random(const char *p, const char *extra, char **ret) {
         if (!filename_is_valid(fn))
                 return -EINVAL;
 
-        if (!extra)
-                extra = "";
+        extra = strempty(extra);
 
         t = new(char, strlen(p) + 2 + strlen(extra) + 16 + 1);
         if (!t)
@@ -1307,8 +1305,7 @@ int tempfn_random_child(const char *p, const char *extra, char **ret) {
                         return r;
         }
 
-        if (!extra)
-                extra = "";
+        extra = strempty(extra);
 
         t = new(char, strlen(p) + 3 + strlen(extra) + 16 + 1);
         if (!t)
index 5ee7442d9e7c4dc847d88340b5f4651f0f3398eb..7cbfef725f4e3a29ba02ba8d6d387480027aa8e2 100644 (file)
@@ -52,15 +52,15 @@ static inline bool streq_ptr(const char *a, const char *b) {
 }
 
 static inline const char* strempty(const char *s) {
-        return s ? s : "";
+        return s ?: "";
 }
 
 static inline const char* strnull(const char *s) {
-        return s ? s : "(null)";
+        return s ?: "(null)";
 }
 
 static inline const char *strna(const char *s) {
-        return s ? s : "n/a";
+        return s ?: "n/a";
 }
 
 static inline bool isempty(const char *p) {