All CLA requirements met.

Codecov Report

Merging#1160 intomaster willnot change coverage.
The diff coverage isn/a.

@@           Coverage Diff           @@##           master    #1160   +/-   ##=======================================  Coverage   86.53%   86.53%           =======================================  Files           1        1             Lines         297      297           =======================================  Hits          257      257             Misses         40       40

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update9fb545a...89d1177. Read thecomment docs.

@lostmsu We should probably do a point-release for this, agree?

@filmor I have another one too in the recent params array change.

BTW,@DanBarzilian , a personal thank you 10x! This finding and the fix for it just saved me after 3 workdays of chasing a semi-random segfault in tests. I don't know how much time I would have wasted trying to find this.

Funnily, while searching for (apparently) this issue, I stumbled and fixed another less impacting one:losttech@74e7d62

@filmor@koubaa we should mandate using*Reference types in new PRs. This bug would have been prevented by proper typing.

@lostmsu now it explains why the CI has been unreliable recently. Big whoops for missing this and yes more evidence to use Reference types. Good catch@DanBarzilian