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] |
On Tue, 2003-04-01 at 09:31, Igor Pechtchanski wrote: > > Rob, > > The reason I wanted to keep the to_log parameter is that I wanted to keep > the code around that directs output to console. If there are no > objections, I'll just make the default value of to_log TRUE, and remove > the third parameter from the calls. I'd like to keep the parameter for > now, though. We're not using that plain code today. CVS will keep it forever, but I don't want setup kept more complex than it needs to be. Here are some options I'll accept: option A) Make output a Strategy of the script object. Allow the output strategy to determine window sizing and logging (or not). (This would probably be three patchs: patch 1) - Extend Script to have more responsibilities from the non OOP script code. patch 2) - Give it a base strategy class for logging, containing the current if (to_log) conditionals, and a boolean in the constructor. patch 3) - Create a subclass of the strategy for logging to files. - Create another subclass for logging to console. - Make the base strategy methods pure virtual as appropriate. - Only link in the log to file strategy. option B) Strip out the unused code. Leaving the code there, simply so that it's there doesn't make sense. I'll accept 'leave it there so we can do X in the future' - in which case option A above is an easy path to separate out the two bits of code and allow X. But leaving it there for no purpose doesn't make sense to me. Rob -- GPG key available at: <http://users.bigpond.net.au/robertc/keys.txt>.
Attachment:
signature.asc
Description: This is a digitally signed message part
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |