So yes, this review becomes a bit messy, I see that this is already outdated so processed?

Anyway, please keep template <typename Strategy> at the method-level and *not* at the function level. Sorry if you already changed this. We prefer to have it at the method level (for Menelaos: for your information).

As also remarked by Bruno and now agreed by Menelaos.