[GRASS-dev] [GRASS GIS] #3235: Vect_cidx_find_all should not allow mixing GV_AREA with other geometries

[GRASS GIS]

#3235: Vect_cidx_find_all should not allow mixing GV_AREA with other geometries
-------------------------+-------------------------
 Reporter:  marisn       |      Owner:  grass-dev@…
     Type:  defect       |     Status:  new
 Priority:  major        |  Milestone:  7.2.1
Component:  LibVector    |    Version:  svn-trunk
 Keywords:               |        CPU:  Unspecified
 Platform:  Unspecified  |
-------------------------+-------------------------
 At the moment Vect_cidx_find_all accepts any type mask. This leads to
 unusable results in case if typemask allows both GV_LINES and GV_AREA in
 the output. Reason - Vect_cidx_find_next will return line ID for GV_LINES
 and area ID for GV_AREA. As both ID spaces are separate (thus might
 contain identical ID values), mixing them in a single output structure is
 a road to disaster.

 The fix - Vect_cidx_find_all should fail to accept type mask matching
 GV_LINES and GV_AREA (other combinations?) simultaneously.
 Other option - change Vect_cidx_find_all to modify two structs - one with
 line IDs and other with area IDs.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3235>
GRASS GIS <https://grass.osgeo.org>