This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [SetupXP] The two styles for handling activation refusal


> 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.
> 

Yep.

> 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.

That was exactly my originally-submitted patch.  It was refused.

-- 
Gary R. Van Sickle
Brewer.  Patriot. 


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]