Commons Math
  1. Commons Math
  2. MATH-257

FastFourierTransformer.MultiDimensionalComplexMatrix.get() not consistent with FastFourierTransformer.MultiDimensionalComplexMatrix.set()

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0
    • Labels:
      None

      Description

      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

        Activity

        Sebb created issue -
        Hide
        Luc Maisonobe added a comment -

        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

        Show
        Luc Maisonobe added a comment - 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
        Luc Maisonobe made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Luc Maisonobe [ luc ]
        Henri Yandell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Sebb made changes -
        Fix Version/s 2.0 [ 12312686 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1h 8m 1 Luc Maisonobe 05/Apr/09 19:43
        Resolved Resolved Closed Closed
        17d 7h 42m 1 Henri Yandell 23/Apr/09 03:26

          People

          • Assignee:
            Luc Maisonobe
            Reporter:
            Sebb
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development