<div dir="ltr"><div><div>I think it should be OK to remove `static`.<br><br></div>Thanks<br></div>Sujin<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 2, 2017 at 3:10 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I wonder... perhaps clang only warns about these templates because they are declared as "static".  It's generally expected that static functions will be used within the execution unit in which they are defined.  If the "static" is removed, then the compiler should treat the templates as inline functions and it will let the linker deal with the ODR.</div><div><br></div><div>Is there any reason not to remove "static" from the declarations of these templates?</div><span class="HOEnZb"><font color="#888888"><div><br></div><div> - David</div></font></span><span class=""><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 2, 2017 at 12:51 PM, Ken Martin <span dir="ltr"><<a href="mailto:ken.martin@kitware.com" target="_blank">ken.martin@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hmm it seems the llvm folks added this due to some ODR violation checking. That having it unused can cause it to be defined multiple times in a translation unit for some reason. It is all a bit beyond me.</div><div class="m_8704997175068582356HOEnZb"><div class="m_8704997175068582356h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 2, 2017 at 11:08 AM, Sujin Philip <span dir="ltr"><<a href="mailto:sujin.philip@kitware.com" target="_blank">sujin.philip@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>Hi Ken,<br><br></div>The vtkSMPTools.h header has parallel algorithms like For and Sort. I think this warning is occurring because some files that are including this are just using Sort. So, I think this warning should be suppressed or else we may have to refactor the code, which will not be a simple fix.<br><br></div>Thanks<br></div>Sujin<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 2, 2017 at 10:38 AM, Ken Martin <span dir="ltr"><<a href="mailto:ken.martin@kitware.com" target="_blank">ken.martin@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Do we care about these? Are they of value? I guess the question is should we fix or suppress?</div><div><br></div><div><br></div><pre class="m_8704997175068582356m_2775539894727977461m_1609717003785916080m_8568329822498050140gmail-compiler-output m_8704997175068582356m_2775539894727977461m_1609717003785916080m_8568329822498050140gmail-ng-binding" name="stderr" style="box-sizing:border-box;overflow-x:scroll;overflow-y:auto;font-family:"Courier New",Consolas,Monaco,"Lucida Console","Liberation Mono","DejaVu Sans Mono","Bitstream Vera Sans Mono",monospace;font-size:14px;padding:9.5px;margin-top:0px;margin-bottom:10px;line-height:1.42857;word-break:break-all;word-wrap:break-word;color:rgb(51,51,51);background-color:rgb(245,245,245);border:1px inset rgb(204,204,204);border-radius:4px;max-height:350px">/Users/builder/external/VTK-cl<wbr>ang-rel-x86_64/Common/Core/vtk<wbr>SMPToolsInternal.h:24:13: warning: unused function template 'vtkSMPTools_Impl_For' [-Wunused-template]</pre><span class="m_8704997175068582356m_2775539894727977461m_1609717003785916080HOEnZb"><font color="#888888"><div><br></div>-- <br><div class="m_8704997175068582356m_2775539894727977461m_1609717003785916080m_8568329822498050140gmail_signature"><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div>28 Corporate Drive<br>Clifton Park NY 12065<br><div><br></div><div><span style="font-size:10pt;font-family:Tahoma,sans-serif">This communication,
including all attachments, contains confidential and legally privileged
information, and it is intended only for the use of the addressee.  Access to this email by anyone else is
unauthorized. If you are not the intended recipient, any disclosure, copying,
distribution or any action taken in reliance on it is prohibited and may be
unlawful. If you received this communication in error please notify us
immediately and destroy the original message. 
Thank you.</span></div></div></div></div></div>
</font></span></div>
<br>______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensou<wbr>rce/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mail<wbr>man/listinfo/vtk-developers</a><br></blockquote></div></div></blockquote></div></div></div></div></blockquote></div><br></div></span></div>
</blockquote></div><br></div>