commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Luc Maisonobe (JIRA)" <j...@apache.org>
Subject [jira] Resolved: (MATH-257) FastFourierTransformer.MultiDimensionalComplexMatrix.get() not consistent with FastFourierTransformer.MultiDimensionalComplexMatrix.set()
Date Sun, 05 Apr 2009 18:45:13 GMT

     [ https://issues.apache.org/jira/browse/MATH-257?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Luc Maisonobe resolved MATH-257.
--------------------------------

    Resolution: Fixed
      Assignee: Luc Maisonobe

fixed in subversion repository as of r762131.
I took the opportunity to also simplify the initialization of the dimensionSize array, using
a two passes algorithm.
The inconsistency by itself was removed by having bot set and get be no-op for null vectors

> FastFourierTransformer.MultiDimensionalComplexMatrix.get() not consistent with FastFourierTransformer.MultiDimensionalComplexMatrix.set()
> -----------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: MATH-257
>                 URL: https://issues.apache.org/jira/browse/MATH-257
>             Project: Commons Math
>          Issue Type: Bug
>            Reporter: Sebb
>            Assignee: Luc Maisonobe
>
> FastFourierTransformer.MultiDimensionalComplexMatrix.get() is not consistent with FastFourierTransformer.MultiDimensionalComplexMatrix.set().
> The set() method does not allow a null parameter to get past the first check.
> The get() method allows a null parameter provided dimensionSize.length <= 1.
> This seems wrong, both because it is inconsistent and because it may allow an NPE later
if dimensionSize.length == 1

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


Mime
View raw message