core (and possibly other modules) use raw function pointers - std::function would be more appropriate
The topic was mentioned in a comment in !1914 (merged).
Using "raw" function pointers is questionable (with C++20), as std::function
offers - among other things - more control on the function being pointed, and makes the code easier to read.
A good MR should:
- Identify where raw function pointers are used, the core module is not the only place they've been used, and
- Propose a way to change them to
std::function
Edited by Peter Barnes