I've just begun to read the hands on (still in procedural...) but one of the modifications I have made is impactful for most hands on so I deemed it better to share it immediately.
There is a function which was called display_sum() which I renamed display_multiply()... which is not really better. I have renamed it DisplaySumOfMultiply(, which albeit ugly is correct. The notebook has been corrected as well (a display_sum slipped through and was unduly kept).
Others modifications are much more minor:
- I heeded a warning from CMake and changed the type of the variable which indicated which C++ standard was used.
- A typo was corrected.