Changes between Initial Version and Version 1 of Ticket #830, comment 3


Ignore:
Timestamp:
Apr 12, 2017, 6:09:57 AM (7 years ago)
Author:
mloskot

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #830, comment 3

    initial v1  
    1 Kurt, this is classic lack of RAII in GEOS which. Switch to C++11 should help, replacing auto_ptr with unique_ptr and using unique_ptr in as many places as possible.
     1Kurt, this is classic lack of RAII in GEOS. Switch to C++11 should help, replacing auto_ptr with unique_ptr and using unique_ptr in as many places as possible.
    22
    33In this particular case, `vector<Geometry *> *polygons=new vector<Geometry *>(` could be replaced with intermediate vector on stack, once fully created, allocate new on heap and move `polygons` data into it before passing to `createMultiPolygon`.