Advance null check to before cast

Otherwise, clang analyzer thought dfunc could be nil

Change-Id: I443c31efbe38a300173ba75c739d4ab8d350abef
Fix possible null deref of ‘dsa’

Made similar to other already existing checks.

Change-Id: I609db5905bd8c5c2ec77bf46c888bbcf7d3f6226
Added vtkErrorMacro to code paths that give null ptrs

Initialize() sets this->Array to null, and that
results in many errors about null derefs.

This doesn’t actually fix those warnings mind you.

Change-Id: I85da8ed1664a6e7dcb1a6c7d987fbfbd309558db
Removed harmless-looking dead code

Change-Id: I6f150f184244e2dc2c348759793960b6b47f81f8
Whitespace/spelling

Change-Id: I6371fd07ee079a2f1abbe48da4e9c9ac343ee226
Fix warning about use-after-free

Not sure if the analyzer is right, but setting
the ivar to null after delete is safe and simple.

Change-Id: I0ac894f12312e7722e9f5b23d6e69a86cf47894e
Fix possible null deref warnings

Removed other checks which are making it think
other things could be null.  Looking elsewhere in
these files, many other places assume these things
aren’t null.

Also fix a call to GetMaximum that should have been
GetMTime.

Change-Id: Ie880a0992b64ce4389ab9b40d48aa32692595362
Merge topic 'analyzer-warnings5' into master

efc3a4e9 Fix possible null deref warnings
53c1f9fb Fix warning about use-after-free
c02f83d0 Whitespace/spelling
ea3f62ea Removed harmless-looking dead code
42815afe Added vtkErrorMacro to code paths that give null ptrs
d5feacbb Fix possible null deref of ‘dsa’
666d78c2 Advance null check to before cast