refs #9048 Fixing code which breaks Win comp in RelWithDebug mode

for Curve fitting though it can break Mac build in return.
refs #9048 should fix one of Coverity false alarms
refs #9048 Coverity warning CID 1075687

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11026259&defectInstanceId=3876757&mergedDefectId=528792
refs #9048 fixing Coverity noise for CompareMDWorkspaces.cpp

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11026543&defectInstanceId=3878116&mergedDefectId=528435
refs #9048 Coverity noise in IMDEventWorkspace

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11024017&defectInstanceId=3877305&mergedDefectId=528995
initialized m_FileNeedsUpdating, though I do not think it is necessary/used any more.
refs #9048 Trying to fix Coverity noise in ConvToMDSelector

at https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11025316&defectInstanceId=3876567&mergedDefectId=528357

do not understand what Coverity is complaining about -- default occurs if neither EventWS nor Matrix2D ws is provided (e.g. Table) Rewritten in a way which hopefully is more clear for Coverity
refs #9048 Avoid unnecessary shared pointer increment
refs #9048 Coverity noise at SlicingAlgorithm

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11027297&defectInstanceId=3877465&mergedDefectId=529167
clear noise -- it is initialized in the algorithm itself.
refs #9048 Coverity noise ? in CoordTransformAffine

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11025303&defectInstanceId=3876983&mergedDefectId=528902

It may be actually a real memory leak -- not sure how delete [] figures out the range of array it deallocates and it is not standardized. But the changes should make Coverity happy for very low price of additional pointer stored in the class.
refs #9048 Ubuntu does not like this kind of default algorithm

, so need to get rid of it. Defaults are not used in its present form anyway.
refs #9048 Fixing build

Changed form is actually used in the test but this can be moderated.
refs #9048 removed unused header from the BoxController.h
refs #9048 Coverity bug in ConvToMDEventsWS

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11025021&defectInstanceId=3876993&mergedDefectId=528900
It looks like real resource leak, fixed by wrapping the thread scheduler into unique pointer.
refs #9084 Did the same as in the previous commit with ConvToMDHistoWS
refs #9048 Coverity noise

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11025006&defectInstanceId=3877350&mergedDefectId=529173
uninitialized members in ConvToMDBase.cpp constructor.
refs #9084 fixing build

and probably reverting Coverity warning bout the resource leak. But in fact tread sheduler is deleted by thread pool.
refs #9048 Coverity bug?

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11025833&defectInstanceId=3878261&mergedDefectId=528972
not sure if Coverity just mean in this case, but have just rewritten this function to accept missing experiment info entries. Do not think its logic is tested anywhere.
refs #9048 Coverity Noise

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11026571&defectInstanceId=3877375&mergedDefectId=529159
for ConvertToDiffractionMDWorkspace
refs #9048 Coverity noise for MDTransfModQ

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11026159&defectInstanceId=3877341&mergedDefectId=529178
just useless constructor initialization.
refs #9048 COverity noise for MDTransfModQ.h

https://scan6.coverity.com:8443/reports.htm#v29240/p10083/fileInstanceId=11026160&defectInstanceId=3876763&mergedDefectId=528801
do not understand this message and do not see any comma. Moved operation into from h to cpp in the hope to quiet this message.