river-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gregg Wonderly <ge...@cox.net>
Subject Re: svn commit: r1497973 - /river/jtsk/skunk/qa_refactor/trunk/src/com/sun/jini/outrigger/OperationJournal.java
Date Sat, 29 Jun 2013 19:14:06 GMT
This is one of those things where you just say, really!

Gregg

On Jun 28, 2013, at 8:53 PM, peter_firmstone@apache.org wrote:

> Author: peter_firmstone
> Date: Sat Jun 29 01:53:28 2013
> New Revision: 1497973
> 
> URL: http://svn.apache.org/r1497973
> Log:
> Ensure remaining transitions are processed prior to termination in OperationJournal
> 
> Modified:
>    river/jtsk/skunk/qa_refactor/trunk/src/com/sun/jini/outrigger/OperationJournal.java
> 
> Modified: river/jtsk/skunk/qa_refactor/trunk/src/com/sun/jini/outrigger/OperationJournal.java
> URL: http://svn.apache.org/viewvc/river/jtsk/skunk/qa_refactor/trunk/src/com/sun/jini/outrigger/OperationJournal.java?rev=1497973&r1=1497972&r2=1497973&view=diff
> ==============================================================================
> --- river/jtsk/skunk/qa_refactor/trunk/src/com/sun/jini/outrigger/OperationJournal.java
(original)
> +++ river/jtsk/skunk/qa_refactor/trunk/src/com/sun/jini/outrigger/OperationJournal.java
Sat Jun 29 01:53:28 2013
> @@ -402,8 +402,12 @@ class OperationJournal extends Thread {
>      * Terminate queue processing.
>      */
>     void terminate() {
> -	dead = true;
> +        /* Only set dead to true while synchronized, this means that all
> +         * pending transitions will be processed then wait() is called
> +         * releasing the lock so queue processing can terminate.
> +         */
>         synchronized (this){
> +            dead = true;
>             notifyAll();
>         }
>     }
> 
> 


Mime
View raw message