<div dir="ltr">And this highlights Utkarsh's good idea to have the toolkit specific buildbot scripts or some of them in the toolkit tree. That way we could turn on the flag for that topic to test it out without impacting the other topics in testing or master. <br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 10, 2018 at 1:27 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"><div>So... I am seeing lots of warning suddenly pop up on eeloo. As we clearly want to keep our buildbots green, so that people notice when they introduce new warnings or errors, these need to be either</div><div><br></div><div>1) promptly fixed<br></div><div><br></div><div>or if no one is opting for (1) then<br></div><div><br></div><div>2) suppressed until such as time that someone can fix them</div><div><br></div><div>Is anyone doing (1) ? If not then we need to do option (2) or some other creative idea ...<br></div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 6, 2018 at 9:08 AM, Ben Boeckel <span dir="ltr"><<a href="mailto:ben.boeckel@kitware.com" target="_blank">ben.boeckel@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jul 03, 2018 at 12:48:23 -0400, Ken Martin wrote:<br>
> I think that is a crazy warning gcc is producing and we should disable it.<br>
> Way too many false positives with their approach. Can you change the<br>
> compile flags on eeloo to remove that warning?<br>
<br>
It's not a crazy warning. I saw warnings from the Catalyst builds after<br>
updating `megas`. The one in vtkObjectFactory is a false positive, but<br>
rewriting it to use `std::string` instead of `strncat` just makes the<br>
code easier to read anyways. Errors in <a href="http://pv-forward.c.in" rel="noreferrer" target="_blank">pv-forward.c.in</a> were real and<br>
(since it is C), `snprintf` is the better solution there.<br>
<br>
Basically, `strncat` is a function that is really easy to screw up. We<br>
really shouldn't use it in C code and certainly not at all in C++ code.<br>
<span class="m_2825959092351752382HOEnZb"><font color="#888888"><br>
--Ben<span class="HOEnZb"><font color="#888888"><br>
</font></span></font></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_2825959092351752382gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div>101 East Weaver Street<br>Carrboro, North Carolina<br>
27510 USA<br><br><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></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Ken Martin PhD<div>Distinguished Engineer<br><span style="font-size:12.8px">Kitware Inc.</span><br></div><div>101 East Weaver Street<br>Carrboro, North Carolina<br>
27510 USA<br><br><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></div></div>
</div>