[setup PATCH] Rework do_download [next_dialog removal (2b)]

Max Bowsher maxb@ukf.net
Tue Sep 30 11:49:00 GMT 2003


Robert Collins wrote:
> On Thu, 2003-07-31 at 08:16, Max Bowsher wrote:
>> And another...
>
> This seems more complex than the previous code. You're moving window
> specific code from the specific class into the generic threebar class.
> This doesn't seem right to me.

I think the code I moved is better classified as program flow logic.
threebar.cc is already our main controller of program flow once the initial
few pages of settings are complete. At the moment, as individual window is
handling decisions on what to display next - *that* doesn't seem right to
me - it's exactly what wantsActivation was introduced to avoid.

I can agree that threebar.cc isn't the most elegant place to be handling
program flow, but that fact doesn't reduce the merits of tidying and
collecting program flow logic into a central location.


Max.



More information about the Cygwin-apps mailing list