remove mapnik::point_datasource from c++ and mapnik.PointDatasource from python bindings

CHANGELOG

@@ -14,6 +14,8 @@ For a complete change history, see the SVN log.
 Mapnik 2.1.0
 ------------
 
+- Removed PointDatasource - use more robust MemoryDatasource instead (#1032)
+
 - SQLite - Added support for !intersects! token in sql subselects (#809) allow custom positioning of rtree spatial filter.
 
 - New CSV plugin - reads tabular files - autodetecting geo columns, newlines, and delimiters. Uses in-memory featureset for fast rendering and is not designed for large files (#902)

bindings/python/mapnik/__init__.py

@@ -667,7 +667,6 @@ __all__ = [
     'Path',
     'Parameter',
     'Parameters',
-    'PointDatasource',
     'PointSymbolizer',
     'PolygonPatternSymbolizer',
     'PolygonSymbolizer',

bindings/python/mapnik_datasource.cpp

@@ -36,7 +36,6 @@
 
 
 using mapnik::datasource;
-using mapnik::point_datasource;
 using mapnik::memory_datasource;
 using mapnik::layer_descriptor;
 using mapnik::attribute_descriptor;
@@ -182,11 +181,6 @@ void export_datasource()
     def("Describe",&describe);
     def("CreateDatasource",&create_datasource);
 
-    class_<point_datasource, bases<datasource>, boost::noncopyable>("PointDatasource", init<>())
-        .def("add_point",&point_datasource::add_point)
-        ;
-
-
     class_<memory_datasource, bases<datasource>, boost::noncopyable>("MemoryDatasource", init<>())
         .def("add_feature",&memory_datasource::push,
              "Adds a Feature:\n"

include/mapnik/memory_datasource.hpp

@@ -49,22 +49,7 @@ public:
 private:
     std::vector<feature_ptr> features_;
     mapnik::layer_descriptor desc_;
-}; 
-   
-// This class implements a simple way of displaying point-based data
-// TODO -- possible redesign, move into separate file
-//
-   
-class MAPNIK_DECL point_datasource : public memory_datasource {
-public:
-    point_datasource() :
-        feature_id_(1) {}
-    void add_point(double x, double y, const char* key, const char* value);  
-    inline int type() const { return datasource::Vector; }
-      
-private:
-    int feature_id_;
-};   
+};
 }
 
 #endif // MAPNIK_MEMORY_DATASOURCE_HPP

src/memory_datasource.cpp

@@ -112,20 +112,4 @@ void memory_datasource::clear()
     features_.clear();
 }
 
-// point_datasource
-
-void point_datasource::add_point(double x, double y, const char* key, const char* value)
-{
-    // FIXME
-
-    //feature_ptr feature(feature_factory::create(feature_id_));
-    //   ++feature_id_;
-    //   geometry_type * pt = new geometry_type(Point);
-    //   pt->move_to(x,y);
-    //   feature->add_geometry(pt);
-    //   transcoder tr("utf-8");
-    //   (*feature)[key] = tr.transcode(value);
-    //   this->push(feature);
-}
-
 }