Remove several useless stores just before assert(0)
and add 'return' statement too.

Change-Id: I494d11d0ce3312bb82a094ec0d90451aa208b782

Change-Id: Idc7cafc9c7aeed9c01443efed0892cccb459ac0f

Change-Id: Ia2764b7a34514fd83e48ce10d3866f8f8091f987

Change-Id: If42caf9e6935a098f165493bc782bfe38b553905

Added 'return' after an vtkErrorMacro, just like every other vtkErrorMacro in this method.

Change-Id: Ife0f7526c31b21a5e99d2abd7101a4043c4f34d1

Change-Id: I361e472426ce927b301ea01a8c1a6680aa5c6438

If 'numDeleted' is not 2 nor 1, 'tri' was used uninitialized; call vtkErrorMacro instead.

Change-Id: Ia0e508184a37e07f3186e192a166d1f38c79cbdb

a) suppress null deref warning with assert
b) fixed null deref of 'cell' by bailing early
c) changed error test of an existing vtkErrorMacro and added 'return' to prevent 1) infinite loop (of count VTK_LARGE_INTEGER) and 2) null deref of 'minCells'

Change-Id: Iebc64b79b51fd30ac714be4d067e344321d2a17f

If GetNumberOfTuples returns more than 3 (probably never happens generally) the 'for' loop will read past the end of 'eigenvaluesGroundTruth', fixed by defensively bailing early if necessary.

Change-Id: Id7c7c6b8549b42912b993bc1582fdf69422c223f

The store to 'np' is definitely dead, but suspicious.  Perhaps it should be assigning to a different variable? Perhaps it's leftover copy-paste?

Change-Id: I79ec354985f7f0859b790702821a3c08520cbf5f

First store is useless because 'continue' statement results in it being overwritten in the next loop.  Second is useless because value is always overwritten later before it's read.

Change-Id: Ibd5f44dc52a0e58559e4a096743fc78a509fac4d

Change-Id: Ica56d71e3ae4b230b5cbe97df7a784b47970fa83

Exact same assignment is made later on, and right before usage.

Change-Id: I113c924d0d0fc8b2d268cbfbdc857dcb1bd40d7a

An 'abort' variable to exit a loop early was redundant with a 'break' statement in the loop.

Change-Id: Iad88d4b4fbf4d3fea9ee996c6c43a4aa55fc09cd

An 'abort' variable to exit a loop early was redundant with a 'break' statement in the loop.

Change-Id: Ifa7dc3bd9a58742ff9acc30de530f134e642ed85

Store to 'x' in loop is useless since it's overwritten in all cases below.

Change-Id: I6f09ecb8ca37bdbd4abe817856852354109e2e23

Useless store to local variable right before a return.

Change-Id: I0bd514edec4c7e7ac5409f68c50dae6bdee5a056

If neither 'DMapData' nor 'MapData' get set, 'results' is read uninitialized; added 'else' case and bail out instead.

Change-Id: Ifa35e7ff76a1d3d9e74d446f1b9f3c4b42e3bc9a

In an error code path, vtkErrorMacro is used but the 'return' was omitted (every other vtkErrorMacro is paired with return), execution would then continue and pass null to memcpy.

Also removed unrelated useless test for null before using delete.

Change-Id: I9d63fae00080c2fca5974442d8bd2f936fcf0790

The pointer was being checked for null, but only logged; now return too.

Change-Id: Id9af0f968fc4fd626d9eec79af1ee22aebdc6cc6

Change-Id: I20c684c0151ea7dab1575d003b40b78e5da2b809

Change-Id: I6a4fef51ca87fa256fc33ef6919b9bea4dbc3e9b

d5e12400 Initialize row_pointers
adf24a53 Fix memory leak in vtkPNGWriter when OutOfDiskSpaceError

42746d05 Add NaN support in web chart rendering

Change-Id: I4b848a00c553f0e5429521245572b526daa0d565

c68b549e Ensure libproj4 defaults to use pthreads if VTK does

a162be1f 14196: Allow unicode path in vtkLoadPythonTkWidgets.py.

Change-Id: I401d622e28e62d78ed850c36f5270a682f08a2c0

399f778d Disabling translation and scaling now works.

0b42c0e7 InsertNextCell and related method now return the proper vtkIdType.

Change-Id: I294825c2de6d2cdc5854116f12227f25ed88abaa

fe9dc76b Remove accidentally added test data files

Dave DeMarle authored 11 years ago and Brad King committed 11 years ago

In commit 352e3122 (Add test for vtkOpenFOAMReader, 2013-12-24) we
accidentally added files prefixed in "._" in addition to the actual data
files.  These files aren't real and, when you archive and untar a vtk
release, produce file exists warnings.

Change-Id: I13b600357f1b9712d55d8c0284c97fd1b0255683

Change-Id: Ie12cce088a6ed29f8c73a39587610db2cdf2e3ee

Change-Id: Ib5f0adc17054a3a4c490bd45c980b7c8d2de932a

* pvvtk/pv-master:

Change-Id: I370b147e8ffdd5b1ce8f6b15443fb751a86376aa

Utkarsh Ayachit authored 11 years ago and Kitware Robot committed 11 years ago

4fafafc5 Merge branch 'vtkweb-add-selection-rectangle-to-viewport-rel' into release
c2af9cb1 Added code to draw current selection rectangle in viewport.
0aac1cbe Merge branches 'cocoa-configure-event-rel', 'catalyst-web-rel' and 'pvweb-add-filebrowser-cache-option-rel' into release
59ae032d Added code to support a cache option for filebrowser widget.
1a8d823e Add catalyst web widget into VTK
e8746067 Merge branch 'release-add-SourceTarball-script-rel' into release
0bf3a668 Merge branch 'vtkweb-update-vgl-rendering-rel' into release
947ca938 Merge branch 'pvweb-fix-selection-rel' into release
7b81e676 Merge branch 'fix_test_set_get-rel' into release
7ae311e9 Merge branch 'fix_particle_tracking' into release
4021de7b Merge branch 'gcc-4.8.3-compile-error-rel' into release
0d491037 surface LIC - stl and NULL
0d24c40c Merge branch 'clang_libc++11_support-rel' into release
0d96cb99 Merge branch '14503_fix_intel_build-rel' into release
0fb3dd73 BUG: Fix build failure in RenderingParallelLIC on Intel compiler
7134a7d8 Merge branch 'fix-append-filter-for-polyhedron-cells-rel' into release
...

Change-Id: I4578524d30aa205bf677bebbcfa5582f73dbce63

Change-Id: I8ab5c3a36f2e1649cee91552b1a5c3f1be277dec

c2af9cb1 Added code to draw current selection rectangle in viewport.