You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mina.apache.org by "Trustin Lee (JIRA)" <ji...@apache.org> on 2007/07/31 09:24:53 UTC
[jira] Updated: (DIRMINA-409) Duplicate syncronization in
SocketAcceptor.startupWorker()
[ https://issues.apache.org/jira/browse/DIRMINA-409?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Trustin Lee updated DIRMINA-409:
--------------------------------
Fix Version/s: (was: 2.0.0-M1)
1.0.5
Assignee: Trustin Lee
Affects Version/s: 1.0.0
1.0.1
1.0.2
1.0.3
1.1.0
1.0.4
> Duplicate syncronization in SocketAcceptor.startupWorker()
> ----------------------------------------------------------
>
> Key: DIRMINA-409
> URL: https://issues.apache.org/jira/browse/DIRMINA-409
> Project: MINA
> Issue Type: Bug
> Components: Transport
> Affects Versions: 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.1.0, 1.0.4, 1.1.1
> Reporter: im-james
> Assignee: Trustin Lee
> Priority: Trivial
> Fix For: 1.0.5, 1.1.2
>
>
> In org.apache.mina.transport.socket.nio.SocketAcceptor, the method reads like that:
> private synchronized void startupWorker() throws IOException {
> synchronized (lock) {
> if (worker == null) {
> selector = Selector.open();
> worker = new Worker();
> executor.execute(new NamePreservingRunnable(worker));
> }
> }
> }
> The 'synchronized' keyword of the method is, in my opinion, superfluous and should be removed.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.