nettika/geary
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

New composer crashes Geary: Refs bug #730483

Browse source 
Don't start the progress monitor unless it's not running.  Related
to fix for bug #728936 which dealt with reentrancy while folder is
opening.  Suspect this is a timing issue.
This commit is contained in:
Jim Nelson 2014-05-20 18:14:44 -07:00
parent 095140abb2
commit 9dbd0d3d18
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
src/engine/imap-engine/imap-engine-minimal-folder.vala
View file

@ -544,7 +544,8 @@ private class Geary.ImapEngine.MinimalFolder : Geary.AbstractFolder, Geary.Folde
// to ensure this isn't running when open_remote_async() is called again (due to a connection
// reestablishment), stop this monitoring from running *before* launching close_internal_async
// ... in essence, guard against reentrancy, which is possible
opening_monitor.notify_start();
if (!opening_monitor.is_in_progress)
opening_monitor.notify_start();
// following blocks of code are fairly tricky because if the remote open fails need to
// carefully back out and possibly retry