renamed function to allocate and copy memory
authorHolger Schemel <info@artsoft.org>
Sat, 24 Feb 2024 13:03:58 +0000 (14:03 +0100)
committerHolger Schemel <info@artsoft.org>
Sat, 24 Feb 2024 13:03:58 +0000 (14:03 +0100)
src/game_bd/bd_bdcff.c
src/game_bd/bd_cave.c
src/game_bd/bd_caveobject.c
src/libgame/misc.c
src/libgame/misc.h

index 1a204a2fb17b26c888d796694fd157aa687258f6..8f2ee081e54ffc646a9008e1c3c690820d5d6b37 100644 (file)
@@ -1194,12 +1194,12 @@ boolean gd_caveset_load_from_bdcff(const char *contents)
        object.element = cave->initial_border;
        object.fill_element = cave->initial_border;
 
-       cave->objects = list_prepend(cave->objects, getMemCopy(&object, sizeof(object)));
+       cave->objects = list_prepend(cave->objects, get_memcpy(&object, sizeof(object)));
 
        object.x1 = 19;
        object.y1 = 0;    /* 19, as it is also the border */
 
-       cave->objects = list_prepend(cave->objects, getMemCopy(&object, sizeof(object)));    /* another */
+       cave->objects = list_prepend(cave->objects, get_memcpy(&object, sizeof(object)));    /* another */
       }
     }
   }
index a1a5d40959bda960ea062d8221e932edcdc288a1..9fa85405f86d1847fe5ed92480f72723bdf1d655 100644 (file)
@@ -506,7 +506,7 @@ gpointer gd_cave_map_dup_size(const GdCave *cave, const gpointer map, const int
     return NULL;
 
   rows = checked_malloc((cave->h) * sizeof(gpointer));
-  rows[0] = getMemCopy (maplines[0], cell_size * cave->w * cave->h);
+  rows[0] = get_memcpy (maplines[0], cell_size * cave->w * cave->h);
 
   for (y = 1; y < cave->h; y++)
     rows[y] = (char *)rows[0] + cell_size * cave->w * y;
@@ -605,7 +605,7 @@ void gd_cave_copy(GdCave *dest, const GdCave *src)
 
     dest->objects = NULL;    /* new empty list */
     for (iter = src->objects; iter != NULL; iter = iter->next) /* do a deep copy */
-      dest->objects = list_append(dest->objects, getMemCopy (iter->data, sizeof (GdObject)));
+      dest->objects = list_append(dest->objects, get_memcpy (iter->data, sizeof (GdObject)));
   }
 
   /* copy replays */
@@ -1400,7 +1400,7 @@ GdReplay *gd_replay_new_from_replay(GdReplay *orig)
 {
   GdReplay *rep;
 
-  rep = getMemCopy(orig, sizeof(GdReplay));
+  rep = get_memcpy(orig, sizeof(GdReplay));
 
   /* replicate dynamic data */
   rep->comment = getStringCopy(orig->comment);
index d76179068a6aa7037e84c1436f4a6e782d473071..568d2b3a793c966d9d67e04317a664851221b9c6 100644 (file)
@@ -313,7 +313,7 @@ GdObject *gd_object_new_from_string(char *str)
     {
       object.element = gd_get_element_from_string(elem0);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -327,7 +327,7 @@ GdObject *gd_object_new_from_string(char *str)
     {
       object.element = gd_get_element_from_string(elem0);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -341,7 +341,7 @@ GdObject *gd_object_new_from_string(char *str)
       object.type = GD_RECTANGLE;
       object.element = gd_get_element_from_string (elem0);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -360,14 +360,14 @@ GdObject *gd_object_new_from_string(char *str)
       object.element = gd_get_element_from_string (elem0);
       object.fill_element = gd_get_element_from_string (elem1);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     if (paramcount == 5)
     {
       object.element = object.fill_element = gd_get_element_from_string (elem0);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -387,7 +387,7 @@ GdObject *gd_object_new_from_string(char *str)
       object.type = GD_RASTER;
       object.element = gd_get_element_from_string (elem0);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -403,7 +403,7 @@ GdObject *gd_object_new_from_string(char *str)
       object.element = gd_get_element_from_string (elem0);
       object.fill_element = gd_get_element_from_string (elem1);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -418,7 +418,7 @@ GdObject *gd_object_new_from_string(char *str)
       object.fill_element = gd_get_element_from_string (elem0);
       object.element = gd_get_element_from_string (elem1);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -433,7 +433,7 @@ GdObject *gd_object_new_from_string(char *str)
       object.fill_element = gd_get_element_from_string (elem0);
       object.element = gd_get_element_from_string (elem1);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -463,7 +463,7 @@ GdObject *gd_object_new_from_string(char *str)
       object.element = gd_get_element_from_string (elem0);
       object.fill_element = gd_get_element_from_string (elem1);
 
-      return getMemCopy(&object, sizeof (GdObject));
+      return get_memcpy(&object, sizeof (GdObject));
     }
 
     return NULL;
@@ -519,7 +519,7 @@ GdObject *gd_object_new_from_string(char *str)
     if (l > 10 && l % 2 == 1)
       object.element = gd_get_element_from_string(words[l - 1]);
 
-    return getMemCopy(&object, sizeof (GdObject));
+    return get_memcpy(&object, sizeof (GdObject));
   }
 
   /* COPY PASTE OBJECT */
@@ -550,7 +550,7 @@ GdObject *gd_object_new_from_string(char *str)
     else
       Warn("invalid setting for copypaste flip property: %s", flip);
 
-    return getMemCopy(&object, sizeof(GdObject));
+    return get_memcpy(&object, sizeof(GdObject));
   }
 
   return NULL;
index 7450c3a35fd4942d4e0a19a039738b9c997b2650..ed22f2c09544dcdd8ed45818a44bf7349edd024f 100644 (file)
@@ -1084,24 +1084,6 @@ char *getBasePath(char *filename)
 }
 
 
-// ----------------------------------------------------------------------------
-// various memory functions
-// ----------------------------------------------------------------------------
-
-void *getMemCopy(const void *m, size_t size)
-{
-  void *m_copy;
-
-  if (m == NULL)
-    return NULL;
-
-  m_copy = checked_malloc(size);
-  memcpy(m_copy, m, size);
-
-  return m_copy;
-}
-
-
 // ----------------------------------------------------------------------------
 // various string functions
 // ----------------------------------------------------------------------------
@@ -1902,6 +1884,19 @@ void clear_mem(void *ptr, unsigned int size)
 #endif
 }
 
+void *get_memcpy(const void *m, size_t size)
+{
+  void *m_copy;
+
+  if (m == NULL)
+    return NULL;
+
+  m_copy = checked_malloc(size);
+  memcpy(m_copy, m, size);
+
+  return m_copy;
+}
+
 
 // ----------------------------------------------------------------------------
 // various helper functions
index 4548892c46febebc133f32650289e594ef3720b7..1f207c14e5c26efbdebd60286a483c813910ceb9 100644 (file)
@@ -168,8 +168,6 @@ char *getBaseName(char *);
 char *getBaseNamePtr(char *);
 char *getBaseNameNoSuffix(char *);
 
-void *getMemCopy(const void *, size_t);
-
 char *getStringCat2WithSeparator(const char *, const char *, const char *);
 char *getStringCat3WithSeparator(const char *, const char *, const char *, const char *);
 char *getStringCat2(const char *, const char *);
@@ -212,6 +210,7 @@ void *checked_calloc(unsigned int);
 void *checked_realloc(void *, unsigned int);
 void checked_free(void *);
 void clear_mem(void *, unsigned int);
+void *get_memcpy(const void *, size_t);
 
 void swap_numbers(int *, int *);
 void swap_number_pairs(int *, int *, int *, int *);