<div dir="ltr"><div>Hi all,</div><div><br></div><div>Can we discuss cvs_MD5* from <a href="https://trac.osgeo.org/gdal/changeset/41086">https://trac.osgeo.org/gdal/changeset/41086</a> ?</div><div><br></div><div>I very much appreciate the work that bishop is doing and I hate to slow down a contributor<br></div><div><br></div><div>I am really worried about code like this going into GDAL, especially into port/<br></div><div><br></div><div><div>But my worries are:</div></div><div><br></div><div>- this in no way conforms to other code in GDAL</div><div>- has the potential to collide with other code that imports this code</div><div>- it has a very awkward C style</div><div>- the commit message did not point to where it came from (at least it's not hard to guess)</div><div>- the file naming is different than (most) of the rest of the directory</div><div>- and...</div><div><br></div><div>Yes, we have other libraries included, but it seems like a road we don't want to go down</div><div><br></div><div>-kurt</div><div class="gmail_signature"></div>
</div>