Kill warning in RemoteJobManager.cpp

Refs #7994
Pass matrices/unit cell by reference to const in constructors.

Refs #7994
Pass by reference in NexusClasses. Refs #7994
Remove old unused code.

Refs #7994
Move null pointer checks to more sensible place.

Refs #7994
Avoid capturing return values if they are not used.

Refs #7994
Remove pointless check for textAxis pointer in FitMW.

We know from our setup that it's what we will get.
Refs #7994
Use a more sensible argument type for FileFinder::findRun

Refs #7994
Check once for NULL graph pointer in *Curve constructors.

Refs #7994
Swap raw pointer for boost::shared_ptr in BinaryOperation

The BinaryOperationTable was not deleted at the end of the operation
causing a memory leak. The shared_ptr will take care of this properly.
Refs #7994
Merge branch 'bugfix/7994_coverity_errors' into develop

Conflicts:
	Code/Mantid/MantidPlot/src/Mantid/MantidUI.cpp
Refs #7994