Martin,<br><br>I didn't follow all conclusions made earlier in this thread, but I don't think it makes much sense to enable/disable the special fields since those are derived from the attributes of the feature instead of fetching the values physically from the data source. For example OGR_STYLE corresponds to OGRFeature::GetStyleString() which can provide a meaningful value even if the styles are not handled/supported by a particular driver.<br>
<br>Assuming we only deal with the physical attributes it seems to me more reasonable to store a flag in OGRFieldDefn which could be set individually. In this regard each driver would do something like:<br><br><pre class="wiki">
for( int iField = 0; iField < poDefn->GetFieldCount(); iField++ )<br>{<br> if (poDefn->GetFieldDefn(iField)->IsIgnored())<br> continue;<br><br> // fetch field<br>}<br></pre><br><br>This would eliminate the requirement to maintain a separate array of the ignored/desired fields.<br>
<br><br>Best regards,<br><br>Tamas<br><br><br><br><br><div style="visibility: hidden; display: inline;" id="avg_ls_inline_popup"></div><style type="text/css">#avg_ls_inline_popup { position:absolute; z-index:9999; padding: 0px 0px; margin-left: 0px; margin-top: 0px; width: 240px; overflow: hidden; word-wrap: break-word; color: black; font-size: 10px; text-align: left; line-height: 13px;}</style><div class="gmail_quote">
2010/7/29 Martin Dobias <span dir="ltr"><<a href="http://wonder.sk">wonder.sk</a>@<a href="http://gmail.com">gmail.com</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Wed, Jul 21, 2010 at 6:59 PM, Martin Dobias <<a href="http://wonder.sk" target="_blank">wonder.sk</a>@<a href="http://gmail.com" target="_blank">gmail.com</a>> wrote:<br>
><br>
> So, in case there will be no further comments, I'll update the RFC to<br>
> match the API you've proposed.<br>
<br>
</div>Hi Frank,<br>
<br>
I've updated the RFC to reflect your suggestions:<br>
<div class="im"><br>
<a href="http://trac.osgeo.org/gdal/wiki/rfc29_desired_fields" target="_blank">http://trac.osgeo.org/gdal/wiki/rfc29_desired_fields</a><br>
<br>
</div>Given that there has been no further discussion, can we mark the RFC<br>
as proposed and start the voting?<br>
<div><div></div><div class="h5"><br>
Regards<br>
Martin<br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
</div></div></blockquote></div><br>