On 8/15/07, Rafael J. Wysocki <[EMAIL PROTECTED]> wrote:
> On Saturday, 11 August 2007 21:08, Alon Bar-Lev wrote:
> >
> > Hello,
> >
> > Current implementation does not cleanup console settings if abort was
> > selected by user.
> > This should fix it.
>
> I prefer the appended patch.

One of the key rules of procedural programming:
Don't write the same code at different places...
The same for memory pool size and termination handlers...
If someone will add a new initialization/cleanup to this function you
will end up
with the same bug again and again.

Alon.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
Suspend-devel mailing list
Suspend-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/suspend-devel

Reply via email to