Commit b23ee9f2 authored by Sean McBride's avatar Sean McBride
Browse files

Fixed apparent mistake in error checking, where no errors actually caught

cppcheck 1.71 reported:

Result of operator '|' is always true if one operand is non-zero. Did you intend to use '&'?
parent b6bb2bb3
......@@ -1220,15 +1220,15 @@ void vtkXdmf3DataSet::VTKToXdmf(
bool OK = true;
vCoords = dataSet->GetXCoordinates();
OK |= vtkXdmf3DataSet::VTKToXdmfArray(vCoords, xZCoords.get());
OK &= vtkXdmf3DataSet::VTKToXdmfArray(vCoords, xZCoords.get());
if (OK)
{
vCoords = dataSet->GetYCoordinates();
OK |= vtkXdmf3DataSet::VTKToXdmfArray(vCoords, xYCoords.get());
OK &= vtkXdmf3DataSet::VTKToXdmfArray(vCoords, xYCoords.get());
if (OK)
{
vCoords = dataSet->GetZCoordinates();
OK |= vtkXdmf3DataSet::VTKToXdmfArray(vCoords, xXCoords.get());
OK &= vtkXdmf3DataSet::VTKToXdmfArray(vCoords, xXCoords.get());
}
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment