small cleanups to filters.tiler

PDAL · Mar 27, 2014 · f09a81f · f09a81f
1 parent 0827972
commit f09a81f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 16 deletions.
diff --git a/include/pdal/filters/Splitter.hpp b/include/pdal/filters/Splitter.hpp
@@ -51,13 +51,13 @@ class Stage;
 namespace filters
 {
 
-namespace iterators
-{
-namespace sequential
-{
-class Splitter;
-}
-}
+// namespace iterators
+// {
+// namespace sequential
+// {
+// class Splitter;
+// }
+// }
 
 class PDAL_DLL Splitter;
 
@@ -100,7 +100,6 @@ class PDAL_DLL Splitter : public pdal::Filter
     SET_STAGE_LINK("http://pdal.io/stages/filters.splitter.html")
 
     Splitter(Stage& prevStage, const Options&);
-    Splitter(Stage& prevStage);
 
     virtual void initialize();
     static Options getDefaultOptions();

diff --git a/src/filters/Splitter.cpp b/src/filters/Splitter.cpp
@@ -44,7 +44,7 @@ namespace pdal
 namespace filters
 {
 
-Splitter::Splitter(Stage& prevStage, const Options& options)
+Splitter::Splitter(Stage& prevStage, Options const& options)
     : pdal::Filter(prevStage, options)
     , m_leaf_size(0)
     , m_inverse_leaf_size(0)
@@ -60,12 +60,6 @@ Splitter::Splitter(Stage& prevStage, const Options& options)
     m_length = options.getValueOrDefault<boost::uint32_t>("length", 1000);
 }
 
-Splitter::Splitter(Stage& prevStage)
-    : pdal::Filter(prevStage, Options::none())
-{
-    return;
-}
-
 void Splitter::initialize()
 {
     Filter::initialize();
@@ -94,7 +88,10 @@ struct tile_point_idx
     boost::uint32_t tile_idx;
     boost::uint32_t point_idx;
 
-    tile_point_idx(boost::uint32_t tile_idx_, boost::uint32_t point_idx_) : tile_idx(tile_idx_), point_idx(point_idx_) {}
+    tile_point_idx( boost::uint32_t tile_idx_, 
+                    boost::uint32_t point_idx_) 
+        : tile_idx(tile_idx_)
+        , point_idx(point_idx_) {};
     bool operator < (const tile_point_idx &p) const
     {
         return (tile_idx < p.tile_idx);