Wow. Thanks! Will study. Could you possibly rebase with master? Your ChangeLog shows you branched after July 12 so you are missing a little bit. It's not critical -- I could always squash-merge as well. (And will hand-adjust ChangeLog.)

Sure! (I just distractedly updated only the ChangeLog, here comes the rest...)

Interesting changes, eg theClass vsCppClass. Were we operating on the wrong variable? Entirely possible, but also possible this may have side effects :) I trust you tested this?

I reran all unitTests and added my own. (There were some errors but not due to my additions; I didn't fix them to focus on the issue the PR is attempting to solve...)

Codecov Report

Merging#886 intomaster willincrease coverage by0.09%.
The diff coverage is50%.

@@            Coverage Diff             @@##           master     #886      +/-   ##==========================================+ Coverage   90.09%   90.18%   +0.09%==========================================  Files          71       71                Lines        3271     3271              ==========================================+ Hits         2947     2950       +3+ Misses        324      321       -3

Continue to review full report at Codecov.

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

Hopefully this the correct rebase, moduloChangeLog.

Awesome! Thanks for the PR.

While I'm not that familiar with the modules codebase, all of the changes here look sensible and correct, so a thumbs up from me.

Same here. Will run a rev.dep check just in case.

Sorry, got distracted for a bit. Reverse dependency check threw up nothing new, so merging this now.