You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Gilles (JIRA)" <ji...@apache.org> on 2012/10/22 16:08:15 UTC

[jira] [Commented] (MATH-757) ResizableDoubleArray is not thread-safe yet has some synch. methods

    [ https://issues.apache.org/jira/browse/MATH-757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13481377#comment-13481377 ] 

Gilles commented on MATH-757:
-----------------------------

What's the status of this issue?

Is the class in a consistent state?
Should this class really be advertized as thread-safe?

Primarily, it's a utility for CM's internal use; currently, it's used in
* ExponentialDistribution (as a local variable used only at class initialization)
* DescriptiveStatistics (as a private field)
with no need of thread-safety.

If the answer to the above is "no", could we just drop all "synchronized" keywords and be done with this issue?

                
> ResizableDoubleArray is not thread-safe yet has some synch. methods
> -------------------------------------------------------------------
>
>                 Key: MATH-757
>                 URL: https://issues.apache.org/jira/browse/MATH-757
>             Project: Commons Math
>          Issue Type: Bug
>            Reporter: Sebb
>
> ResizableDoubleArray has several synchronised methods, but is not thread-safe, because class variables are not always accessed using the lock.
> Is the class supposed to be thread-safe?
> If so, all accesses (read and write) need to be synch.
> If not, the synch. qualifiers could be dropped.
> In any case, the protected fields need to be made private.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira