Fix logic.

PDAL · Apr 8, 2021 · 66424f5 · 66424f5
1 parent d02d618
commit 66424f5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/pdal/Stage.hpp b/pdal/Stage.hpp
@@ -396,8 +396,8 @@ class PDAL_DLL Stage
       Potentially split a point view into keeps and skips based on a where clause.
 
       \param view  Point view to split.
-      \param keep  PointView to hold the kept points. 
-      \param skip  PointView to hold the skipped points. 
+      \param keep  PointView to hold the kept points.
+      \param skip  PointView to hold the skipped points.
     */
     void splitView(const PointViewPtr& view, PointViewPtr& keep, PointViewPtr& skip);
 

diff --git a/pdal/Streamable.cpp b/pdal/Streamable.cpp
@@ -248,7 +248,9 @@ void Streamable::execute(StreamPointTable& table,
             for (PointId idx = 0; idx < pointLimit; idx++)
             {
                 point.setPointId(idx);
-                if (table.skip(idx) || (!where || !where->eval(point)))
+                if (table.skip(idx))
+                    continue;
+                if (where && !where->eval(point))
                     continue;
                 if (!s->processOne(point))
                     table.setSkip(idx);