Gary's current SetupXP patchset calls 2 member functions on page activation: OnActivate (returns void), and OnAcceptActivation (returns bool). I think this is unnecessarily messy. AFAICS, OnAcceptActivation only exists to prevent the need to change the return type of the existing OnActivate function.
I would very much prefer changing OnActivate to return bool, combining the purpose of both functions. Yes, this does require changes in all derived classes, but the changes are trivial, and the end result is a cleaner, more logical API. Max.
