Re #7737. Add (failing) tests that expose the bug.
Re #7737. EventList::filterByPulseTime throws if filtering in-place.

Since this method clears the 'output' EventList, it can't be the
same one as the input.
Re #7737. Always create an output workspace.

You cannot filter in-place.
Re #7737. Formatting only: change tabs to spaces.
Re #7737. Fix EventList cppcheck warnings.
Re #7737. Add a performance test.
Re #7737. Create algorithms on the stack, not the heap.
Re #7737. Refactor test to not load data.
Re #7737. Remove no-longer-used variables.
Merge remote-tracking branch 'origin/bugfix/7737_filterbytime_same_input_output'