From ddb4b922a45233948407b4cf9811fc33e1526405 Mon Sep 17 00:00:00 2001 From: Martin Heistermann Date: Fri, 4 Apr 2025 17:22:13 +0200 Subject: [PATCH] Fix warning by not calling now-empty and deprecated retain(). ``` OpenMesh/Core/Utils/PropertyManager.hh:393:16: warning: 'retain' is deprecated [-Wdeprecated-declarations] 393 | pm.retain(); | ^ OpenMesh/Core/Utils/PropertyManager.hh:325:9: note: 'retain' has been explicitly marked deprecated here 325 | OM_DEPRECATED("retain no longer has any effect. Instead, named properties are always retained, while unnamed ones are not.") ``` --- src/OpenMesh/Core/Utils/PropertyManager.hh | 1 - 1 file changed, 1 deletion(-) diff --git a/src/OpenMesh/Core/Utils/PropertyManager.hh b/src/OpenMesh/Core/Utils/PropertyManager.hh index 71452301..4bab30cc 100644 --- a/src/OpenMesh/Core/Utils/PropertyManager.hh +++ b/src/OpenMesh/Core/Utils/PropertyManager.hh @@ -390,7 +390,6 @@ class PropertyManager { const PROP_VALUE &init_value) { const bool exists = propertyExists(mesh, propname); PropertyManager pm(mesh, propname, exists); - pm.retain(); if (!exists) pm.set_range(begin, end, init_value); return std::move(pm);