More refactoring in RefDetectorViewer

Refs #10441
Remove call to copy constructor in V3D::distance

When called millions of times in a loop, e.g. Monte Carlo, then this can
make a noticeably difference.
Refs #10169
Merge remote-tracking branch 'origin/feature/10408_autocomplete_tweaks'
Merge branch 'master' into feature/10431_refl_presenter_use_model

Refs #10431

Conflicts:
	Code/Mantid/MantidQt/CustomInterfaces/test/ReflMainViewMockObjects.h
Merge remote-tracking branch 'origin/feature/10431_refl_presenter_use_model'
Use a list instead of a vector inside intersection tests.

The lists are small so traversal time is not an issue but it is
quicker to push back to a list.
Refs #10169
A few minor things to improve efficiency for intersection calcs

Refs #10169
Reduce the number of events per run.

It brings the unit test time down to an acceptable place.
Refs #10169
Make the UI keep it's shape a bit better

Refs #10441
Refectored logging in SPectrum View

Refs #10441
Merge branch 'master' into feature/10347_refl_interpret_existing_actions

Refs #10347

Conflicts:
	Code/Mantid/MantidQt/CustomInterfaces/src/ReflMainViewPresenter.cpp
	Code/Mantid/MantidQt/CustomInterfaces/test/ReflMainViewPresenterTest.h
Improve resizing on Spectrum View

Refs #10441
Trim text for title and info lists

Refs #10441
Re #10213 Better D17 description, updated loader
Add include line for std::accumulate.

Refs #10169
Fix boost random usage for older boost versions.

Refs #10169
Don't use void pointers

Refs #10441
Correct the close window slot name

Refs #10441
Fix ambiguity error on intel compiler.

Refs #10169
refs #10385. Remove log values


This fixes the regression test. We clear the logs off the temporary workspace, that way we don't have to do a similar thing in the loop, and we avoid copying them over each time. This is not a fully formed solution, but it's a start.