<div dir="ltr">+1 from a non-voting person.  It will definitely make testing a lot cleaner / easier without having to run a separate binary to get most of the testing done.  There is also an enormous amount of python code that runs os.system / subprocess.  That code is very error prone / a pain to maintain.<div><br></div><div>General thoughts / questions:  (I only skimmed, so apologies for dumb comments)</div><div><br></div><div>- Will this stuff be thread safe?</div><div>- Is it possible to have an optional ptr to an error return?  This would make thread safety easier.  Or is that *bUsageError?</div><div>- If the argument parsing to options data structure was also a part of the library, we could test the arg parsing in C++ without having to shell out to a program.</div><div>- Can we get away from 0/FALSE and TRUE/1 for bools in in C++.  e.g. psOptions->bQuiet</div><div><br></div><div>-kurt </div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 26, 2015 at 12:56 PM, Kyle Shannon <span dir="ltr"><<a href="mailto:kyle@pobox.com" target="_blank">kyle@pobox.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Aug 26, 2015 at 1:55 PM, Even Rouault<br>
<div><div class="h5"><<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>> wrote:<br>
> Le mercredi 26 août 2015 21:44:10, Kyle Shannon a écrit :<br>
>> Hi,<br>
>><br>
>> On Wed, Aug 26, 2015 at 12:27 PM, Even Rouault<br>
>><br>
>> <<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>> wrote:<br>
>> > Hi,<br>
>> ><br>
>> > Summer of code 2015 being finished now, Faza's work now include<br>
>> > librarification of gdalinfo, gdal_translate, gdalwarp and ogr2ogr. Faza<br>
>> > will continue working on some other utilities.<br>
>><br>
>> Which other utilities are being considered?  Has there been any<br>
>> discussion or thoughts on this?<br>
><br>
> I think we discussed with Faza about pursuing with gdalbuildvrt. gdaldem would<br>
> probably be interesting too. Anyone interested can contribute to the effort.<br>
><br>
>><br>
>> > We'd be happy to hear about your comments, especially on the API. So<br>
>> > speak now please.<br>
>><br>
>> The API looks good to me.<br>
>><br>
>> How does documentation work for utilities now?<br>
><br>
> You mean for the binaries ? Well, it is unchanged. Mainly in<br>
> gdal_utilities.dox, and for some of them (gdalwarp at least), in the .cpp.<br>
><br>
>> I guess changes should<br>
>> be documented in the *_lib.cpp or *_lib.h and also gdal_utilities.dox,<br>
>> similar to how the utilities closely coupled with certain alg/ APIs<br>
>> (gdalgrid for example).<br>
>><br>
>> > The overview of the work is there:<br>
>> > <a href="https://trac.osgeo.org/gdal/wiki/rfc59_utilities_as_a_library" rel="noreferrer" target="_blank">https://trac.osgeo.org/gdal/wiki/rfc59_utilities_as_a_library</a><br>
>><br>
>> If I am reading correctly, a separate lib will now be created.  Is<br>
>> this to keep the building cleaner? Is there a problem with just adding<br>
>> it to the libgdal C API?<br>
><br>
> The idea was to be a bit more modular, and avoid growing the core library.<br>
><br>
>><br>
>> > I'd like to call for a vote soon as I'd want that to be merged as soon as<br>
>> > possible, so it gets more widely tested and to avoid increasing merge<br>
>> > difficulties.<br>
>> ><br>
>> > Even<br>
>> ><br>
>> > --<br>
>> > Spatialys - Geospatial professional services<br>
>> > <a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><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" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a><br>
>><br>
>> Thanks.<br>
><br>
> --<br>
> Spatialys - Geospatial professional services<br>
> <a href="http://www.spatialys.com" rel="noreferrer" target="_blank">http://www.spatialys.com</a><br>
<br>
</div></div>Thanks Even...<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Kyle<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>
</div>