You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@uima.apache.org by "Marshall Schor (JIRA)" <ui...@incubator.apache.org> on 2008/01/03 16:39:33 UTC

[jira] Created: (UIMA-687) Remove redundant notifyAll when calling casPool.releaseCas(...)

Remove redundant notifyAll when calling casPool.releaseCas(...)
---------------------------------------------------------------

                 Key: UIMA-687
                 URL: https://issues.apache.org/jira/browse/UIMA-687
             Project: UIMA
          Issue Type: Improvement
          Components: Collection Processing
    Affects Versions: 2.2.1
            Reporter: Marshall Schor
            Assignee: Marshall Schor
            Priority: Trivial


Several places in the code there is a call to casPool.releaseCas(aCas), followed by

      synchronized (casPool) { casPool.notifyAll(); }

But this is redundant, since the casPool.releaseCas(...) already does a notifyAll.  Remove the redundant notifyAlls.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[jira] Closed: (UIMA-687) Remove redundant notifyAll when calling casPool.releaseCas(...)

Posted by "Marshall Schor (JIRA)" <ui...@incubator.apache.org>.
     [ https://issues.apache.org/jira/browse/UIMA-687?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marshall Schor closed UIMA-687.
-------------------------------

       Resolution: Fixed
    Fix Version/s: 2.2.2

> Remove redundant notifyAll when calling casPool.releaseCas(...)
> ---------------------------------------------------------------
>
>                 Key: UIMA-687
>                 URL: https://issues.apache.org/jira/browse/UIMA-687
>             Project: UIMA
>          Issue Type: Improvement
>          Components: Collection Processing
>    Affects Versions: 2.2.1
>            Reporter: Marshall Schor
>            Assignee: Marshall Schor
>            Priority: Trivial
>             Fix For: 2.2.2
>
>
> Several places in the code there is a call to casPool.releaseCas(aCas), followed by
>       synchronized (casPool) { casPool.notifyAll(); }
> But this is redundant, since the casPool.releaseCas(...) already does a notifyAll.  Remove the redundant notifyAlls.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.