After this is figured out, someone let me know and I'll resolve the bug the prompted the commit-<br><br>  <a href="http://www.vtk.org/Bug/view.php?id=10788">http://www.vtk.org/Bug/view.php?id=10788</a><br><br>Pat<br><br>
<div class="gmail_quote">On Wed, Jun 23, 2010 at 7:39 PM, David Thompson <span dir="ltr"><<a href="mailto:dcthomp@sandia.gov">dcthomp@sandia.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div class="h5">Matthew Wong wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
In the vtkMySQLQuery.cxx file, in the Execute() function, after the following line:<br>
    int result = mysql_query(db, this->Query);<br>
there is a check to see if the result == 0.  If the result == 0, then there is a comment in the code that says "The query probably succeeded".  If the result is not 0, there is a comment in the code that says "      // result != 0; the query succeeded"<br>

<br>
However, according to<br>
<a href="http://dev.mysql.com/doc/refman/5.0/en/mysql-query.html" target="_blank">http://dev.mysql.com/doc/refman/5.0/en/mysql-query.html</a> the return value is:<br>
   "Zero if the statement was successful. Nonzero if an error occurred. "<br>
I was wondering what motivation for the "the query succeeded" comment was, and it looks incorrect according to the <a href="http://dev.mysql.com" target="_blank">dev.mysql.com</a> documentation.<br>
</blockquote>
<br></div></div>
Matthew didn't mention it, but the reason we found this was because we ran into a case where in fact result != 0 and the query had failed. Is there some reason that the change you made on June 4:<br>
  <a href="http://vtk.org/gitweb?p=VTK.git;a=commitdiff;h=6992568a4cb9243a305f1efd383b4fab4c45ae88" target="_blank">http://vtk.org/gitweb?p=VTK.git;a=commitdiff;h=6992568a4cb9243a305f1efd383b4fab4c45ae88</a><br>
affected both branches of the conditional instead of just the branch with result == 0?<div><div></div><div class="h5"><br>
<br>
        Thanks,<br>
        David<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br>