<div dir="auto">I have not looked in the wms code much, but my comment was not about adding an external dependency. It is about code quality in the core of GDAL.</div><div class="gmail_extra"><br><div class="gmail_quote">On Dec 20, 2017 6:07 AM, "Dmitry Baryshnikov" <<a href="mailto:bishop.dev@gmail.com">bishop.dev@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF">
    <p>Hi Kurt.</p>
    <p>The md5.cpp and md5.h were in frmst/wms driver folder. As I see
      from header this files are not initially from GDAL project.</p>
    <p>I moved this files to port with minimum changes - only macros for
      suppress Doxygen parsing.</p>
    <p>The motivation of this moving was:</p>
    <p>1. md5 functions used in several drivers (WMS and WFS) and may be
      others in future.<br>
    </p>
    <p>2. need API function to get the same md5 hash as used in WMS for
      autotesting functionality.</p>
    <p>3. downloading area of interest into the WMS file based cache by
      external programs. May be some other utilization. <span style="color:rgb(119,119,119);font-family:arial,sans-serif;font-size:13px;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:nowrap;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;display:inline!important;float:none"><span></span></span></p>
    <p>I think ideally the OpenSSL MD5 functions must be using. But this
      will be one more dependency. Do we need it?<br>
    </p>
    <p><br>
    </p>
    <pre class="m_8058998897030728351moz-signature" cols="72">Best regards,
    Dmitry</pre>
    <div class="m_8058998897030728351moz-cite-prefix">20.12.2017 16:43, Kurt Schwehr пишет:<br>
    </div>
    <blockquote type="cite">
      <pre>Hi all,

Can we discuss cvs_MD5* from <a class="m_8058998897030728351moz-txt-link-freetext" href="https://trac.osgeo.org/gdal/changeset/41086" target="_blank">https://trac.osgeo.org/gdal/<wbr>changeset/41086</a> ?

I very much appreciate the work that bishop is doing and I hate to slow
down a contributor

I am really worried about code like this going into GDAL, especially into
port/

But my worries are:

- this in no way conforms to other code in GDAL
- has the potential to collide with other code that imports this code
- it has a very awkward C style
- the commit message did not point to where it came from (at least it's not
hard to guess)
- the file naming is different than (most) of the rest of the directory
- and...

Yes, we have other libraries included, but it seems like a road we don't
want to go down

-kurt

</pre>
      <br>
      <fieldset class="m_8058998897030728351mimeAttachmentHeader"></fieldset>
      <br>
      <pre>______________________________<wbr>_________________
gdal-dev mailing list
<a class="m_8058998897030728351moz-txt-link-abbreviated" href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a>
<a class="m_8058998897030728351moz-txt-link-freetext" href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/gdal-dev</a></pre>
    </blockquote>
    <br>
  </div>

<br>______________________________<wbr>_________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/gdal-dev</a><br></blockquote></div></div>