Abel,
You should not change the type of field types (or the number of fields)
once GetLayerDefn() has been called on the layer, after which the layer
definition must be considered as sealed (cf recent RFC
https://gdal.org/development/rfc/rfc97_feature_and_fielddefn_sealing.html),
until potential calls from the user to CreateField()/DeleteField()/etc.
which might unseal it. Changing field types during feature iteration
would break expectations from users of the GDAL API (included GDAL tools
such as ogr2ogr) and could cause nasty crashing bugs.
If you don't have a way to figure out the field type by looking at the
header of your dataset, then you need to do a first reading pass to
establish it (like done for example in the GeoJSON driver)
Even
Le 20/12/2023 à 20:58, Abel Pau via gdal-dev a écrit :
HI,
In the GetNewtRawFeature() I am processing all fields and a previusly
OFTString type field has to be changed into OFTStringList type. After
that, It’ll be probably a OFTString again.
I try to do that:
poFeature->GetDefnRef()->GetFieldDefn(nIField)->SetType(OFTStringList);
And when the destination driver gets it, the OFTStringList is again
OFTString. I’ve lost the property by magic.
The line that does that is poDstFeature->SetFDefnUnsafe(poDstFDefn);
in Translate() line5746 (probably).
So, there is a good way to change the type I’ve assigned in the
constructor by the same but a “List” instead in the GetNextRawFeature()?
Sorry for asking a question (again) but I’ve been a long time trying
to figure out what happens!
Thanks!
*Abel Pau Garcia*
*GIS developer*
https://www.creaf.cat/sites/default/files/creaf-signature.png
*a....@creaf.uab.cat* <mailto:a....@creaf.uab.cat>
*Let's chat on Teams!*
<https://teams.microsoft.com/l/chat/0/0?users=a....@creaf.uab.cat>
*Tel. +34 934814277*
https://www.creaf.cat/sites/default/files/so-en-signature.png
https://www.creaf.cat/sites/default/files/twitter-icon-signature.png
<https://twitter.com/CREAF_ecologia>https://www.creaf.cat/sites/default/files/linkedin-icon-signature.png
<https://www.linkedin.com/company/1363052?trk=tyah&trkInfo=clickedVertical:company,clickedEntityId:1363052,idx:2-1-2,tarId:1465807877789,tas:creaf>https://www.creaf.cat/sites/default/files/youtube-icon-signature.png
<https://www.youtube.com/c/creafecologia>https://www.creaf.cat/sites/default/files/instagram-icon-signature.png
<https://www.instagram.com/CREAF_ecologia/>
*www.creaf.cat* <http://www.creaf.cat>*| **http://blog.creaf.cat*
<http://blog.creaf.cat>
https://www.creaf.cat/sites/default/files/uab_logo_signatura.png
CREAF. Campus UAB. Edifici C. 08193 Bellaterra (Barcelona)
Before printing this electronic message, think about the environment.
http://www.creaf.uab.cat/_signatura/line.gif
_______________________________________________
gdal-dev mailing list
gdal-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/gdal-dev
--
http://www.spatialys.com
My software is free, but my time generally not.
_______________________________________________
gdal-dev mailing list
gdal-dev@lists.osgeo.org
https://lists.osgeo.org/mailman/listinfo/gdal-dev
