Cleaner std::move'ing of positions during height sampling.

CesiumGS · Sep 30, 2024 · 2c2c031 · 2c2c031
1 parent 1a9ff05
commit 2c2c031
Showing 1 changed file with 25 additions and 22 deletions.
diff --git a/Source/CesiumRuntime/Private/Cesium3DTileset.cpp b/Source/CesiumRuntime/Private/Cesium3DTileset.cpp
@@ -144,28 +144,31 @@ void ACesium3DTileset::SampleHeightMostDetailed(
 
   size_t count = positions.size();
 
-  CesiumAsync::Future<Cesium3DTilesSelection::SampleHeightResult> future =
-      this->_pTileset
-          ? this->_pTileset->sampleHeightMostDetailed(positions)
-          : getAsyncSystem().createResolvedFuture(
-                Cesium3DTilesSelection::SampleHeightResult{
-                    std::move(positions),
-                    std::vector<bool>(count, false),
-                    {"Could not sample heights from tileset because it has not "
-                     "been created."}});
-
-  std::move(future)
-      .catchImmediately([positions = std::move(positions)](
-                            std::exception&& exception) mutable {
-        size_t count = positions.size();
-        return Cesium3DTilesSelection::SampleHeightResult{
-            std::move(positions),
-            std::vector<bool>(count, false),
-            {exception.what()}};
-      })
-      .thenImmediately([this, OnHeightsSampled = std::move(OnHeightsSampled)](
-                           Cesium3DTilesSelection::SampleHeightResult&&
-                               result) {
+  auto sampleHeights = [this, &positions]() mutable {
+    if (this->_pTileset) {
+      return this->_pTileset->sampleHeightMostDetailed(positions)
+          .catchImmediately([positions = std::move(positions)](
+                                std::exception&& exception) mutable {
+            std::vector<bool> sampleSuccess(positions.size(), false);
+            return Cesium3DTilesSelection::SampleHeightResult{
+                std::move(positions),
+                std::move(sampleSuccess),
+                {exception.what()}};
+          });
+    } else {
+      std::vector<bool> sampleSuccess(positions.size(), false);
+      return getAsyncSystem().createResolvedFuture(
+          Cesium3DTilesSelection::SampleHeightResult{
+              std::move(positions),
+              std::move(sampleSuccess),
+              {"Could not sample heights from tileset because it has not "
+               "been created."}});
+    }
+  };
+
+  sampleHeights().thenImmediately(
+      [this, OnHeightsSampled = std::move(OnHeightsSampled)](
+          Cesium3DTilesSelection::SampleHeightResult&& result) {
         if (!IsValid(this))
           return;