Revision: 4243
          http://skim-app.svn.sourceforge.net/skim-app/?rev=4243&view=rev
Author:   hofman
Date:     2008-07-17 19:54:01 +0000 (Thu, 17 Jul 2008)

Log Message:
-----------
Reverse order of setting stop flag and sending message to top DO server. The 
current order was not accepted, as a message was send to the server after it 
had stopped. This ordering should now be safe because we always clean up and 
also set the stop flag in the message send over DO.

Modified Paths:
--------------
    trunk/SKPDFSynchronizer.m

Modified: trunk/SKPDFSynchronizer.m
===================================================================
--- trunk/SKPDFSynchronizer.m   2008-07-17 19:41:36 UTC (rev 4242)
+++ trunk/SKPDFSynchronizer.m   2008-07-17 19:54:01 UTC (rev 4243)
@@ -155,10 +155,10 @@
 #pragma mark | API
 
 - (void)stopDOServer {
+    // this is really only necessary to tickle the server thread's runloop so 
it will finish
+    [serverOnServerThread stopRunning];
     // set the stop flag so any running task may finish
     OSAtomicCompareAndSwap32Barrier(1, 0, (int32_t 
*)&serverFlags->shouldKeepRunning);
-    // this is really only necessary to tickle the server thread's runloop so 
it will finish
-    [serverOnServerThread stopRunning];
     
     // clean up the connection in the main thread; don't invalidate the ports, 
since they're still in use
     [mainThreadConnection setRootObject:nil];


This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
Skim-app-commit mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/skim-app-commit

Reply via email to