<div dir="ltr"><div><div><div><div>Hi Even,<br><br><a href="http://trac.osgeo.org/gdal/ticket/5129">http://trac.osgeo.org/gdal/ticket/5129</a><br><br></div>Yes I have recently updated the patch with the changes you and Frank suggested to GetValueAsString so the returned pointer is temporary. I have also changed the definition of GetDefaultRAT to return a non-const pointer and updated relevant drivers. <br>

<br></div>I'm actually just doing some final testing on my HFA patches and hope to have something to you in the next few hours. These will rely on the first ones being applied obviously. Up to you whether you apply them all at once, or as I submit them. Any feedback on the currently attached patch would be great.<br>
<br></div>Thanks,<br></div>Sam.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 21 June 2013 08:47, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@mines-paris.org" target="_blank">even.rouault@mines-paris.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Sam,<br>
<br>
I see you've attached recently a patch in<br>
<a href="http://trac.osgeo.org/gdal/wiki/rfc40_enhanced_rat_support" target="_blank">http://trac.osgeo.org/gdal/wiki/rfc40_enhanced_rat_support</a> that does the<br>
refactoring of GDALRasterAttributeTable vs GDALTableRasterAttributeTable. Do<br>
you intend it to be committed in trunk before tackling the implementation in<br>
the HFA driver ?<br>
<br>
I'd suggest you open a ticket in Trac to follow more easily the implementation<br>
of RFC 40.<br>
<br>
Best regards,<br>
<br>
Even<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Geospatial professional services<br>
<a href="http://even.rouault.free.fr/services.html" target="_blank">http://even.rouault.free.fr/services.html</a><br>
</font></span></blockquote></div><br></div>