fix exception in Queries.QueryCell (backported from 5.2.15)

stefanmaierhofer · stefanmaierhofer · commit 2da3a2052f0b · 2023-01-03T09:22:27.000+01:00
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
@@ -4,6 +4,7 @@ on:
     branches:
       - master
       - newbuild
+      - branch_5.2.10
     paths:
       - RELEASE_NOTES.md
       - .github/workflows/publish.yml
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -1,3 +1,6 @@
+### 5.2.10.1
+- fix exception in Queries.QueryCell (backported from 5.2.15)
+
 ### 5.2.10
 - LASzip importer now supports LAS 1.4
 - fix GenericChunk.Empty
diff --git a/src/Aardvark.Geometry.PointSet/Queries/QueriesCells.cs b/src/Aardvark.Geometry.PointSet/Queries/QueriesCells.cs
@@ -168,7 +168,7 @@ public static CellQueryResult QueryCell(this IPointCloudNode root, Cell cell)
 
             CellQueryResult QueryCellRecursive(IPointCloudNode n)
             {
-                if (n.Cell == cell)
+                if (n.Cell == cell || n.IsLeaf)
                 {
                     // found!
                     return new CellQueryResult(root, cell, n);

Original file line number	Diff line number	Diff line change
`@@ -168,7 +168,7 @@ public static CellQueryResult QueryCell(this IPointCloudNode root, Cell cell)`
`168`	`168`
`169`	`169`	`CellQueryResult QueryCellRecursive(IPointCloudNode n)`
`170`	`170`	`{`
`171`		`- if (n.Cell == cell)`
	`171`	`+ if (n.Cell == cell \|\| n.IsLeaf)`
`172`	`172`	`{`
`173`	`173`	`// found!`
`174`	`174`	`return new CellQueryResult(root, cell, n);`