Brian,<div><br></div><div>I cannot say much about the style tables. But, I would like to know about the constants for the TestCapability() methods.<br><br><div class="gmail_quote">On Thu, Mar 25, 2010 at 9:09 PM, brian <span dir="ltr"><<a href="mailto:rush@winkey.org">rush@winkey.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
<br>
I have two issues with the datasource and layer methods to get and set<br>
style tables.<br>
<br>
1) The GetStyleTable(), SetStyleTableDirectly(), and SetStyletable()<br>
methods are not virtual therefore only the memory driver can use them.<br>
<br>
2) The SetStyleTableDirectly() and SetStyletable() methods both return<br>
void. it is plausable that a datasource or layer could not support them.<br>
<br>
The API changes i am proposing are.<br>
<br>
1) make the GetStyleTable(), SetStyleTableDirectly(), and<br>
SetStyletable() methods virtual.<br>
<br>
2) make the SetStyleTableDirectly() and SetStyletable() methods return<br>
OGRErr.<br>
<br>
If a layer or datasource does not have a style table the GetStyleTable()<br>
method would already return NULL.<br>
<br>
I am also wondering if it makes any sence to add a couple of constants<br>
for the TestCapability() methods.<br>
<br>
Any feedback would be appreciated.<br>
<br>
Brian<br>
<br>
<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>
</blockquote></div><br><br clear="all"><br>-- <br>Best regards,<br>Chaitanya kumar CH.<br>/tʃaɪθənjə/ /kʊmɑr/ <br>+91-9848167848<br>17.2416N 80.1426E<br>
</div>