windows: use the proper format specifier for DWORD

Change-Id: If655342a6a7de4ee5fe176eda0b54cf0def6d491
windows: put parentheses around mixed && and || operators

Change-Id: I564e13887ecefeaf35d361f37c6b75f545720fba
vtkWin32OpenGLRenderWindow: avoid narrowing warnings

The array is of BYTE, but there are some integers floating around.
Static cast them to BYTEs.

Change-Id: I6ff0ae85a78908fdcb3fb9df45bb5e8d5efca2da
Fix SetAllArrayStatus on the ExodusII reader.

Calling SetAllArrayStatus with any "_CONN" enum has no effect because
these are not the enums that the API expects to be used for connectivity.
However, the ones it does expect (e.g., SIDE_SET instead of
SIDE_SET_CONN) are being used by SetAllArrayStatus for indicating whether
result arrays defined on cells (e.g. pressure, temperature, ...) should
be loaded. So, for this method only, convert the "_CONN" enum into the
expected enum before continuing.

Change-Id: I0d0064e82c635ad94b3b293d39bb0d64f2607bbd
Bail early on null parameter

This convinces clang analyzer that there’s no
null dereference.

Change-Id: I2cc59ed9471bdad0137329c9d27e9492ea8581bd
vtkKdNode, vtkOctreePointLocatorNode: don't assign after a short-circuit

Conditionals may short-circuit and assignments anywhere other than the
first sub-expression is not guaranteed to be executed, so just always do
the subtraction.

Change-Id: I8e1ed5ad4f1ac3257209ee0386c5965cfd8d1e2f
vtkPolyhedron: initialize the iterator

In the loop, the map can cause the current iterator in cpSetIt to be
erased from cpSet, but it is not guaranteed to be initialized.

Change-Id: I793dc4b50c0ffcc616f59b9550c507e8f5e04479
HyperTree example: use the iterator before invalidating it

Change-Id: I88b20ac27e198a759d1c9a1c8d9e4b6ddc1d3da6
Widgets example: fix up logic around planeSource

Change-Id: I5bf5a76e0d57c2a33ffd8f95f63e83bdf27313fd
vtkYoungsMaterialInterface: plug memory leaks

Change-Id: I6fcdd2e8faf210bf7dadc8bafb49a1d38ac23f63
vtkUnstructuredGridGeometryFilter: plug memory leak

Change-Id: I84a9f361ed120c4a46a308dd772a2d5df259811b
vtkPKdTree: plug memory leak

Change-Id: I73c6692d5eb0585d7f6504cad3f2c74531d5fdd0
vtkDistributedDataFilter: plug memory leaks

Change-Id: If46406100cb9dddd762aa4353065c8dbde9aae49
vtkKMeansStatistics: plug memory leak

Change-Id: I78895a6ca1841d2177efeef289d3642b0a9a1257
vtkAMRFlashReaderInternal: don't return a pointer to temporary memory

The string only lives as long as its expression, so don't return its
internal c_str.

Change-Id: I32f9fc8ef9847baa645d5743d399b55782122114
vtkExodusIIWriter: use a larger buffer

Missing space for a NUL byte.

Change-Id: I130ca197de1c37c087884b27be30e50de7e9909b
TestIncrementalOctreePointLocator: close file handles

Change-Id: I98c0548b968ad06d97373d89d0e375cd71a058c0
vtkCGMWriter: fix typo

Change-Id: Ic781ec4a635566d7a25aa071910cb8516888c3c4
vtkNIFTIImageReader: plug memory leaks

Change-Id: I96903ef5343f319a4e33e84f0afaaf709cfb8753
vtkSLCReader: plug memory leak

Change-Id: I94a508d159219f833d6363ae1c9078c94aa06c47