Commons Math
  1. Commons Math
  2. MATH-1050

Deprecate "pow(int, long)" and "pow(long,long)" in "ArithmeticUtils"

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 3.2
    • Fix Version/s: 4.0
    • Labels:

      Description

      Those two methods uses a "long" for the exponent, but overflow will occur when the exponent is larger than 31 (except for the trivial case where the base is 1).

      See also this thread on the dev ML.

        Activity

        Gilles created issue -
        Hide
        Gilles added a comment -

        Deprecation introduced in revision 1537279.

        Methods to be removed before releasing the next major version.

        Show
        Gilles added a comment - Deprecation introduced in revision 1537279. Methods to be removed before releasing the next major version.
        Hide
        Thomas Neidhart added a comment -

        Added changelog entry in r1566074.

        Show
        Thomas Neidhart added a comment - Added changelog entry in r1566074.
        Hide
        Luc Maisonobe added a comment -

        This issue was fixed a long time ago.

        Show
        Luc Maisonobe added a comment - This issue was fixed a long time ago.
        Luc Maisonobe made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Gilles added a comment -

        Hi Luc.

        ^^^^
        I think that the issue should be kept open until the method is
        actually deleted.

        Regards,
        Gilles

        Show
        Gilles added a comment - Hi Luc. ^^^^ I think that the issue should be kept open until the method is actually deleted. Regards, Gilles
        Hide
        Luc Maisonobe added a comment -

        Reopening, waiting for the actual deletion of the methods.

        Show
        Luc Maisonobe added a comment - Reopening, waiting for the actual deletion of the methods.
        Luc Maisonobe made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Luc Maisonobe made changes -
        Fix Version/s 3.3 [ 12324600 ]
        Hide
        Thomas Neidhart added a comment -

        Removed methods in commit 745d383af12137ccbcbe1f3cb4c9db73f87a66ca.

        Show
        Thomas Neidhart added a comment - Removed methods in commit 745d383af12137ccbcbe1f3cb4c9db73f87a66ca.
        Thomas Neidhart made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        112d 20h 40m 1 Luc Maisonobe 20/Feb/14 09:23
        Resolved Resolved Reopened Reopened
        1h 7m 1 Luc Maisonobe 20/Feb/14 10:31
        Reopened Reopened Resolved Resolved
        361d 12h 43m 1 Thomas Neidhart 16/Feb/15 23:14

          People

          • Assignee:
            Gilles
            Reporter:
            Gilles
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development