Skip to content

Fix segfault in FilterSvs

Thomas Boettcher requested to merge thboettc_filter_svs_overrun into master

Resolves #519 (closed)

This MR fixes an issue where FilterSvs and CalcMaxCombos have different event lists in some edge case. This can happen because CalcMaxCombos takes an aggregate of containers as input, while FilterSvs just takes the SV container. This MR adds the SV container to CalcMaxCombos as an input, but doesn't use it. This insures that these algorithms have the same event list.

This is a bit of a hack. @cagapopo I think you understand the event list machinery better than I do. Please let me know if you can think of a better fix!

FYI @acasaisv

Merge request reports