]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
errno-list: errno_from_name() is pure but not const
authorMike Yuan <me@yhndnzj.com>
Mon, 26 May 2025 18:19:59 +0000 (20:19 +0200)
committerMike Yuan <me@yhndnzj.com>
Mon, 26 May 2025 18:24:43 +0000 (20:24 +0200)
Follow-up for 0c15577abe013e07dba47d5aac126a63ab2dfd33

src/basic/errno-list.c
src/basic/errno-list.h
src/basic/errno-util.h

index b39e15a7680a15ae828ae84d6845932b049d837d..5bfd8aa076a7030e8cb1706c1d2590d92ac0f742 100644 (file)
@@ -9,6 +9,19 @@ static const struct errno_name* lookup_errno(register const char *str,
 
 #include "errno-from-name.inc"
 
+int errno_from_name(const char *name) {
+        const struct errno_name *sc;
+
+        assert(name);
+
+        sc = lookup_errno(name, strlen(name));
+        if (!sc)
+                return -EINVAL;
+
+        assert(sc->id > 0);
+        return sc->id;
+}
+
 #if HAVE_STRERRORNAME_NP
 const char* errno_to_name(int id) {
         if (id == 0) /* To stay in line with our own impl */
@@ -17,7 +30,7 @@ const char* errno_to_name(int id) {
         return strerrorname_np(ABS(id));
 }
 #else
-#include "errno-to-name.inc"
+#  include "errno-to-name.inc"
 
 const char* errno_to_name(int id) {
 
@@ -30,16 +43,3 @@ const char* errno_to_name(int id) {
         return errno_names[id];
 }
 #endif
-
-int errno_from_name(const char *name) {
-        const struct errno_name *sc;
-
-        assert(name);
-
-        sc = lookup_errno(name, strlen(name));
-        if (!sc)
-                return -EINVAL;
-
-        assert(sc->id > 0);
-        return sc->id;
-}
index ce9516068d033b3350d9dab6d2342812fc250f67..9040211b17fa16b15126577a6e88d423309003e7 100644 (file)
@@ -4,8 +4,7 @@
 #include "forward.h"
 
 const char* errno_to_name(int id);
-
-int errno_from_name(const char *name) _const_;
+int errno_from_name(const char *name) _pure_;
 
 static inline bool errno_is_valid(int n) {
         return n > 0 && n <= ERRNO_MAX;
index fa53744f6140d57848776184c43d448dc578aae2..ea54378d789f0e37c9e27181674e5a05a138e8fb 100644 (file)
@@ -52,7 +52,6 @@ static inline int negative_errno(void) {
          * It will suppress bogus gcc warnings in case it assumes 'errno' might
          * be 0 and thus the caller's error-handling might not be triggered. */
         assert_return(errno > 0, -EINVAL);
-
         return -errno;
 }