You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2022/04/17 17:27:20 UTC

[GitHub] [commons-statistics] aherbert commented on pull request #33: Math.min() in favor of manual calculations.

aherbert commented on PR #33:
URL: https://github.com/apache/commons-statistics/pull/33#issuecomment-1100919937

   Math.min has performance implications when used with float values. In this case a clipped distribution does occur for extreme values. Here math min would use a conversion to raw bits and is slower. The simple less than operator is fine. It also allows code coverage checks to show the code path is executed and thus is required.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org