Fri Jul 26 03:46:08 1996 Ulrich Drepper <drepper@cygnus.com>
authorroland <roland>
Fri, 26 Jul 1996 04:32:54 +0000 (04:32 +0000)
committerroland <roland>
Fri, 26 Jul 1996 04:32:54 +0000 (04:32 +0000)
* locale/setlocale.c (new_composite_name): Little optimization.
Use return value of `memcpy'.

locale/setlocale.c

index eab1a33..a32fab3 100644 (file)
@@ -53,12 +53,12 @@ static const struct locale_data * *const _nl_current[] =
 /* Array indexed by category of pointers to _nl_C_CATEGORY slots.
    Elements are zero for categories whose data is never used.  */
 const struct locale_data *const _nl_C[] =
-{
+  {
 #define DEFINE_CATEGORY(category, category_name, items, a, b, c, d) \
-  [category] = &_nl_C_##category,
+    [category] = &_nl_C_##category,
 #include "categories.def"
 #undef DEFINE_CATEGORY
-};
+  };
 
 
 /* Define an array of category names (also the environment variable names),
@@ -169,11 +169,8 @@ new_composite_name (int category, char *newnames[LC_ALL])
        return (char *) _nl_C_name;
 
       new = malloc (last_len + 1);
-      if (new == NULL)
-       return NULL;
 
-      memcpy (new, newnames[0], last_len + 1);
-      return new;
+      return new == NULL ? NULL : memcpy (new, newnames[0], last_len + 1);
     }
 
   new = malloc (cumlen);