Merge pull request #1201 from arcaneframework/dev/gg-move-checkresizearray-in-memoryutils

Move helper function 'checkResizeArray()' from 'MeshUtils' to 'MemoryUtils'

Author: grospelliergilles

arcane/src/arcane/core/MeshUtils.cc

@@ -1790,20 +1790,6 @@ findOneItem(IItemFamily* family, ItemUniqueId unique_id)
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
-Int64 MeshUtils::impl::
-computeCapacity(Int64 size)
-{
-  Int64 new_capacity = size * 2;
-  if (size > 5000000)
-    new_capacity = static_cast<Int64>(static_cast<double>(size) * 1.2);
-  else if (size > 500000)
-    new_capacity = static_cast<Int64>(static_cast<double>(size) * 1.5);
-  return new_capacity;
-}
-
-/*---------------------------------------------------------------------------*/
-/*---------------------------------------------------------------------------*/
-
 namespace MeshUtils::impl
 {
   //! Retourne le max des uniqueId() des entités de \a group

arcane/src/arcane/core/MeshUtils.h

@@ -15,6 +15,7 @@
 /*---------------------------------------------------------------------------*/
 
 #include "arcane/utils/FunctorUtils.h"
+#include "arcane/utils/MemoryUtils.h"
 
 #include "arcane/Item.h"
 
@@ -335,9 +336,10 @@ visitGroups(IMesh* mesh, const LambdaType& f)
 
 namespace impl
 {
-  //! Calcule une capacité adaptée pour une taille de \a size
-  extern "C++" ARCANE_CORE_EXPORT Int64
-  computeCapacity(Int64 size);
+  inline Int64 computeCapacity(Int64 size)
+  {
+    return Arcane::MemoryUtils::impl::computeCapacity(size);
+  }
 } // namespace impl
 
 /*---------------------------------------------------------------------------*/
@@ -361,15 +363,7 @@ namespace impl
 template <typename DataType> inline bool
 checkResizeArray(Array<DataType>& array, Int64 new_size, bool force_resize)
 {
-  Int64 s = array.largeSize();
-  if (new_size > s || force_resize) {
-    if (new_size > array.capacity()) {
-      array.reserve(impl::computeCapacity(new_size));
-    }
-    array.resize(new_size);
-    return true;
-  }
-  return false;
+  return Arcane::MemoryUtils::checkResizeArrayWithCapacity(array, new_size, force_resize);
 }
 
 /*---------------------------------------------------------------------------*/

arcane/src/arcane/utils/MemoryUtils.cc

@@ -1,11 +1,11 @@
 // -*- tab-width: 2; indent-tabs-mode: nil; coding: utf-8-with-signature -*-
 //-----------------------------------------------------------------------------
-// Copyright 2000-2022 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
+// Copyright 2000-2024 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
 // See the top-level COPYRIGHT file for details.
 // SPDX-License-Identifier: Apache-2.0
 //-----------------------------------------------------------------------------
 /*---------------------------------------------------------------------------*/
-/* MemoryUtils.cc                                              (C) 2000-2023 */
+/* MemoryUtils.cc                                              (C) 2000-2024 */
 /*                                                                           */
 /* Fonctions utilitaires de gestion mémoire.                                 */
 /*---------------------------------------------------------------------------*/
@@ -52,6 +52,20 @@ getAllocatorForMostlyReadOnlyData()
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
+Int64 MemoryUtils::impl::
+computeCapacity(Int64 size)
+{
+  Int64 new_capacity = size * 2;
+  if (size > 5000000)
+    new_capacity = static_cast<Int64>(static_cast<double>(size) * 1.2);
+  else if (size > 500000)
+    new_capacity = static_cast<Int64>(static_cast<double>(size) * 1.5);
+  return new_capacity;
+}
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+
 } // End namespace Arcane
 
 /*---------------------------------------------------------------------------*/

arcane/src/arcane/utils/MemoryUtils.h

@@ -1,11 +1,11 @@
 // -*- tab-width: 2; indent-tabs-mode: nil; coding: utf-8-with-signature -*-
 //-----------------------------------------------------------------------------
-// Copyright 2000-2022 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
+// Copyright 2000-2024 CEA (www.cea.fr) IFPEN (www.ifpenergiesnouvelles.com)
 // See the top-level COPYRIGHT file for details.
 // SPDX-License-Identifier: Apache-2.0
 //-----------------------------------------------------------------------------
 /*---------------------------------------------------------------------------*/
-/* MemoryUtils.h                                               (C) 2000-2023 */
+/* MemoryUtils.h                                               (C) 2000-2024 */
 /*                                                                           */
 /* Fonctions utilitaires de gestion mémoire.                                 */
 /*---------------------------------------------------------------------------*/
@@ -67,6 +67,44 @@ getAllocatorForMostlyReadOnlyData();
 /*---------------------------------------------------------------------------*/
 /*---------------------------------------------------------------------------*/
 
+namespace impl
+{
+  //! Calcule une capacité adaptée pour une taille de \a size
+  extern "C++" ARCANE_UTILS_EXPORT Int64
+  computeCapacity(Int64 size);
+} // namespace impl
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+/*!
+ * \brief Redimensionne un tableau en ajoutant une réserve de mémoire.
+ *
+ * Le tableau \a array est redimensionné uniquement si \a new_size est
+ * supérieure à la taille actuelle du tableau ou si \a force_resize est vrai.
+ *
+ * Si le tableau est redimensionné, on réserve une capacité supplémentaire
+ * pour éviter de réallouer à chaque fois.
+ *
+ * \retval true si un redimensionnement a eu lieu
+ * \retval false sinon
+ */
+template <typename DataType> inline bool
+checkResizeArrayWithCapacity(Array<DataType>& array, Int64 new_size, bool force_resize)
+{
+  Int64 s = array.largeSize();
+  if (new_size > s || force_resize) {
+    if (new_size > array.capacity()) {
+      array.reserve(impl::computeCapacity(new_size));
+    }
+    array.resize(new_size);
+    return true;
+  }
+  return false;
+}
+
+/*---------------------------------------------------------------------------*/
+/*---------------------------------------------------------------------------*/
+
 } // namespace Arcane::MemoryUtils
 
 /*---------------------------------------------------------------------------*/