Change-Id: I04ffba759412a4e3877b8278d499a03f0a219199

a5d30fbc vtkWin32OpenGLRenderWindow: wrap assignments in while() in parens
1932e6c2 vtkWin32OpenGLRenderWindow: use size_t for integer storage of pointers
fd5e0ef1 vtkWin32OpenGLRenderWindow: avoid narrowing warnings
0ba047df windows: put parentheses around mixed && and || operators
581598f8 windows: use the proper format specifier for DWORD

Change-Id: I6dcfe3eb8103b616a1260e39c0f363bf5ea3892b

Also explicitly reinterpret_cast them.

Change-Id: I5ec4ae799e205e743c868cecffcf9ee400ded60f

9771254f Before c99, variable decls go before other statements.

Change-Id: I949d737991fabc7f186f90273c38a590dde25241

9791802c Before c99, variable decls go before other statements.

Change-Id: Id26fa70a3c162b63476e4d3000b780c706e388e6

87935c2b TestIncrementalOctreePointLocator: remove dead code
898b425b vtkOpenGLPolyDataMapper2D: comment possible divide-by-zero
a59bd8c8 vtkReebGraph: add a comment about realloc usage
529dccb1 vtkParse: address cppcheck messages
ae3fa6e9 vtkParse: regenerate lex.yy.c
d6d6e8fc TestIncrementalOctreePointLocator: use malloc
79abddf1 vtkWrapHierarchy: handle realloc errors properly
86e47ae8 vtkParsePreprocess: fix up some free() logic for params
8aa578f3 vtkParsePreprocess: handle realloc errors properly
d28c44a6 vtkParseMain: close the input file
9aef0e71 vtkParseMain: handle realloc errors properly
f4e756f9 vtkResliceCursorRepresentation: avoid divide-by-zero
b0bd0465 vtkImagePlaneWidget: avoid divide-by-zero
e736ad42 vtkEnSightWriter: terminate buffers properly
b4b8206b vtkPLY: handle realloc errors properly
9a78aa1a vtkSLCReader: plug memory leak
...

0a789764 Check whether volume property is null.

This addresses clang analyzer warnings found by Sean.  Checks for
property=NULL were done inconsistently.

Change-Id: I2183824cc0c8c16b95add12b8c191a33b5d1fc29

6dee6fde Fix KdTree::GenerateRepresentation and add test.

Change-Id: I15c9ecb05fc32bca7b4fd14bd0d88e76cf904929

149064b5 ADIOS: Add an initial polydata write-readback test

Note: Currently fails

Change-Id: Id80f603e566d44ba9b10944ef7cd47d8d424423b

57d3811f Fixed obvious null dereferences
ce0618c2 Fixed null dereference
1fa832c3 Fixed false positive warning about null deref
e2dad201 Fixed null dereference
c5917828 Fixed dead store warning by reformulating loop
8f0a469e Fixed dead store warning by removing redundant 'break'
832f091a Fixed a few dead store warnings
b8af0b7c Fixed analyzer warning about possible null deref
70d8784b Fixed analyzer warning by removing null check
93b12c9a Put initial assignment at declaration
ab36a82e Added some consts in a few places
a0c7098d Remove dead store to fix warning.
ca321864 Bail early on null parameter

c80a4db2 Make vtkContextScene ButtonPress/ReleaseEvent invoke a button event.

This event might be catched (for instance in PV) to provide interaction
with items.

Change-Id: I3ff3f9a4a121914faebcf6e5d87afc6bc886fe5f

Change-Id: Ia58a2fe42374dade0fb7005f2eb8d84c35401f29

If the big for loop is never entered,
then slice1scalars would be null and dereferenced

Change-Id: I4c8300c0b082c9f1a8349659f83297f81efa9894

At the changed line, “graph” and “this->Graph”
should be the same, but the analyzer only sees
that the macro has branches for “graph” being null
or not, then sees it unconditionally dereferenced.

Change-Id: I80f12f4ae3bb31ddadbf72ad0bb522bfba015a20

Body of an ‘if’ set ‘ef = null’ then next ‘if’
unconditionally dereferenced it.

Change-Id: I1decc17579c5c9f7d3bfc5c5dc45ebb328573529

Switched while to do-while to fix warning about initial 'rayMag' setting
being dead.

Change-Id: I3944988ec87f9c9b51aa1b19a1ee3aaa2ad85b47

analyzer warned that setting 'nextPass' was never read afterwards,
which was true because there was a 'break' right after.  So just remove
the 'break' and allow the already-commented 'nextPass' logic to end the loop.
Changed 'nextPass' to bool also.

Change-Id: Iff85503f0235348a6c0d3ad9b0406cd96f1c4e35

Eliminated some dead stores, and also moved some
declarations & initial definitions together

Change-Id: Ic0bc1a1d1fc6fb6315bae8006584e4102ff51996

Added check for null, other code above was doing it.

Change-Id: I0f58a1957d4586801b98805e0bd9bdcdf115eeb2

The analyzer saw a check for null so assumed it could happen,
but it must be impossible, because there’s an unconditional
dereference just below.

Change-Id: Ibdb0bd6aee89ff1eadd8442d35b7d33f54486eb4

Change-Id: Id2c6b1f3497100b2b9b4509acc30dde5a1258869

Change-Id: Ib7a96b7685e282456642c144cc7614197ae9093e

Looks like copy-paste.

Change-Id: I75810ebcd42c6c19110088b8411d71e65375b357

35df8484 fix a bug in triangle frustum intersection

Change-Id: I9b39632a0f42d07453fd3744fa9920dbb979fed7

db8d3e20 Make hdf5 properties advanced.

Change-Id: Ic4397f9b17da17c7c76597a1fcedd635bd55fd60

132fec1b snprintf from HDF5 conflicts with the one from GMVReader

Change-Id: I0ab3e1fc3e9e11d7aeb09a684f1d3ccc404063d8

61e7f03d Fixed various cleanup issues
7f9c70a7 Addressed minor code clean up issues
2cd1a322 Added baseline and data for the testing
781d1b5d Fixed various minor issues
2de16eeb Added GDAL based raster reader

cdc8d8ae Fix unreachable-code-break warnings.

The comments on these preprocessor symbols mention that they should
never be turned on. Delete code that expects them to be on.

Change-Id: I501083fe79aa96a3a0b21e69edd5a0f8027afe3a

Change-Id: Ib0d2ef47d1a09c710324602e1e73217603e2f34c