@filmor Any idea why the build is broken?

The Appveyor build just timed out for one part, I triggered it again and it seems to work fine now. The Travis build looks a bit trickier, the Mono version pulled in seems to be broken...

Meh, the travis error is broken. The symbol it is missing is _ZTIPi which is the C++ type info for int*, so maybe there are incompatible C++ ABIs at play here ...

How can the build just break? Were there changes made?

The mono version is not pinned right now, so since the 18th we pull in mono 6. I'll see whether I can lock us to 5.20 for now.

Codecov Report

Merging #925 into master will increase coverage by 1.76%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #925      +/-   ##
==========================================
+ Coverage   74.48%   76.25%   +1.76%     
==========================================
  Files          62       65       +3     
  Lines        5902     6030     +128     
  Branches      967      990      +23     
==========================================
+ Hits         4396     4598     +202     
+ Misses       1153     1099      -54     
+ Partials      353      333      -20

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f635a4...f6a9954. Read the comment docs.

@filmor Can be merged?

Yeah, looks good to me. @lostmsu Be aware of this one if you are looking into switching to delegates.