Do less stuff when sz is 0 in compat53_strerror().

1 files changed, 10 insertions, 10 deletions

diff --git a/c-api/compat-5.3.c b/c-api/compat-5.3.c
index eb62009..9b304c1 100644
--- a/c-api/compat-5.3.c
+++ b/c-api/compat-5.3.c
@@ -53,17 +53,17 @@
 static char* compat53_strerror (int en, char* buff, size_t sz) {
 #if COMPAT53_HAVE_STRERROR_R
   /* use strerror_r here, because it's available on these specific platforms */
-  if (sz > 0)
+  if (sz > 0) {
     buff[0] = '\0';
-  /* we don't care whether the GNU version or the XSI version is used: */
-  strerror_r(en, buff, sz);
-  if (sz == 0 || buff[0] != '\0')
-    return buff; /* strerror_r wrote into the buffer */
-  else
-    /* buffer is unchanged, so we probably have called GNU strerror_r which
-     * returned a static constant string. Chances are that strerror will
-     * return the same static constant string and therefore be thread-safe. */
-    return strerror(en);
+    /* we don't care whether the GNU version or the XSI version is used: */
+    strerror_r(en, buff, sz);
+    if (buff[0] == '\0')
+      /* buffer is unchanged, so we probably have called GNU strerror_r which
+       * returned a static constant string. Chances are that strerror will
+       * return the same static constant string and therefore be thread-safe. */
+      return strerror(en);
+  }
+  return buff; /* sz is 0 *or* strerror_r wrote into the buffer */
 #elif COMPAT53_HAVE_STRERROR_S
   /* for MSVC and other C11 implementations, use strerror_s since it's
    * provided by default by the libraries */