[GRASS-dev] Vect_new_map_struct

Hi,

I have created new issue https://trac.osgeo.org/grass/ticket/2729
requesting to add Vect_new_map_struct, patch is supplied.

Radim

Hi Radim,
your patch was applied in r66048 as is. I think to make sure that the
Map_info substructures are correctly de-allocated, all de-allocation
events in the vector lib must be found and analysed, if the
corresponding pointer are set to NULL. Then you can implement a
recursive de-allocation of the Map_info structure in
Vect_destroy_map_struct().

in r66046 i have implemented point,line,boundary and area to WKT and
WKB converter, based on the GRASS GEOS support in
"lib/vector/Vlib/geos.c". This may be useful in the QGIS GRASS vector
iterator?

Please have a look at:
https://trac.osgeo.org/grass/browser/grass/trunk/lib/vector/Vlib/geos_to_wktb.c?rev=66046

Best regards
Soeren

2015-08-26 19:37 GMT+02:00 Radim Blazek <radim.blazek@gmail.com>:

Hi,

I have created new issue https://trac.osgeo.org/grass/ticket/2729
requesting to add Vect_new_map_struct, patch is supplied.

Radim
_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev

On Thu, Aug 27, 2015 at 3:45 PM, Sören Gebbert
<soerengebbert@googlemail.com> wrote:

Hi Radim,
your patch was applied in r66048 as is. I think to make sure that the
Map_info substructures are correctly de-allocated, all de-allocation
events in the vector lib must be found and analysed, if the
corresponding pointer are set to NULL. Then you can implement a
recursive de-allocation of the Map_info structure in
Vect_destroy_map_struct().

Thanks.

in r66046 i have implemented point,line,boundary and area to WKT and
WKB converter, based on the GRASS GEOS support in
"lib/vector/Vlib/geos.c". This may be useful in the QGIS GRASS vector
iterator?

Please have a look at:
https://trac.osgeo.org/grass/browser/grass/trunk/lib/vector/Vlib/geos_to_wktb.c?rev=66046

Definitely useful, it could be used to replace the same functions in
the provider, unfortunately we have to wait few years until we drop
GRASS 6 and GRASS 7.0 support.

Radim

Best regards
Soeren

2015-08-26 19:37 GMT+02:00 Radim Blazek <radim.blazek@gmail.com>:

Hi,

I have created new issue https://trac.osgeo.org/grass/ticket/2729
requesting to add Vect_new_map_struct, patch is supplied.

Radim
_______________________________________________
grass-dev mailing list
grass-dev@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/grass-dev