Aligned the calling convention of visit to match that of std::visit. Renamed the more convenient version of visit to match to have the best of the two worlds: better ergonomics and not clashing with std::visit. This time for real.

Include the VisitorTest.cpp into the CMakeList.txt.

Regarding deriving from using other kinds of functors: I have not yet seen a need for this. My preference is to cross that bridge when we reach it. :)

Aligned the calling convention of visit to match that of std::visit. Renamed the more convenient version of visit to match to have the best of the two worlds: better ergonomics and not clashing with std::visit.

Added docs with usage example. Hid implementation in detail namespace.

Replaced class with typename in template definition.

Replaced class with typename in template definition.

Re: handleAllErrors. Maybe. It might result in better runtime performance (because there is not recursion on the handlers), but it's also a rewrite. I am not sure if you are asking me to do something. :)

Thanks for taking a look, Mehdi and David! PTAL.

Added a unit test.

Added a unit test.