http://zeitlin.homeunix.com/cgi-bin/mbugs/show_bug.cgi?id=808
[EMAIL PROTECTED] changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
------- Additional Comments From [EMAIL PROTECTED] 2003-10-03 13:17 -------
I have finally fixed it. Function that copies messages didn't report
that copying was cancelled and one function above deleted all those
messages thinking they were copied successfully.
Lessons to learn from this:
1) Cancelling is error (from point of view of the code). It just has no
associated error message.
2) Failure indicator variables are evil, because somebody will
inevitably forget to set them. All errors should be followed by return
false, leaving cleanup for destructors.
3) We should consider using exceptions, so that return false cannot be
ignored. Then we can make progress dialog throw exception when cancel is
pressed.
Also, I had to shut up error messages in AsyncStatusHandler (they now
appear only in status bar). Only error source knows what message is
suitable, if any.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Mahogany-Developers mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/mahogany-developers
