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


Gary R. Van Sickle wrote:
>> Gary R. Van Sickle wrote:
>>> and a reversion of the wizard page titles to "Cygwin Setup"
>>> (which I need to bring up for discussion separately).
>>
>> Indeed. First, I *really* don't see this change happening as is. It was
>> originally done to assist automation programs. I don't think we should
pull
>> the plug on them.
>
> Nor I; On the contrary, I think we should do all we can, both for
automation
> people and the accessibility people.  But I've never seen a Windows
> application that doesn't have its name in the title bar.
>
>> Perhaps we can compromize on "Cygwin Setup -
>> CurrentTitleOfPage"?
>
> That would probably work.  I've seen one installer that did that, and I
see
> that my Platform SDK help browser thing does that too.

OK, I'll put together a patch+ChangeLog for this tonight (GMTDT, i.e. +1)
unless you beat me to it.

----

>> "make LogFile::exit() not exit if exit_code == 0":
>> I don't understand the reasoning behind this change. Can you explain?
>
> This is one small step for man towards moving the app's exit()ing and UI
> interaction out of the LogFile class.  I'm only changing the success (==0)
> case currently because there's only one code path that succeeds, while
there
> are several places in the code (and in the UI) that do this:
> "LogSingleton::GetInstance().exit(1);".  Eliminating those will require
more
> extensive changes.

Wait! The status quo must remain, until we have confirmed evidence that
static destructors do always run on exit from -mno-cygwin programs. Then and
*only* then can we move to using simple CRT exit(). Until then I don't think
there is any harm in using constructs like
LogSingleton::GetInstance().exit(1).

Robert, your opinion?


>> P.S.: Please could you either add a last-changed-date to your webpage,
>
> Good idea, will do.

Thanks.

Max


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